fix : Update test files to use UUID instead of Long for postDTO

Author: mplmihir

hexfeed-backend/src/test/java/com/hexfeed/controller/FeedControllerSimpleTest.java

@@ -26,6 +26,7 @@
 import java.util.Collections;
 import java.util.List;
 import java.util.Map;
+import java.util.UUID;
 
 import static org.junit.jupiter.api.Assertions.*;
 import static org.mockito.ArgumentMatchers.*;
@@ -330,9 +331,11 @@ private FeedResponse createSampleFeedResponse() {
 
     private PostDTO createSamplePost(Long id, String content) {
         PostDTO post = new PostDTO();
-        post.setId(id);
+        UUID postId = UUID.randomUUID();
+        UUID userId = UUID.randomUUID();
+        post.setPostId(postId);
         post.setContent(content);
-        post.setUserId(id);
+        post.setUserId(userId);
         post.setTimestamp(LocalDateTime.now());
         post.setType("TEXT");
         post.setVisibility("public");

hexfeed-backend/src/test/java/com/hexfeed/model/dto/FeedDTOsTest.java

@@ -13,6 +13,7 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Set;
+import java.util.UUID;
 
 import static org.junit.jupiter.api.Assertions.*;
 
@@ -78,8 +79,10 @@ void testFeedRequestValidation() {
     @DisplayName("PostDTO should populate all fields correctly")
     void testPostDTOPopulation() {
         PostDTO post = new PostDTO();
-        post.setId(1L);
-        post.setUserId(123L);
+        UUID postId = UUID.randomUUID();
+        UUID userId = UUID.randomUUID();
+        post.setPostId(postId);
+        post.setUserId(userId);
         post.setUsername("testuser");
         post.setDisplayName("Test User");
         post.setContent("This is a test post");
@@ -93,8 +96,8 @@ void testPostDTOPopulation() {
         post.setVisibility("PUBLIC");
 
         // Test basic fields
-        assertEquals(1L, post.getId());
-        assertEquals(123L, post.getUserId());
+        assertEquals(postId, post.getPostId());
+        assertEquals(userId, post.getUserId());
         assertEquals("testuser", post.getUsername());
         assertEquals("This is a test post", post.getContent());
         assertEquals("872830828ffffff", post.getHexId());
@@ -170,13 +173,13 @@ void testPaginationInfoDerivedFields() {
     void testFeedResponseBuilder() {
         // Create sample posts
         PostDTO post1 = new PostDTO();
-        post1.setId(1L);
+        post1.setPostId(UUID.randomUUID());
         post1.setUsername("user1");
         post1.setContent("First post");
         post1.setHexId("872830828ffffff");
 
         PostDTO post2 = new PostDTO();
-        post2.setId(2L);
+        post2.setPostId(UUID.randomUUID());
         post2.setUsername("user2");
         post2.setContent("Second post");
         post2.setHexId("872830829ffffff");

hexfeed-backend/src/test/java/com/hexfeed/service/FeedAggregationServiceIntegrationTest.java

@@ -281,16 +281,16 @@ void shouldVerifyPaginationWorks() {
         assertTrue(page2Response.getPagination().isHasPrevious(), "Page 2 should have previous");
 
         // Verify different posts on different pages
-        Set<Long> page1PostIds = page1Response.getPosts().stream()
-            .map(PostDTO::getId)
+        Set<UUID> page1PostIds = page1Response.getPosts().stream()
+            .map(PostDTO::getPostId)
             .collect(Collectors.toSet());
 
-        Set<Long> page2PostIds = page2Response.getPosts().stream()
-            .map(PostDTO::getId)
+        Set<UUID> page2PostIds = page2Response.getPosts().stream()
+            .map(PostDTO::getPostId)
             .collect(Collectors.toSet());
 
         // No overlap between pages (different posts)
-        Set<Long> intersection = new HashSet<>(page1PostIds);
+        Set<UUID> intersection = new HashSet<>(page1PostIds);
         intersection.retainAll(page2PostIds);
         assertTrue(intersection.isEmpty(), "Pages should have different posts (no overlap)");
 
@@ -343,8 +343,8 @@ void shouldTestCacheHitMissScenarios() {
         assertEquals(response1.getPosts().size(), response2.getPosts().size(), "Both responses should have same number of posts");
 
         // Verify post IDs are the same (same data)
-        List<Long> postIds1 = response1.getPosts().stream().map(PostDTO::getId).sorted().collect(Collectors.toList());
-        List<Long> postIds2 = response2.getPosts().stream().map(PostDTO::getId).sorted().collect(Collectors.toList());
+        List<UUID> postIds1 = response1.getPosts().stream().map(PostDTO::getPostId).sorted(UUID::compareTo).collect(Collectors.toList());
+        List<UUID> postIds2 = response2.getPosts().stream().map(PostDTO::getPostId).sorted(UUID::compareTo).collect(Collectors.toList());
         assertEquals(postIds1, postIds2, "Both responses should have the same posts");
 
         // Test cache performance (second call should be faster or similar)