chore: aggiunti TODO esercizio 5

Author: matteobaglini

03_driver_reactive/src/test/java/io/doubleloop/driverreactive/DependencyRulesTest.java

@@ -6,7 +6,7 @@
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
-@Disabled("Run after moving classes into various packages")
+@Disabled
 class DependencyRulesTest {
 
   private final JavaClasses classes = new ClassFileImporter().importPackages(DependencyRulesTest.class.getPackageName());

04_driven_pull/src/test/java/io/doubleloop/drivenpull/DependencyRulesTest.java

@@ -6,7 +6,7 @@
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
-@Disabled("Run after moving classes into various packages")
+@Disabled
 class DependencyRulesTest {
 
   private final JavaClasses classes = new ClassFileImporter().importPackages(DependencyRulesTest.class.getPackageName());

05_driven_push/TODO.md

@@ -1,5 +1,15 @@
 ## TODO:
-- [ ] rinomina le classi infrastrutturali 
-- [ ] estrai le interfacce usando nome di dominio
-- [ ] elimina l'uso di mongodb e del vero repository dalla suite DefaultDailyBalanceServiceTest
-- [ ] sposta classi e interfacce nelle rispettive cartelle
+
+- [ ] Leggi il codice di [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java).
+- [ ] Aggiungi una nuova classe infrastrutturale `SpringRabbitMqPostNotifier` con il codice relativo alla pubblicazione
+  di `PostMessageCommand` preso da [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java)
+- [ ] Utilizza `SpringRabbitMqPostNotifier`
+  in [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java) al posto di `RabbitTemplate` e
+  `Jsonb`.
+- [ ] Estrai un'interfaccia `PostNotifier`.
+- [ ] Utilizza `PostNotifier` in [DefaultPostService](src/main/java/io/doubleloop/drivenpush/DefaultPostService.java) al
+  posto di `SpringRabbitMqPostNotifier`.
+- [ ] Sposta i file nei rispettivi package/folder in base alla responsabilità.
+- [ ] Rimuovi l'annotazione `@Disabled`
+  da [DependencyRulesTest](src/test/java/io/doubleloop/drivenpush/DependencyRulesTest.java#L9) e
+  verifica che i test siano verdi.

05_driven_push/src/main/java/io/doubleloop/drivenpush/DefaultPostService.java

@@ -26,11 +26,13 @@ public void postMessage(PostMessageCommand command) {
     if (found.isEmpty()) {
       throw new IllegalArgumentException("User not found");
     }
+
     final var user = found.get();
     if (user.isBlocked()) {
       throw new IllegalArgumentException("User is blocked");
     }
 
-    rabbitTemplate.convertAndSend(exchangeName, "posts.%s.new".formatted(command.getUserId()), jsonb.toJson(command));
+    final var message = jsonb.toJson(command);
+    rabbitTemplate.convertAndSend(exchangeName, "posts.%s.new".formatted(command.getUserId()), message);
   }
 }

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

@@ -6,7 +6,7 @@
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
-@Disabled("Run after moving classes into various packages")
+@Disabled
 class DependencyRulesTest {
 
   private final JavaClasses classes = new ClassFileImporter().importPackages(DependencyRulesTest.class.getPackageName());