Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[css-backgrounds] move tests from vendor-imports into the main suite #27489

Merged
merged 1 commit into from Feb 5, 2021

Conversation

foolip
Copy link
Member

@foolip foolip commented Feb 4, 2021

These tests all pass on at least some products on wpt.fyi:
https://wpt.fyi/results/css/vendor-imports/mozilla/mozilla-central-reftests/background?run_id=5763984030433280&run_id=5723142917980160&run_id=5663034011811840&run_id=5651434144006144&run_id=5135050226008064&run_id=5683699381174272

Whether the tests match spec requirements was not evaluated. That many
pass on some browsers and fail on others means they are exposing some
interoperability issue, which is valuable even if the pass condition
turns out to be wrong.

support/border.png already exists and was exactly the same file, so the
vendor-imports copy is deleted.

Part of #8615.

These tests all pass on at least some products on wpt.fyi:
https://wpt.fyi/results/css/vendor-imports/mozilla/mozilla-central-reftests/background?run_id=5763984030433280&run_id=5723142917980160&run_id=5663034011811840&run_id=5651434144006144&run_id=5135050226008064&run_id=5683699381174272

Whether the tests match spec requirements was not evaluated. That many
pass on some browsers and fail on others means they are exposing some
interoperability issue, which is valuable even if the pass condition
turns out to be wrong.

support/border.png already exists and was exactly the same file, so the
vendor-imports copy is deleted.

Part of #8615.
@Ms2ger Ms2ger merged commit b10b8a0 into master Feb 5, 2021
@Ms2ger Ms2ger deleted the foolip/vendor-imports-backgrounds branch February 5, 2021 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants