feat: enable all verification in ci (#390)

* feat: enable all verification in ci

1. change default maven.test.skip to false
2. change surefire scope for all test as /cn/idev/excel/test/**/*.java

* feat: enable all verification in ci

1. fix maven build issue for jdk greater than 17

* feat: enable all verification in ci

1. fix maven build issue for jdk greater than 17

Author: alaahong

.github/workflows/ci.yml

@@ -38,6 +38,10 @@ jobs:
         if: ${{ matrix.java != '8' }}
         run: ./mvnw test -B -Dmaven.test.skip=false -DargLine="--add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/sun.reflect.annotation=ALL-UNNAMED"
       - name: Maven Build
-        run: ./mvnw install -B -V
+        if: ${{ matrix.java == '8' }}
+        run: ./mvnw install -B -V -Dmaven.surefire.plugin.include.scope="/cn/idev/excel/test/core/**/*.java"
+      - name: Maven Build
+        if: ${{ matrix.java != '8' }}
+        run: ./mvnw install -B -V -Dmaven.surefire.plugin.include.scope="/cn/idev/excel/test/core/**/*.java" -DargLine="--add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/sun.reflect.annotation=ALL-UNNAMED"
       - name: Java Doc
         run: ./mvnw javadoc:javadoc

fastexcel-test/pom.xml

@@ -16,6 +16,7 @@
 
     <properties>
         <maven.deploy.skip>true</maven.deploy.skip>
+        <maven.surefire.plugin.include.scope>/cn/idev/excel/test/**/*.java</maven.surefire.plugin.include.scope>
     </properties>
 
     <dependencies>
@@ -71,7 +72,7 @@
                 <artifactId>maven-surefire-plugin</artifactId>
                 <configuration>
                     <includes>
-                        <include>/cn/idev/excel/test/core/**/*.java</include>
+                        <include>${maven.surefire.plugin.include.scope}</include>
                     </includes>
                     <testFailureIgnore>false</testFailureIgnore>
                 </configuration>

pom.xml

@@ -31,7 +31,7 @@
         <maven.compiler.source>1.8</maven.compiler.source>
         <gpg.skip>true</gpg.skip>
         <maven.javadoc.skip>true</maven.javadoc.skip>
-        <maven.test.skip>true</maven.test.skip>
+        <maven.test.skip>false</maven.test.skip>
         <commons-csv.version>1.11.0</commons-csv.version>
         <poi.version>5.4.1</poi.version>
         <poi-ooxml.version>5.4.1</poi-ooxml.version>