Minor change

Author: seanlaw

.github/workflows/github-actions.yml

@@ -214,9 +214,5 @@ jobs:
         run: ls
         shell: bash
       - name: Generate Coverage Report
-        run: ./test.sh report coverage.stumpy.json
-        shell: bash
-      - name: Get Coverage Percentage
-        run: |
-          export TOTAL=$(python -c "import json;print(json.load(open('coverage.stumpy.json'))['totals']['percent_covered_display'])")
+        run: ./test.sh report coverage.html
         shell: bash

test.sh

@@ -5,7 +5,7 @@ print_mode="verbose"
 custom_testfiles=()
 max_iter=10
 site_pkgs=$(python -c 'import site; print(site.getsitepackages()[0])')
-fcoveragejson="coverage.stumpy.json"
+fcoveragehtml="coverage.html"
 # Parse command line arguments
 for var in "$@"
 do
@@ -33,8 +33,8 @@ do
         custom_testfiles+=("$var")
     elif [[ $var =~ ^[\-0-9]+$ ]]; then
         max_iter=$var
-    elif [[ "$var" == *".json" ]]; then
-        fcoveragejson=$var
+    elif [[ "$var" == *".html" ]]; then
+        fcoveragehtml=$var
     elif [[ "$var" == "links" ]]; then
         test_mode="links"
     else
@@ -175,11 +175,11 @@ show_coverage_report()
     coverage report -m --fail-under=100 --skip-covered --omit=fastmath.py,docstring.py,min_versions.py,ray_python_version.py $fcoveragerc
 }
 
-gen_coverage_json_report()
+gen_coverage_html_report()
 {
     # This function saves the coverage report in JSON format
     set_ray_coveragerc
-    coverage json -o $fcoveragejson --skip-empty --fail-under=100 --omit=fastmath.py,docstring.py,min_versions.py,ray_python_version.py $fcoveragerc
+    coverage html -o $fcoveragehtml --skip-empty --fail-under=100 --omit=fastmath.py,docstring.py,min_versions.py,ray_python_version.py $fcoveragerc
 }
 
 combine_coverage_data_files()
@@ -391,7 +391,7 @@ elif [[ $test_mode == "report" ]]; then
     echo "Generate Coverage Report Only"
     # Assume coverage tests have already been executed
     # and a coverage file exists
-    gen_coverage_json_report
+    gen_coverage_html_report
 elif [[ $test_mode == "combine" ]]; then
     echo "Combine All Coverage Data Files"
     # Assume coverage tests have already been executed