Skip to content

Commit

Permalink
fix(config): use builder to create empty HealthCheckConfiguration due…
Browse files Browse the repository at this point in the history
… to private constructor
  • Loading branch information
poikilotherm committed Jan 8, 2024
1 parent 87667a3 commit d629ffe
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,8 @@ private HealthCheckConfiguration extractHealthCheck(HealthCheckConfiguration con
return config;
}
if (config == null) {
config = new HealthCheckConfiguration();
// Create empty config to allow null safe fallback value requests during extraction
config = new HealthCheckConfiguration.Builder().build();

Check warning on line 318 in src/main/java/io/fabric8/maven/docker/config/handler/property/PropertyConfigHandler.java

View check run for this annotation

Codecov / codecov/patch

src/main/java/io/fabric8/maven/docker/config/handler/property/PropertyConfigHandler.java#L318

Added line #L318 was not covered by tests
}

return new HealthCheckConfiguration.Builder()
Expand Down

0 comments on commit d629ffe

Please sign in to comment.