Change skippedPackages access modifier to private

swagger-api · Jan 15, 2024 · 5545dd2 · 5545dd2
1 parent 796edea
commit 5545dd2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/modules/swagger-core/src/main/java/io/swagger/v3/core/converter/ModelConverters.java b/modules/swagger-core/src/main/java/io/swagger/v3/core/converter/ModelConverters.java
@@ -25,7 +25,7 @@ public class ModelConverters {
     private static ModelConverters SINGLETON31 = null;
     static Logger LOGGER = LoggerFactory.getLogger(ModelConverters.class);
     private final List<ModelConverter> converters;
-    public final Set<String> skippedPackages = new HashSet<>();
+    private final Set<String> skippedPackages = new HashSet<>();
     private final Set<String> skippedClasses = new HashSet<>();
 
     public ModelConverters() {
@@ -42,6 +42,10 @@ public ModelConverters(boolean openapi31) {
         }
     }
 
+    public Set<String> getSkippedPackages() {
+        return skippedPackages;
+    }
+
     public static ModelConverters getInstance(boolean openapi31) {
         if (openapi31) {
             if (SINGLETON31 == null) {

diff --git a/modules/swagger-core/src/test/java/io/swagger/v3/core/converting/ModelConverterTest.java b/modules/swagger-core/src/test/java/io/swagger/v3/core/converting/ModelConverterTest.java
@@ -387,8 +387,8 @@ public void indirectPropertiesRecognized() {
     public void checkDefaultSkippedPackages(){
         ModelConverters modelConverters = ModelConverters.getInstance();
 
-        assertTrue(modelConverters.skippedPackages.contains("java.lang"));
-        assertTrue(modelConverters.skippedPackages.contains("groovy.lang"));
+        assertTrue(modelConverters.getSkippedPackages().contains("java.lang"));
+        assertTrue(modelConverters.getSkippedPackages().contains("groovy.lang"));
     }
 
     @Test(description = "It should not process skipped package")