Skip to content

Commit 537bfd6

Browse files
author
Adarsh Sharma
committed
Fixing NPE if new or old file doesn't have any paths
1 parent d3aa7a4 commit 537bfd6

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/main/java/com/qdesrame/openapi/diff/compare/OpenApiDiff.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
import com.qdesrame.openapi.diff.utils.EndpointUtils;
66
import io.swagger.v3.oas.models.OpenAPI;
77
import io.swagger.v3.oas.models.PathItem;
8+
import io.swagger.v3.oas.models.Paths;
89
import io.swagger.v3.oas.models.security.SecurityRequirement;
910
import io.swagger.v3.parser.OpenAPIV3Parser;
1011
import io.swagger.v3.parser.core.models.AuthorizationValue;
@@ -131,7 +132,9 @@ private OpenApiDiff(OpenAPI oldSpecOpenApi, OpenAPI newSpecOpenApi) {
131132
private ChangedOpenApi compare() {
132133
preProcess(oldSpecOpenApi);
133134
preProcess(newSpecOpenApi);
134-
Optional<ChangedPaths> paths = this.pathsDiff.diff(oldSpecOpenApi.getPaths(), newSpecOpenApi.getPaths());
135+
Paths oldPaths = oldSpecOpenApi.getPaths() == null? new Paths(): oldSpecOpenApi.getPaths();
136+
Paths newPaths = newSpecOpenApi.getPaths() == null? new Paths(): newSpecOpenApi.getPaths();
137+
Optional<ChangedPaths> paths = this.pathsDiff.diff(oldPaths, newPaths);
135138
this.newEndpoints = new ArrayList<>();
136139
this.missingEndpoints = new ArrayList<>();
137140
this.changedOperations = new ArrayList<>();

0 commit comments

Comments
 (0)