codecov[bot]
commented
1 year ago Codecov Report
Patch coverage: 73.46% and project coverage change: -1.04 :warning:
Comparison is base (
b228574) 82.05% compared to head (1522e72) 81.01%.
Additional details and impacted files
```diff @@ Coverage Diff @@ ## master #857 +/- ## ========================================== - Coverage 82.05% 81.01% -1.04% ========================================== Files 20 20 Lines 3365 3513 +148 ========================================== + Hits 2761 2846 +85 - Misses 604 667 +63 ``` | [Impacted Files](https://app.codecov.io/gh/servo/rust-url/pull/857?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=servo) | Coverage Δ | | |---|---|---| | [url/tests/wpt.rs](https://app.codecov.io/gh/servo/rust-url/pull/857?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=servo#diff-dXJsL3Rlc3RzL3dwdC5ycw==) | `73.46% <73.46%> (ø)` | | ... and [6 files with indirect coverage changes](https://app.codecov.io/gh/servo/rust-url/pull/857/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=servo):umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
This PR rewrites the WPT test data runner for
urlto use a custom harness. This has the following benefits:Tests can now be filtered, for example
cargo test --test url_wpt "set username"runs only tests with "set username" in the name.There is now an "expected_failures.txt" file that lists tests that are expected to fail, by name. The test harness will ensure that these tests fail, and will report a failure if they pass.