chore: run tests by default in Maven builds

.github/copilot-instructions.md

@@ -11,11 +11,14 @@ Apache Fesod (Incubating) is a Java library for processing spreadsheets (XLS/XLS
 ## Build & Test Commands
 
 ```bash
-# Build (skips tests by default)
+# Build (runs tests by default, per Maven/Apache convention)
+./mvnw clean install
+
+# Fast build without running tests
 ./mvnw clean install -DskipTests
 
-# Run tests (must explicitly enable)
-./mvnw clean package -Dmaven.test.skip=false -pl fesod-common,fesod-shaded,fesod-sheet
+# Run tests for the library modules only
+./mvnw clean package -pl fesod-common,fesod-shaded,fesod-sheet
 
 # Format check / auto-format
 ./mvnw spotless:check

.github/workflows/ci.yml

@@ -86,13 +86,14 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-m2
       - name: Test with Maven
-        run: ./mvnw clean package -B -Dmaven.test.skip=false -pl fesod-common,fesod-shaded,fesod-sheet
+        run: ./mvnw clean package -B -pl fesod-common,fesod-shaded,fesod-sheet
       - name: Publish Unit Test Results
         uses: EnricoMi/publish-unit-test-result-action@v2
         if: (!cancelled())
         with:
           files: "**/target/surefire-reports/*.xml"
       - name: Maven Build
-        run: ./mvnw install -B -V
+        # Tests already ran in the previous step; skip here to avoid a second full run
+        run: ./mvnw install -B -V -DskipTests
       - name: Java Doc
         run: ./mvnw javadoc:javadoc

.github/workflows/fuzz-tests.yml

@@ -49,8 +49,8 @@ jobs:
           JAZZER_ARGS: "-artifact_prefix=${{ runner.temp }}/jazzer-artifacts/ -max_total_time=300  -rss_limit_mb=2048"
         run: |
           mkdir -p "${{ runner.temp }}/jazzer-artifacts/"
-          # Execute only tests under fuzz package and force tests not to be skipped
-          mvn -B -pl fesod-sheet -am -D"maven.test.skip=false" -DtrimStackTrace=false -D"test=org.apache.fesod.sheet.fuzz.*Test"  -D"surefire.failIfNoSpecifiedTests=false"  package
+          # Execute only tests under the fuzz package; tests run by default in Maven builds
+          mvn -B -pl fesod-sheet -am -DtrimStackTrace=false -D"test=org.apache.fesod.sheet.fuzz.*Test"  -D"surefire.failIfNoSpecifiedTests=false"  package
 
       - name: Upload Jazzer artifacts (crashes, repros)
         if: always()

.github/workflows/nightly.yml

@@ -49,7 +49,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-m2
       - name: Test with Maven
-        run: ./mvnw clean install -B -V -Dmaven.test.skip=false
+        run: ./mvnw clean install -B -V
       - name: Publish Unit Test Results
         uses: EnricoMi/publish-unit-test-result-action@v2
         if: (!cancelled())

pom.xml

@@ -74,7 +74,6 @@ under the License.
         <maven-compiler-plugin>3.14.1</maven-compiler-plugin>
         <maven-surefire-plugin.version>3.5.4</maven-surefire-plugin.version>
         <maven.javadoc.skip>true</maven.javadoc.skip>
-        <maven.test.skip>true</maven.test.skip>
         <commons-collections4.version>4.5.0</commons-collections4.version>
         <commons-csv.version>1.14.1</commons-csv.version>
         <commons-lang3.version>3.18.0</commons-lang3.version>

website/community/contribution/contribute-code.md

@@ -115,10 +115,10 @@ When submitting a PR, please ensure that the changes are based on the `main` bra
 
 ## Compile
 
-Run the following command to compile
+Run the following command to compile (unit tests run by default)
 
 ```bash
-mvn clean install -DskipTests
+mvn clean install
 ```
 
 To speed up the build process, you can:

website/community/release/release-version.md

@@ -344,7 +344,7 @@ mvn versions:set-property -Dproperty=project.build.outputTimestamp -DnewVersion=
 Compile the source code of the pre-release RC branch and push the binary packages to the [staging repository](https://repository.apache.org/#stagingRepositories).
 
 ```bash
-mvn clean deploy -Papache-release -DskipTests -Dgpg.skip=false
+mvn clean deploy -Papache-release -Dgpg.skip=false
 ```
 
 Log in to the ASF Nexus repository with your Apache ID, locate the released version, and click **Close**.
@@ -476,7 +476,7 @@ which can be found in the keys file:
 https://downloads.apache.org/incubator/fesod/KEYS
 
 Build Environment: JDK 8+, Apache Maven 3.6.0+.
-./mvnw clean package -DskipTests
+./mvnw clean package
 
 The vote will be open for at least 72 hours.
 
@@ -608,7 +608,7 @@ to [ xxxx@apache.org ] which can be found in the keys file:
 https://downloads.apache.org/incubator/fesod/KEYS
 
 Build Environment: JDK 8+, Apache Maven 3.6.0+.
-./mvnw clean package -DskipTests
+./mvnw clean package
 
 The vote will be open for at least 72 hours.
 

website/i18n/zh-cn/docusaurus-plugin-content-docs-community/current/contribution/contribute-code.md

@@ -117,10 +117,10 @@ git push origin <your_branch_name>
 
 ## 编译
 
-运行以下命令编译
+运行以下命令编译（默认会运行单元测试）
 
 ```bash
-mvn clean install -DskipTests
+mvn clean install
 ```
 
 为了加快构建速度，您可以：

website/i18n/zh-cn/docusaurus-plugin-content-docs-community/current/release/release-version.md

@@ -344,7 +344,7 @@ mvn versions:set-property -Dproperty=project.build.outputTimestamp -DnewVersion=
 编译预发布RC版本分支源码，并推送二进制包到 [预发仓库](https://repository.apache.org/#stagingRepositories)
 
 ```bash
-mvn clean deploy -Papache-release -DskipTests -Dgpg.skip=false
+mvn clean deploy -Papache-release -Dgpg.skip=false
 ```
 
 使用 Apache ID 登录 ASF Nexus 仓库，找到发布的版本并点击 **Close**。
@@ -476,7 +476,7 @@ which can be found in the keys file:
 https://downloads.apache.org/incubator/fesod/KEYS
 
 Build Environment: JDK 8+, Apache Maven 3.6.0+.
-./mvnw clean package -DskipTests
+./mvnw clean package
 
 The vote will be open for at least 72 hours.
 
@@ -608,7 +608,7 @@ to [ xxxx@apache.org ] which can be found in the keys file:
 https://downloads.apache.org/incubator/fesod/KEYS
 
 Build Environment: JDK 8+, Apache Maven 3.6.0+.
-./mvnw clean package -DskipTests
+./mvnw clean package
 
 The vote will be open for at least 72 hours.