From 6f7fc70c4c4b48992f10b7e26c153724c70b36aa Mon Sep 17 00:00:00 2001
From: MAILLAL Anas <Anas.Maillal@bordeaux-inp.fr>
Date: Tue, 29 Apr 2025 18:50:25 +0200
Subject: [PATCH] didn't use the correct function for seting ids, test fixed

---
 .../myinpulse/SharedApiServiceTest.java        | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 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..7eb2b60 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;
@@ -344,7 +344,9 @@ public class SharedApiServiceTest {
                         "Content V2 (Middle)",
                         LocalDateTime.now().minusDays(2), // Middle date, before filter
                         sharedIdReference);
-        sectionCellService.addNewSectionCell(middleVersion);
+        middleVersion = sectionCellService.addNewSectionCell(middleVersion);
+        sectionCellService.updateSectionCellReferenceId(
+                middleVersion.getIdSectionCell(), sharedIdReference);
 
         SectionCell latestBeforeFilter =
                 getTestSectionCell(
@@ -353,7 +355,9 @@ public class SharedApiServiceTest {
                         "Content V3 (Latest Before Filter)",
                         LocalDateTime.now().minusDays(1), // Latest date before filter
                         sharedIdReference);
-        sectionCellService.addNewSectionCell(latestBeforeFilter);
+        latestBeforeFilter = sectionCellService.addNewSectionCell(latestBeforeFilter);
+        sectionCellService.updateSectionCellReferenceId(
+                latestBeforeFilter.getIdSectionCell(), sharedIdReference);
 
         SectionCell futureVersion =
                 getTestSectionCell(
@@ -362,7 +366,9 @@ public class SharedApiServiceTest {
                         "Content V4 (Future - Should Be Excluded)",
                         LocalDateTime.now().plusDays(1), // Date is AFTER the filter
                         sharedIdReference);
-        sectionCellService.addNewSectionCell(futureVersion);
+        futureVersion = sectionCellService.addNewSectionCell(futureVersion);
+        sectionCellService.updateSectionCellReferenceId(
+                futureVersion.getIdSectionCell(), sharedIdReference);
 
         // --- Create other SectionCells that should NOT be included (different sectionId or
         // project) ---
@@ -386,13 +392,13 @@ public class SharedApiServiceTest {
         List<SectionCell> resultList = TestUtils.toList(result);
 
         // 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");
+                latestBeforeFilter.getIdReference(), resultList.get(2).getIdReference(), "The 2");*/
 
         assertEquals(1, resultList.size());
         // Verify that the returned cell is the 'latestBeforeFilter' cell