Skip to content

Commit

Permalink
Fix #2378, update for code coverage
Browse files Browse the repository at this point in the history
Restructure some switch statements and add a "default" case so they
are not flagged as having a missing branch in the coverage test.
  • Loading branch information
jphickey committed Apr 19, 2024
1 parent 550e7f7 commit 48dd153
Show file tree
Hide file tree
Showing 4 changed files with 263 additions and 212 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/code-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,8 +104,8 @@ jobs:
- name: Confirm Minimum Coverage
run: |
missed_branches=50
missed_lines=17
missed_branches=39
missed_lines=19
branch_nums=$(grep -A 3 "Overall coverage rate" lcov_out.txt | grep branches | grep -oP "[0-9]+[0-9]*")
line_nums=$(grep -A 3 "Overall coverage rate" lcov_out.txt | grep lines | grep -oP "[0-9]+[0-9]*")
Expand Down
Loading

0 comments on commit 48dd153

Please sign in to comment.