imbajin commented on code in PR #3161:
URL: https://github.com/apache/hugegraph/pull/3161#discussion_r3817555644


##########
.github/workflows/pd-store-ci.yml:
##########
@@ -228,32 +268,52 @@ jobs:
 
       - name: Run common test
         run: |
-          mvn test -pl hugegraph-store/hg-store-test -am -P store-common-test
+          mvn test -pl hugegraph-store/hg-store-test -am \
+            -P store-common-test -Djacoco.sessionId=store-common-test
 
       - name: Run client test
         run: |
-          mvn test -pl hugegraph-store/hg-store-test -am -P store-client-test
-
-      - name: Run core test
-        run: |
-          mvn test -pl hugegraph-store/hg-store-test -am -P store-core-test
+          mvn test -pl hugegraph-store/hg-store-test -am \
+            -P store-client-test -Djacoco.sessionId=store-client-test
 
       - name: Run rocksdb test
         run: |
-          mvn test -pl hugegraph-store/hg-store-test -am -P store-rocksdb-test
-
-      - name: Run server test
-        run: |
-          mvn test -pl hugegraph-store/hg-store-test -am -P store-server-test
+          mvn test -pl hugegraph-store/hg-store-test -am \
+            -P store-rocksdb-test -Djacoco.sessionId=store-rocksdb-test
 
       - name: Run raft-core test
         run: |
-          mvn test -pl hugegraph-store/hg-store-test -am -P store-raftcore-test
+          mvn test -pl hugegraph-store/hg-store-test -am \
+            -P store-raftcore-test -Djacoco.sessionId=store-raftcore-test
+
+      - name: Generate aggregate coverage report

Review Comment:
   ⚠️ Important
   
   Blocking: yes. The Store job no longer runs the existing `store-core-test` 
or `store-server-test` suites, even though both executions remain in 
`hg-store-test/pom.xml` and the aggregate report still claims coverage for 
`hg-store-core` and `hg-store-node`. This makes the CI gate a partial test run 
while presenting a full Store report. Restore both profiles and require their 
reports/sessions, or explicitly remove them from the covered scope with a 
documented replacement.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to