Add missing java.sql require (#1999)

* Add missing java.sql * Update maven checkstyle * Fix gradle checkstyle * Bump surefire plugin * Skip modules in tests
JSQLParser · Apr 30, 2024 · df48c4b · df48c4b
1 parent 761b45b
commit df48c4b
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git a/build.gradle b/build.gradle
@@ -340,6 +340,7 @@ tasks.withType(Checkstyle).configureEach {
  xml.required = false
  html.required = true
  }
+ excludes = [ "**/module-info.java" ]
 }
 
 tasks.register('renderRR') {

diff --git a/pom.xml b/pom.xml
@@ -378,9 +378,11 @@
  <plugin>
  <groupId>org.apache.maven.plugins</groupId>
  <artifactId>maven-surefire-plugin</artifactId>
- <version>3.2.3</version>
+ <version>3.2.5</version>
  <configuration>
  <trimStackTrace>false</trimStackTrace>
+ <!-- AlterViewTest does deep reflection -->
+ <useModulePath>false</useModulePath>
  </configuration>
  </plugin>
  <plugin>
@@ -555,6 +557,7 @@
  <logViolationsToConsole>true</logViolationsToConsole>
  <includeTestSourceDirectory>true</includeTestSourceDirectory>
  <sourceDirectories>${project.build.sourceDirectory}</sourceDirectories>
+ <excludes>**/module-info.java</excludes>
  <checkstyleRules>
  <module name="Checker">
  <module name="SuppressWarningsFilter" />

diff --git a/src/main/java/module-info.java b/src/main/java/module-info.java
@@ -8,6 +8,8 @@
  * #L%
  */
 module net.sf.jsqlparser {
+ requires java.sql;
+
  exports net.sf.jsqlparser;
  exports net.sf.jsqlparser.expression;
  exports net.sf.jsqlparser.expression.operators.arithmetic;