Fixed: contributor names retrieved in dataset version summaries

Author: GPortas

src/main/java/edu/harvard/iq/dataverse/engine/command/impl/GetDatasetVersionSummariesCommand.java

@@ -30,8 +30,9 @@ public GetDatasetVersionSummariesCommand(DataverseRequest request, Dataset datas
      * <p>
      * This method first checks if the user has permission to view unpublished
      * versions of the dataset. It then fetches the appropriate {@link DatasetVersion}s,
-     * respecting pagination parameters (limit and offset), and converts each
-     * version into a {@link DatasetVersionSummary}.
+     * respecting pagination parameters (limit and offset). Each version is then
+     * enriched with its contributor names before being converted into a
+     * {@link DatasetVersionSummary}.
      *
      * @param ctxt The command context.
      * @return A list of {@link DatasetVersionSummary} objects.
@@ -52,6 +53,10 @@ public List<DatasetVersionSummary> executeCommand(CommandContext ctxt) {
                 true
         );
 
+        for (DatasetVersion version : versions) {
+            version.setContributorNames(ctxt.datasetVersion().getContributorsNames(version));
+        }
+
         return versions.stream()
                 .map(DatasetVersionSummary::from)
                 .flatMap(Optional::stream)

src/test/java/edu/harvard/iq/dataverse/api/DatasetsIT.java

@@ -6537,8 +6537,10 @@ public void testSummaryDatasetVersionsDifferencesAPI() throws InterruptedExcepti
                 .body("totalCount", is(2))
                 .body("data.size()", is(2))
                 .body("data[1].versionNumber", equalTo("1.0"))
+                .body("data[1].contributors", notNullValue())
                 .body("data[1].summary", equalTo("firstPublished"))
                 .body("data[0].versionNumber", equalTo("DRAFT"))
+                .body("data[0].contributors", notNullValue())
                 .body("data[0].summary.'Citation Metadata'.Author.added", equalTo(2))
                 .body("data[0].summary.'Citation Metadata'.Subject.added", equalTo(2))
                 .body("data[0].summary.'Additional Citation Metadata'.changed", equalTo(0))

src/test/java/edu/harvard/iq/dataverse/engine/command/impl/GetDatasetVersionSummariesCommandTest.java

@@ -144,13 +144,17 @@ void execute_should_pass_pagination_parameters_correctly() throws CommandExcepti
     }
 
     @Test
-    @DisplayName("execute should correctly convert DatasetVersion list to DatasetVersionSummary list")
-    void execute_should_convert_versions_to_summaries() throws CommandException {
+    @DisplayName("execute should enrich contributors and convert versions to summaries")
+    void execute_should_enrich_contributors_and_convert_to_summaries() throws CommandException {
         // Arrange
         setupMocksForSuccessfulExecution();
         when(versionServiceMock.findVersions(anyLong(), any(), any(), anyBoolean(), anyBoolean()))
                 .thenReturn(List.of(versionMock));
 
+        // Arrange: Mock contributor names retrieval
+        String expectedContributors = "Contributor";
+        when(versionServiceMock.getContributorsNames(versionMock)).thenReturn(expectedContributors);
+
         // Arrange: Prepare a dummy summary object
         DatasetVersionSummary dummySummary = new DatasetVersionSummary(1L, "V1", null, null, null, null);
 
@@ -165,6 +169,10 @@ void execute_should_convert_versions_to_summaries() throws CommandException {
             List<DatasetVersionSummary> result = command.execute(contextMock);
 
             // Assert
+            verify(versionServiceMock).getContributorsNames(versionMock);
+            verify(versionMock).setContributorNames(expectedContributors);
+
+            // Verify final conversion
             assertThat(result)
                     .isNotNull()
                     .hasSize(1)