From a4e13b0f0ac8945f699bb77a52583d16a47b5917 Mon Sep 17 00:00:00 2001
From: MAILLAL Anas <Anas.Maillal@bordeaux-inp.fr>
Date: Tue, 29 Apr 2025 13:00:18 +0200
Subject: [PATCH] Revert "forgot to comment out said test"

This reverts commit b116771375cd61c86d842c35ff2ca7fa5a2dea82.
---
 .../myinpulse/SharedApiServiceTest.java       | 109 +++++++++---------
 1 file changed, 52 insertions(+), 57 deletions(-)

diff --git a/MyINPulse-back/src/test/java/enseirb/myinpulse/SharedApiServiceTest.java b/MyINPulse-back/src/test/java/enseirb/myinpulse/SharedApiServiceTest.java
index 3dc5b77..281870c 100644
--- a/MyINPulse-back/src/test/java/enseirb/myinpulse/SharedApiServiceTest.java
+++ b/MyINPulse-back/src/test/java/enseirb/myinpulse/SharedApiServiceTest.java
@@ -309,7 +309,7 @@ public class SharedApiServiceTest {
      * when the user is authorized and matching cells exist.
      * Verifies that only the correct cells are returned.
      */
-    /*@Test*/
+    @Test
     // Commenting out failing test
     void testGetSectionCells_Authorized_Found() {
         Long targetSectionId = 1L;
@@ -319,61 +319,63 @@ public class SharedApiServiceTest {
         // Creating versions of the SAME SectionCell (share the same idReference)
 
         // the first version. This will get a GENERATED idReference.
-        SectionCell firstVersion =
-                getTestSectionCell(
-                        staticAuthorizedProject,
-                        targetSectionId,
-                        "Content V1 (Oldest)",
-                        LocalDateTime.now().minusDays(3) // Oldest date
-                        );
+        SectionCell firstVersion = getTestSectionCell(
+            staticAuthorizedProject,
+            targetSectionId,
+            "Content V1 (Oldest)",
+            LocalDateTime.now().minusDays(3) // Oldest date
+        );
         sectionCellService.addNewSectionCell(firstVersion);
 
         Long sharedIdReference = firstVersion.getIdReference();
-        assertNotNull(
-                sharedIdReference,
-                "idReference should be generated after saving the first version");
+        assertNotNull(sharedIdReference, "idReference should be generated after saving the first version");
         System.out.println("Generated sharedIdReference: " + sharedIdReference);
 
+
         // Create subsequent versions and MANUALLY set the SAME idReference.
         // These represent updates to the cell identified by sharedIdReference.
 
-        SectionCell middleVersion =
-                getTestSectionCell(
-                        staticAuthorizedProject,
-                        targetSectionId,
-                        "Content V2 (Middle)",
-                        LocalDateTime.now().minusDays(2), // Middle date, before filter
-                        sharedIdReference);
+        SectionCell middleVersion = getTestSectionCell(
+            staticAuthorizedProject,
+            targetSectionId,
+            "Content V2 (Middle)",
+            LocalDateTime.now().minusDays(2), // Middle date, before filter
+            sharedIdReference
+        );
         sectionCellService.addNewSectionCell(middleVersion);
 
-        SectionCell latestBeforeFilter =
-                getTestSectionCell(
-                        staticAuthorizedProject,
-                        targetSectionId,
-                        "Content V3 (Latest Before Filter)",
-                        LocalDateTime.now().minusDays(1), // Latest date before filter
-                        sharedIdReference);
+
+        SectionCell latestBeforeFilter = getTestSectionCell(
+            staticAuthorizedProject,
+            targetSectionId,
+            "Content V3 (Latest Before Filter)",
+            LocalDateTime.now().minusDays(1), // Latest date before filter
+            sharedIdReference
+        );
         sectionCellService.addNewSectionCell(latestBeforeFilter);
 
-        SectionCell futureVersion =
-                getTestSectionCell(
-                        staticAuthorizedProject,
-                        targetSectionId,
-                        "Content V4 (Future - Should Be Excluded)",
-                        LocalDateTime.now().plusDays(1), // Date is AFTER the filter
-                        sharedIdReference);
+
+        SectionCell futureVersion = getTestSectionCell(
+            staticAuthorizedProject,
+            targetSectionId,
+            "Content V4 (Future - Should Be Excluded)",
+            LocalDateTime.now().plusDays(1), // Date is AFTER the filter
+            sharedIdReference
+        );
         sectionCellService.addNewSectionCell(futureVersion);
 
-        // --- Create other SectionCells that should NOT be included (different sectionId or
-        // project) ---
+
+        // --- Create other SectionCells that should NOT be included (different sectionId or project) ---
 
         // Cell in a different section ID
         sectionCellService.addNewSectionCell(
-                getTestSectionCell(
-                        staticAuthorizedProject,
-                        99L, // Different sectionId
-                        "Content in Different Section",
-                        LocalDateTime.now()));
+            getTestSectionCell(
+                staticAuthorizedProject,
+                99L, // Different sectionId
+                "Content in Different Section",
+                LocalDateTime.now()
+            )
+        );
 
         // Act
         Iterable<SectionCell> result =
@@ -387,30 +389,20 @@ public class SharedApiServiceTest {
 
         // Assert
 
-        assertEquals(
-                latestBeforeFilter.getIdReference(), resultList.get(0).getIdReference(), "The 0");
-        assertEquals(
-                latestBeforeFilter.getIdReference(), resultList.get(1).getIdReference(), "The 1");
-        assertEquals(
-                latestBeforeFilter.getIdReference(), resultList.get(2).getIdReference(), "The 2");
+        assertEquals(latestBeforeFilter.getIdReference(), resultList.get(0).getIdReference(), "The 0");
+        assertEquals(latestBeforeFilter.getIdReference(), resultList.get(1).getIdReference(), "The 1");
+        assertEquals(latestBeforeFilter.getIdReference(), resultList.get(2).getIdReference(), "The 2"); 
 
         assertEquals(1, resultList.size());
         // Verify that the returned cell is the 'latestBeforeFilter' cell
         // Comparing by idSectionCell is a good way to verify the exact entity
-        assertEquals(
-                latestBeforeFilter.getIdSectionCell(),
-                resultList.get(0).getIdSectionCell(),
-                "The returned SectionCell should be the one with the latest modification date before the filter.");
+        assertEquals(latestBeforeFilter.getIdSectionCell(), resultList.get(0).getIdSectionCell(),
+        "The returned SectionCell should be the one with the latest modification date before the filter.");
 
         // Also assert the idReference and content
-        assertEquals(
-                sharedIdReference,
-                resultList.get(0).getIdReference(),
-                "The returned cell should have the shared idReference.");
-        assertEquals(
-                "Content V3 (Latest Before Filter)",
-                resultList.get(0).getContentSectionCell(),
-                "The returned cell should have the correct content.");
+        assertEquals(sharedIdReference, resultList.get(0).getIdReference(), "The returned cell should have the shared idReference.");
+        assertEquals("Content V3 (Latest Before Filter)", resultList.get(0).getContentSectionCell(), "The returned cell should have the correct content.");
+
     }
 
     /*
@@ -696,6 +688,8 @@ public class SharedApiServiceTest {
         assertEquals(HttpStatus.UNAUTHORIZED, exception.getStatusCode());
     }
 
+    
+
     /*
 
              _____     _ _          _
@@ -718,6 +712,7 @@ public class SharedApiServiceTest {
      * probably and look at peer tests to see what they have done but for now
      * I pushed this half-human code.
      */
+    
 
     /*
      * Tests retrieving the most recent section cell for each unique idReference