fix(ci): switch test shards from scalatest:test to mvn test + add verify step

scalatest:test (direct plugin goal) skips the Maven lifecycle so
test-classes are not put on the classpath — result is 0 tests found.
Switch to `mvn test` so Zinc detects the pre-compiled artifacts and
runs ScalaTest without recompiling.

Also adds a "Verify compiled output" step on each shard to confirm
that obp-api/target/test-classes is populated before tests run.

Author: constantine2nd

.github/workflows/build_container.yml

@@ -164,6 +164,19 @@ jobs:
         with:
           name: compiled-output
 
+      - name: Verify compiled output
+        run: |
+          echo "=== obp-api/target/ layout ==="
+          ls -la obp-api/target/ 2>/dev/null || echo "MISSING: obp-api/target/"
+          echo ""
+          echo "=== .class file counts ==="
+          echo "main:  $(find obp-api/target/classes     -name '*.class' 2>/dev/null | wc -l)"
+          echo "test:  $(find obp-api/target/test-classes -name '*.class' 2>/dev/null | wc -l)"
+          echo "commons: $(find obp-commons/target/classes -name '*.class' 2>/dev/null | wc -l)"
+          echo ""
+          echo "=== sample test-classes paths ==="
+          find obp-api/target/test-classes -name '*.class' 2>/dev/null | head -5 || echo "none found"
+
       - name: Install local artifacts into Maven repo
         run: |
           # The compile runner's ~/.m2 is discarded after that job completes.
@@ -224,14 +237,14 @@ jobs:
 
       - name: Run tests — shard ${{ matrix.shard }} (${{ matrix.name }})
         run: |
-          # scalatest:test (direct plugin goal) runs tests without triggering
-          # the compile lifecycle — compiled classes from the artifact are used.
-          # Tests in this project are run by scalatest-maven-plugin, not surefire
-          # (surefire has <skipTests>true</skipTests> in pom.xml).
-          # -DwildcardSuites passes space-separated package prefixes as -w args
-          # to the ScalaTest runner, matching all suites in those packages.
+          # mvn test runs the full lifecycle. With compiled classes already
+          # present in target/ (from the artifact), Zinc incremental compilation
+          # detects no source changes and skips recompilation. Tests are run by
+          # scalatest-maven-plugin (surefire is disabled in pom.xml).
+          # -DwildcardSuites filters to the packages for this shard.
+          # See README.md: "mvn -DwildcardSuites=code.api.directloginTest test"
           MAVEN_OPTS="-Xmx3G -Xss2m -XX:MaxMetaspaceSize=1G" \
-            mvn scalatest:test -pl obp-api \
+            mvn test \
               -DwildcardSuites="${{ matrix.test_filter }}" \
             > maven-build-shard${{ matrix.shard }}.log 2>&1
 

.github/workflows/build_pull_request.yml

@@ -164,6 +164,19 @@ jobs:
         with:
           name: compiled-output
 
+      - name: Verify compiled output
+        run: |
+          echo "=== obp-api/target/ layout ==="
+          ls -la obp-api/target/ 2>/dev/null || echo "MISSING: obp-api/target/"
+          echo ""
+          echo "=== .class file counts ==="
+          echo "main:  $(find obp-api/target/classes     -name '*.class' 2>/dev/null | wc -l)"
+          echo "test:  $(find obp-api/target/test-classes -name '*.class' 2>/dev/null | wc -l)"
+          echo "commons: $(find obp-commons/target/classes -name '*.class' 2>/dev/null | wc -l)"
+          echo ""
+          echo "=== sample test-classes paths ==="
+          find obp-api/target/test-classes -name '*.class' 2>/dev/null | head -5 || echo "none found"
+
       - name: Install local artifacts into Maven repo
         run: |
           # The compile runner's ~/.m2 is discarded after that job completes.
@@ -224,14 +237,14 @@ jobs:
 
       - name: Run tests — shard ${{ matrix.shard }} (${{ matrix.name }})
         run: |
-          # scalatest:test (direct plugin goal) runs tests without triggering
-          # the compile lifecycle — compiled classes from the artifact are used.
-          # Tests in this project are run by scalatest-maven-plugin, not surefire
-          # (surefire has <skipTests>true</skipTests> in pom.xml).
-          # -DwildcardSuites passes space-separated package prefixes as -w args
-          # to the ScalaTest runner, matching all suites in those packages.
+          # mvn test runs the full lifecycle. With compiled classes already
+          # present in target/ (from the artifact), Zinc incremental compilation
+          # detects no source changes and skips recompilation. Tests are run by
+          # scalatest-maven-plugin (surefire is disabled in pom.xml).
+          # -DwildcardSuites filters to the packages for this shard.
+          # See README.md: "mvn -DwildcardSuites=code.api.directloginTest test"
           MAVEN_OPTS="-Xmx3G -Xss2m -XX:MaxMetaspaceSize=1G" \
-            mvn scalatest:test -pl obp-api \
+            mvn test \
               -DwildcardSuites="${{ matrix.test_filter }}" \
             > maven-build-shard${{ matrix.shard }}.log 2>&1