Enable strict compiler warnings

exasol · Mar 12, 2024 · c85e562 · c85e562
1 parent 6f6cb03
commit c85e562
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 0 deletions.
diff --git a/extension-manager-client-java/pom.xml b/extension-manager-client-java/pom.xml
@@ -88,6 +88,17 @@
     </dependencies>
     <build>
         <plugins>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-compiler-plugin</artifactId>
+                <configuration>
+                    <compilerArgs>
+                        <!-- Exclude warnings in generated code -->
+                        <arg>-Xlint:all,-serial,-rawtypes,-deprecation</arg>
+                        <arg>-Werror</arg>
+                    </compilerArgs>
+                </configuration>
+            </plugin>
             <plugin>
                 <groupId>org.codehaus.mojo</groupId>
                 <artifactId>versions-maven-plugin</artifactId>

diff --git a/extension-manager-integration-test-java/pom.xml b/extension-manager-integration-test-java/pom.xml
@@ -79,6 +79,16 @@
     </dependencies>
     <build>
         <plugins>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-compiler-plugin</artifactId>
+                <configuration>
+                    <compilerArgs>
+                        <arg>-Xlint:all,-path</arg>
+                        <arg>-Werror</arg>
+                    </compilerArgs>
+                </configuration>
+            </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-jar-plugin</artifactId>

diff --git a/...ion-test-java/src/main/java/com/exasol/extensionmanager/itest/ExtensionManagerClient.java b/...ion-test-java/src/main/java/com/exasol/extensionmanager/itest/ExtensionManagerClient.java
@@ -255,6 +255,8 @@ public void assertRequestFails(final Executable executable, final Matcher<String
                 () -> assertThat("status code", error.getJsonNumber("code").intValue(), statusMatcher));
     }
 
+    @SuppressWarnings("try") // auto-closeable resource Jsonb has a member method close() that could throw
+                             // InterruptedException
     JsonObject parseErrorMessageJson(final String errorMessage) throws MultipleFailuresError {
         try (Jsonb jsonb = JsonbBuilder.create()) {
             return jsonb.fromJson(errorMessage, JsonObject.class);