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 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