Skip to content

Commit 5545dd2

Browse files
micrycfrantuma
authored andcommitted
Change skippedPackages access modifier to private
1 parent 796edea commit 5545dd2

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

modules/swagger-core/src/main/java/io/swagger/v3/core/converter/ModelConverters.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ public class ModelConverters {
2525
private static ModelConverters SINGLETON31 = null;
2626
static Logger LOGGER = LoggerFactory.getLogger(ModelConverters.class);
2727
private final List<ModelConverter> converters;
28-
public final Set<String> skippedPackages = new HashSet<>();
28+
private final Set<String> skippedPackages = new HashSet<>();
2929
private final Set<String> skippedClasses = new HashSet<>();
3030

3131
public ModelConverters() {
@@ -42,6 +42,10 @@ public ModelConverters(boolean openapi31) {
4242
}
4343
}
4444

45+
public Set<String> getSkippedPackages() {
46+
return skippedPackages;
47+
}
48+
4549
public static ModelConverters getInstance(boolean openapi31) {
4650
if (openapi31) {
4751
if (SINGLETON31 == null) {

modules/swagger-core/src/test/java/io/swagger/v3/core/converting/ModelConverterTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -387,8 +387,8 @@ public void indirectPropertiesRecognized() {
387387
public void checkDefaultSkippedPackages(){
388388
ModelConverters modelConverters = ModelConverters.getInstance();
389389

390-
assertTrue(modelConverters.skippedPackages.contains("java.lang"));
391-
assertTrue(modelConverters.skippedPackages.contains("groovy.lang"));
390+
assertTrue(modelConverters.getSkippedPackages().contains("java.lang"));
391+
assertTrue(modelConverters.getSkippedPackages().contains("groovy.lang"));
392392
}
393393

394394
@Test(description = "It should not process skipped package")

0 commit comments

Comments
 (0)