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

Still test interface members when they have namespace clashes #19611

Merged
merged 2 commits into from Oct 10, 2019

Conversation

@lukebjerring
Copy link
Contributor

lukebjerring commented Oct 10, 2019

As pointed out in w3c/csswg-drafts#4084 (comment) we are failing to get coverage for interfaces after testing for namespace clashes.

It's more valuable to test them anyway.

Luke Bjerring
Luke Bjerring
@lukebjerring

This comment has been minimized.

Copy link
Contributor Author

lukebjerring commented Oct 10, 2019

@foolip
foolip approved these changes Oct 10, 2019
@foolip

This comment has been minimized.

Copy link
Contributor

foolip commented Oct 10, 2019

Taskcluster is timing out because of too many affected tests, I'll admin merge.

@foolip foolip merged commit e501cac into master Oct 10, 2019
14 of 15 checks passed
14 of 15 checks passed
build-and-publish
Details
Taskcluster (pull_request) TaskGroup: failure
Details
Azure Pipelines Build #20191010.17 succeeded
Details
Azure Pipelines (./wpt test-jobs) ./wpt test-jobs succeeded
Details
Azure Pipelines (affected tests without changes: Safari Technology Preview) affected tests without changes: Safari Technology Preview succeeded
Details
Azure Pipelines (affected tests: Safari Technology Preview) affected tests: Safari Technology Preview succeeded
Details
Azure Pipelines (infrastructure/ tests: macOS) infrastructure/ tests: macOS succeeded
Details
Azure Pipelines (wpt.fyi hook: safari-preview-affected-tests) wpt.fyi hook: safari-preview-affected-tests succeeded
Details
Azure Pipelines (wpt.fyi hook: safari-preview-affected-tests-without-changes) wpt.fyi hook: safari-preview-affected-tests-without-changes succeeded
Details
staging.wpt.fyi - chrome[experimental] Chrome results
Details
staging.wpt.fyi - firefox[experimental] Firefox results
Details
staging.wpt.fyi - safari[experimental] Safari results
Details
wpt.fyi - chrome[experimental] Chrome results
Details
wpt.fyi - firefox[experimental] Firefox results
Details
wpt.fyi - safari[experimental] Safari results
Details
@foolip foolip deleted the idlharness branch Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.