Skip to content

Commit 67187a7

Browse files
committed
chore: aggiornato TODO
1 parent e14cb82 commit 67187a7

File tree

10 files changed

+34
-9
lines changed

10 files changed

+34
-9
lines changed

00_problems/src/main/java/io/doubleloop/problems/BirthdayService.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,12 @@
22

33
import jakarta.mail.MessagingException;
44
import org.springframework.stereotype.Component;
5+
import org.springframework.stereotype.Service;
56

67
import java.io.IOException;
78
import java.time.LocalDate;
89

9-
@Component
10+
@Service
1011
public class BirthdayService {
1112

1213
private final EmployeeRepository employeeRepository;

01_driver_implicit/TODO.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
test siano verdi.
1414
- [ ] Rimuovi l'annotazione `@Disabled`
1515
da [UserControllerNoSpringTest](src/test/java/io/doubleloop/driverimplicit/UserControllerNoSpringTest.java#L9) e
16-
verifica che tutti i test siano verdi. Un test è fallito, perché?
16+
verifica che i test siano verdi tranne uno, non correggerlo, perché non é verde?
1717
- [ ] Sposta i file nei rispettivi package/folder in base alla loro responsabilità.
1818
- [ ] Rimuovi l'annotazione `@Disabled`
1919
da [DependencyRulesTest](src/test/java/io/doubleloop/driverimplicit/DependencyRulesTest.java#L9) e

02_driver_explicit/TODO.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
- [ ] Assicurati che il field [userService](src/main/java/io/doubleloop/driverexplicit/UserController.java#L16) sia
1212
di tipo `UserService` (interface).
1313
- [ ] Aggiungi un field `userService`
14-
in [UserControllerTest](src/test/java/io/doubleloop/driverexplicit/UserControllerTest.java) e annotalo con `@Mock`.
14+
in [UserControllerTest](src/test/java/io/doubleloop/driverexplicit/UserControllerTest.java) e annotalo con `@MockBean`.
1515
- [ ] Aggiungi una `when(userService...).thenReturns(...)` expression in ogni test
1616
di [UserControllerTest](src/test/java/io/doubleloop/driverexplicit/UserControllerTest.java) per pilotare lo stub.
1717
- [ ] Verifica che tutti i test
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
11
package io.doubleloop.driverexplicit;
22

33
import org.springframework.data.mongodb.repository.MongoRepository;
4+
import org.springframework.stereotype.Repository;
45

56
import java.util.Optional;
67

8+
@Repository
79
public interface UserRepository extends MongoRepository<User, String> {
810
Optional<User> findByEmail(String email);
911
}

04_driven_pull/TODO.md

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,21 @@
77
- [ ] Rinomina la classe
88
infrastrutturale [OperationRepository](src/main/java/io/doubleloop/drivenpull/OperationRepository.java) aggiungendo un
99
prefisso che identifica la tecnologia utilizzata.
10-
- [ ] Estrai l'interfaccia `OperationRepository` di dominio ed usala in [DefaultDailyBalanceService](src/main/java/io/doubleloop/drivenpull/DefaultDailyBalanceService.java) al posto della relativa classe.
10+
- [ ] Estrai l'interfaccia `OperationRepository` di dominio e usala
11+
in [DefaultDailyBalanceService](src/main/java/io/doubleloop/drivenpull/DefaultDailyBalanceService.java) al posto della
12+
relativa classe.
1113
- [ ] Rinomina la classe
1214
infrastrutturale [ExchangeProvider](src/main/java/io/doubleloop/drivenpull/ExchangeProvider.java) aggiungendo un
1315
prefisso che identifica la tecnologia utilizzata.
14-
- [ ] Estrai l'interfaccia `ExchangeProvider` di dominio ed usala in [DefaultDailyBalanceService](src/main/java/io/doubleloop/drivenpull/DefaultDailyBalanceService.java) al posto della relativa classe.
15-
- [ ] Elimina l'uso del container MongoDB e del vero repository dalla
16-
suite [DefaultDailyBalanceServiceTest](src/test/java/io/doubleloop/drivenpull/DefaultDailyBalanceServiceTest.java).
16+
- [ ] Estrai l'interfaccia `ExchangeProvider` di dominio e usala
17+
in [DefaultDailyBalanceService](src/main/java/io/doubleloop/drivenpull/DefaultDailyBalanceService.java) al posto della
18+
relativa classe.
19+
- [ ] Elimina l'uso di `@Testcontainers`, del container MongoDB e del vero repository dalla
20+
suite [DefaultDailyBalanceServiceTest](src/test/java/io/doubleloop/drivenpull/DefaultDailyBalanceServiceTest.java)
21+
attraverso l'uso di `@MockBean`.
22+
- [ ] Elimina l'uso di `@SpringBootTest` dalla
23+
suite [DefaultDailyBalanceServiceTest](src/test/java/io/doubleloop/drivenpull/DefaultDailyBalanceServiceTest.java)
24+
attraverso l'uso di `Mockito.mock(class)`.
1725
- [ ] Sposta i file nei rispettivi package/folder in base alla loro responsabilità.
1826
- [ ] Rimuovi l'annotazione `@Disabled`
1927
da [DependencyRulesTest](src/test/java/io/doubleloop/drivenpull/DependencyRulesTest.java#L9) e
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
package io.doubleloop.drivenpull;
22

33
import org.springframework.data.mongodb.repository.MongoRepository;
4+
import org.springframework.stereotype.Repository;
45

56
import java.time.LocalDate;
67
import java.util.List;
78

9+
@Repository
810
public interface OperationRepository extends MongoRepository<Operation, String> {
911
List<Operation> findByUserIdAndDate(String userId, LocalDate date);
1012
}

05_driven_push/TODO.md

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,21 @@
22

33
- [ ] Leggi il codice di [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java).
44
- [ ] Aggiungi una nuova classe infrastrutturale `SpringRabbitMqPostNotifier` con il codice relativo alla pubblicazione
5-
del `PostMessageCommand` preso da [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java)
5+
del `PostMessageCommand` preso
6+
da [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java#L34-L35)
67
- [ ] Utilizza `SpringRabbitMqPostNotifier`
78
in [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java) al posto di `RabbitTemplate` e
89
`Jsonb`.
910
- [ ] Estrai un'interfaccia `PostNotifier`.
1011
- [ ] Utilizza `PostNotifier` in [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java) al
1112
posto di `SpringRabbitMqPostNotifier`.
13+
- [ ] Elimina l'uso di `@Testcontainers`, del container MongoDB, - del vero repository dalla e l'uso di
14+
`@SpringBootTest` dalla
15+
suite [DefaultDailyBalanceServiceTest](src/test/java/io/doubleloop/drivenpush/DefaultPostServiceTest.java) attraverso
16+
l'uso di `Mockito.mock(class)`.
1217
- [ ] Sposta i file nei rispettivi package/folder in base alla loro responsabilità.
1318
- [ ] Rimuovi l'annotazione `@Disabled`
1419
da [DependencyRulesTest](src/test/java/io/doubleloop/drivenpush/DependencyRulesTest.java#L9) e
1520
verifica che i test siano verdi.
21+
22+
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
package io.doubleloop.drivenpush;
22

33
import org.springframework.data.mongodb.repository.MongoRepository;
4+
import org.springframework.stereotype.Repository;
45

6+
@Repository
57
public interface SpringMongoUserRepository extends MongoRepository<User, String>, UserRepository {
68
}

05_driven_push/src/test/java/io/doubleloop/drivenpush/DefaultPostServiceTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,9 @@ class DefaultPostServiceTest {
3333

3434
@Autowired
3535
private SpringMongoUserRepository userRepository;
36+
3637
@Autowired
37-
private PostService postService;
38+
private DefaultPostService postService;
3839

3940
@BeforeAll
4041
static void setup() {

07_bank/src/main/java/io/doubleloop/bank/adapter/SpringMongoAccountRepository.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22

33
import io.doubleloop.bank.domain.Account;
44
import org.springframework.data.mongodb.repository.MongoRepository;
5+
import org.springframework.stereotype.Repository;
56

7+
@Repository
68
public interface SpringMongoAccountRepository extends MongoRepository<Account, String> {
79
}

0 commit comments

Comments
 (0)