Skip to content

Commit 711c507

Browse files
committed
build(deps): bump swagger plugin to 2.2.40
1 parent 9109cf4 commit 711c507

3 files changed

Lines changed: 22 additions & 18 deletions

File tree

gradle/libs.versions.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jupiter = "6.0.1"
1414
mockito = "5.20.0"
1515
openapi-merger = "1.0.5"
1616
openapi-generator = "7.16.0"
17-
swagger = "2.2.26"
17+
swagger = "2.2.40"
1818
swagger-parser = "2.1.35"
1919

2020
[libraries]

plugins/edc-build/src/main/java/org/eclipse/edc/plugins/edcbuild/conventions/SwaggerResolveConvention.java

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ public void apply(Project target) {
6161

6262
var outputDir = Path.of(swaggerExt.getOutputDirectory().getOrElse(fallbackOutputDir.toFile()).toURI())
6363
.resolve(apiGroup)
64-
.toFile();
64+
.toAbsolutePath().toString();
6565

6666
task.setOutputFileName(outputFileName);
6767
task.setOutputDir(outputDir);
@@ -76,7 +76,7 @@ public void apply(Project target) {
7676
target.getTasks().register("openapi", ResolveTask.class).configure(task -> {
7777
var outputDir = target.getLayout().getBuildDirectory().getAsFile().get().toPath()
7878
.resolve("docs").resolve("openapi")
79-
.toFile();
79+
.toAbsolutePath().toString();
8080

8181
target.getTasks().findByName("jar").dependsOn(task);
8282
task.setGroup("documentation");
@@ -90,9 +90,6 @@ public void apply(Project target) {
9090
task.setBuildClasspath(task.getClasspath());
9191
task.setResourcePackages(resourcePkgs);
9292
});
93-
94-
target.getConfigurations().all(c -> c.exclude(Map.of("group", "com.fasterxml.jackson.jaxrs",
95-
"module", "jackson-jaxrs-json-provider")));
9693
});
9794
}
9895
}

plugins/edc-build/src/test/java/org/eclipse/edc/plugins/edcbuild/conventions/SwaggerResolveConventionTest.java

Lines changed: 19 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ void setUp() {
3737
project = ProjectBuilder.builder().withName(PROJECT_NAME).build();
3838
project.getPluginManager().apply(SWAGGER_GRADLE_PLUGIN);
3939
project.getPluginManager().apply(JavaPlugin.class);
40+
project.getRepositories().mavenCentral();
4041
project.getExtensions().create("edcBuild", BuildExtension.class, project.getObjects());
4142
}
4243

@@ -45,11 +46,13 @@ void apply_whenApiGroupNotSpecified_shouldUseDefault() {
4546
var convention = new SwaggerResolveConvention();
4647
convention.apply(project);
4748

48-
var resolveTask = (ResolveTask) project.getTasks().getByName("resolve");
49+
var task = project.getTasks().getByName("resolve");
4950

50-
assertThat(resolveTask.getOutputDir().toString()).endsWith("/resources/openapi/yaml");
51-
assertThat(resolveTask.getOutputFileName()).isEqualTo(PROJECT_NAME);
52-
assertThat(resolveTask.getOutputFormat()).isEqualTo(ResolveTask.Format.YAML);
51+
assertThat(task).isInstanceOfSatisfying(ResolveTask.class, resolveTask -> {
52+
assertThat(resolveTask.getOutputDir().get().getAsFile().getAbsolutePath()).endsWith("/resources/openapi/yaml");
53+
assertThat(resolveTask.getOutputFileName().get()).isEqualTo(PROJECT_NAME);
54+
assertThat(resolveTask.getOutputFormat().get()).isEqualTo(ResolveTask.Format.YAML);
55+
});
5356
}
5457

5558
@Test
@@ -59,11 +62,13 @@ void apply_whenApiGroupSpecified_shouldAppend() {
5962
var convention = new SwaggerResolveConvention();
6063
convention.apply(project);
6164

62-
var resolveTask = (ResolveTask) project.getTasks().getByName("resolve");
65+
var task = project.getTasks().getByName("resolve");
6366

64-
assertThat(resolveTask.getOutputDir().toString()).endsWith("/resources/openapi/yaml/test-api");
65-
assertThat(resolveTask.getOutputFileName()).isEqualTo(PROJECT_NAME);
66-
assertThat(resolveTask.getOutputFormat()).isEqualTo(ResolveTask.Format.YAML);
67+
assertThat(task).isInstanceOfSatisfying(ResolveTask.class, resolveTask -> {
68+
assertThat(resolveTask.getOutputDir().get().getAsFile().getAbsolutePath()).endsWith("/resources/openapi/yaml/test-api");
69+
assertThat(resolveTask.getOutputFileName().get()).isEqualTo(PROJECT_NAME);
70+
assertThat(resolveTask.getOutputFormat().get()).isEqualTo(ResolveTask.Format.YAML);
71+
});
6772
}
6873

6974
@Test
@@ -74,10 +79,12 @@ void apply_whenOutputDirSet_shouldAppend() {
7479
var convention = new SwaggerResolveConvention();
7580
convention.apply(project);
7681

77-
var resolveTask = (ResolveTask) project.getTasks().getByName("resolve");
82+
var task = project.getTasks().getByName("resolve");
7883

79-
assertThat(resolveTask.getOutputDir().toString()).endsWith("/some/funny/path/test-api");
80-
assertThat(resolveTask.getOutputFileName()).isEqualTo(PROJECT_NAME);
81-
assertThat(resolveTask.getOutputFormat()).isEqualTo(ResolveTask.Format.YAML);
84+
assertThat(task).isInstanceOfSatisfying(ResolveTask.class, resolveTask -> {
85+
assertThat(resolveTask.getOutputDir().get().getAsFile().getAbsolutePath()).endsWith("/some/funny/path/test-api");
86+
assertThat(resolveTask.getOutputFileName().get()).isEqualTo(PROJECT_NAME);
87+
assertThat(resolveTask.getOutputFormat().get()).isEqualTo(ResolveTask.Format.YAML);
88+
});
8289
}
8390
}

0 commit comments

Comments
 (0)