VSB-TUO/leaving property webui.browse.vocabularies.disabled empty for test (#1202)

* leaving property webui.browse.vocabularies.disabled empty

* added srsc to property for test findAll

* set 6 to 5 size

* set test back to original, set size from 6 to 5

* set back size from 5 to 6

* use 6 element instead of 5 because of the config

* added proeprty

* fix checkstyle

Author: Paurikova2

dspace-api/src/test/data/dspaceFolder/config/local.cfg

@@ -383,3 +383,6 @@ info.app.name = DSpace at My University
 # SubmissionFormsControllerIT.java#userChoiceAnotherLanguageTest
 # SubmissionFormsControllerIT.java#defaultLanguageTest
 choices.plugin.dc.contributor.author =
+
+# Leaving this property empty (as below) means all vocabularies, including 'srsc', are enabled for browsing.
+webui.browse.vocabularies.disabled =

dspace-server-webapp/src/test/java/org/dspace/app/rest/BrowsesResourceControllerIT.java

@@ -58,32 +58,49 @@ public class BrowsesResourceControllerIT extends AbstractControllerIntegrationTe
 
     @Test
     public void findAll() throws Exception {
-        //When we call the root endpoint
+        // Save the original configuration value
+        String[] originalValue = configurationService.getArrayProperty("webui.browse.vocabularies.disabled");
+        try {
+            // Override the configuration specifically for this test to use 'srsc'
+            configurationService.setProperty("webui.browse.vocabularies.disabled", "srsc");
+            // Clear cache to ensure the configuration change takes effect
+            metadataAuthorityService.clearCache();
+            //When we call the root endpoint
         getClient().perform(get("/api/discover/browses"))
-                   //The status has to be 200 OK
-                   .andExpect(status().isOk())
-                   //We expect the content type to be "application/hal+json;charset=UTF-8"
-                   .andExpect(content().contentType(contentType))
+                //The status has to be 200 OK
+                .andExpect(status().isOk())
+                //We expect the content type to be "application/hal+json;charset=UTF-8"
+                .andExpect(content().contentType(contentType))
 
-                   //Our default Discovery config has 6 browse indexes, so we expect this to be reflected in the page
-                   // object
-                   .andExpect(jsonPath("$.page.size", is(20)))
-                   .andExpect(jsonPath("$.page.totalElements", is(5)))
-                   .andExpect(jsonPath("$.page.totalPages", is(1)))
-                   .andExpect(jsonPath("$.page.number", is(0)))
+                //Our default Discovery config has 6 browse indexes, so we expect this to be reflected in the page
+                // object
+                .andExpect(jsonPath("$.page.size", is(20)))
+                .andExpect(jsonPath("$.page.totalElements", is(5)))
+                .andExpect(jsonPath("$.page.totalPages", is(1)))
+                .andExpect(jsonPath("$.page.number", is(0)))
 
-                   //The array of browse index should have a size 6
-                   .andExpect(jsonPath("$._embedded.browses", hasSize(6)))
-
-                   //Check that all (and only) the default browse indexes are present
-                   .andExpect(jsonPath("$._embedded.browses", containsInAnyOrder(
-                       BrowseIndexMatcher.dateIssuedBrowseIndex("asc"),
-                       BrowseIndexMatcher.contributorBrowseIndex("asc"),
-                       BrowseIndexMatcher.titleBrowseIndex("asc"),
-                       BrowseIndexMatcher.subjectBrowseIndex("asc"),
-                       BrowseIndexMatcher.languageBrowseIndex("asc")
-                   )))
+                //The array of browse index should have a size 6
+                .andExpect(jsonPath("$._embedded.browses", hasSize(5)))
+
+                //Check that all (and only) the default browse indexes are present
+                .andExpect(jsonPath("$._embedded.browses", containsInAnyOrder(
+                        BrowseIndexMatcher.dateIssuedBrowseIndex("asc"),
+                        BrowseIndexMatcher.contributorBrowseIndex("asc"),
+                        BrowseIndexMatcher.titleBrowseIndex("asc"),
+                        BrowseIndexMatcher.subjectBrowseIndex("asc"),
+                        BrowseIndexMatcher.languageBrowseIndex("asc")
+                )))
         ;
+        } finally {
+            // Restore the original configuration value
+            if (originalValue != null) {
+                configurationService.setProperty("webui.browse.vocabularies.disabled", originalValue);
+            } else {
+                configurationService.setProperty("webui.browse.vocabularies.disabled", (String) null);
+            }
+            // Clear cache to ensure the original configuration is restored
+            metadataAuthorityService.clearCache();
+        }
     }
 
     @Test