Skip to content

Improve code coverage.#137

Merged
azriel91 merged 50 commits intomainfrom
maintenance/136/increase-coverage
Jun 18, 2023
Merged

Improve code coverage.#137
azriel91 merged 50 commits intomainfrom
maintenance/136/increase-coverage

Conversation

@azriel91
Copy link
Copy Markdown
Owner

@azriel91 azriel91 commented Jun 14, 2023

Closes #136.

  • Removed "output_colorized" feature, and always include colorized output.
  • Removed "output_json" feature, and always include json output format.
  • Add tests where useful cases are missed.
  • Ignore unnecessary missed lines (e.g. panics in tests).

Rationale for feature removal is because we ran into GitHub Action runners' disk space limit. There wasn't very much value feature gating colorization and json, and it makes it easier to maintain fewer features as well. See taiki-e/cargo-llvm-cov#284.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 14, 2023

Codecov Report

Patch coverage: 97.77% and project coverage change: +2.63 🎉

Comparison is base (0e36740) 89.21% compared to head (57e21b6) 91.85%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
+ Coverage   89.21%   91.85%   +2.63%     
==========================================
  Files         359      380      +21     
  Lines       32877    35539    +2662     
==========================================
+ Hits        29332    32645    +3313     
+ Misses       3545     2894     -651     
Impacted Files Coverage Δ
crate/core/src/flow_id.rs 100.00% <ø> (+75.00%) ⬆️
crate/core/src/profile.rs 100.00% <ø> (+60.00%) ⬆️
crate/rt_model_core/src/error.rs 1.51% <ø> (ø)
crate/rt_model_core/src/output/output_format.rs 100.00% <ø> (ø)
...e/rt_model_native/src/output/cli_output_builder.rs 93.68% <ø> (-5.60%) ⬇️
crate/rt_model_native/src/workspace.rs 89.65% <ø> (ø)
crate/static_check_macros/src/lit_str_maybe.rs 100.00% <ø> (+10.00%) ⬆️
...orkspace_tests/src/cfg/progress/progress_update.rs 100.00% <ø> (ø)
...e_tests/src/cfg/progress/progress_update_and_id.rs 100.00% <ø> (ø)
...d_ctx_builder/multi_profile_single_flow_builder.rs 100.00% <ø> (ø)
... and 84 more

... and 51 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@azriel91 azriel91 force-pushed the maintenance/136/increase-coverage branch from 8928d2e to ff30776 Compare June 15, 2023 09:57
azriel91 added 24 commits June 16, 2023 09:16
Regex replacement used in sublime text:

```regex
^    assert!\(
^        matches!([^;]+)
^    \);
```

replace:

```
    ({
        #[cfg_attr(coverage_nightly, no_coverage)]
        || {
            assert!(
                matches!\1
            );
        }
    })();
```
@azriel91 azriel91 marked this pull request as ready for review June 18, 2023 07:43
@azriel91 azriel91 merged commit 5f8eb86 into main Jun 18, 2023
@azriel91 azriel91 deleted the maintenance/136/increase-coverage branch June 18, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Maintenance: Increase Coverage

1 participant