fix: cpp test report broken (#38)

fix #36
4paradigm · Apr 8, 2021 · a72e2b9 · a72e2b9
1 parent e1ca1e2
commit a72e2b9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.github/workflows/hybridse-ci.yml b/.github/workflows/hybridse-ci.yml
@@ -41,19 +41,21 @@ jobs:
       uses: EnricoMi/publish-unit-test-result-action@v1
       if: always()
       with:
-        files: |
-          build/*.xml
+        files: build/*.xml
+        check_name: Cpp Test Report
+        comment_title: Cpp Test Report
 
     - name: Java SDK Test
       run: |
         cd java/
         mvn -U clean compile test
-        
+
     - name: Publish Java Unit Test Results
       uses: EnricoMi/publish-unit-test-result-action@v1
       if: always()
       with:
         files: java/hybridse-sdk/target/**/TEST-*.xml
+        check_name: Java Test Report
         comment_title: Java Test Report
 
     - name: Cleanup

diff --git a/tools/hybridse_core_test.sh b/tools/hybridse_core_test.sh
@@ -30,7 +30,7 @@ fi
 mkdir -p build
 cd build
 cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX="hybridse"
-make -j"$(nproc)" hybridse_core
+make -j"$(nproc)"
 SQL_CASE_BASE_DIR=${HYRBIDSE_DIR} make -j"$(nproc)" test
 
 popd