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

Making create attribute lowercase #9282

Merged
merged 1 commit into from Jan 13, 2016
Merged

Making create attribute lowercase #9282

merged 1 commit into from Jan 13, 2016

Conversation

@mattnenterprise
Copy link
Contributor

mattnenterprise commented Jan 12, 2016

This should fix #9252. I need some help getting the test case to work correctly. I get output that "TITLE" should FAIL, but it PASS. How do I make it to where it should PASS?

Review on Reviewable

@nox
Copy link
Member

nox commented Jan 12, 2016

Run the following:

./mach test-wpt --no-pause --log-raw raw.log /dom/nodes/Document-createAttribute.html
./mach update-wpt --no-patch raw.log

This will change the test expectations for you.

@nox nox self-assigned this Jan 12, 2016
@jdm
Copy link
Member

jdm commented Jan 12, 2016

If the test file is tests/wpt/web-platform-tests/path/to/test.html, you need to modify (or remove, if all subtests pass) tests/wpt/metadata/path/to/test.html.ini, as described in "Upating test expectations" in tests/wpt/README.md.

@mattnenterprise
Copy link
Contributor Author

mattnenterprise commented Jan 13, 2016

I followed the instructions for updating test expectations. The tests now pass.

@nox
Copy link
Member

nox commented Jan 13, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

📌 Commit e684a20 has been approved by nox

@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

Testing commit e684a20 with merge 023fa86...

bors-servo added a commit that referenced this pull request Jan 13, 2016
Making create attribute lowercase

This should fix #9252. I need some help getting the test case to work correctly. I get output that "TITLE" should FAIL, but it PASS. How do I make it to where it should PASS?

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9282)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

Testing commit e684a20 with merge dccddc6...

bors-servo added a commit that referenced this pull request Jan 13, 2016
Making create attribute lowercase

This should fix #9252. I need some help getting the test case to work correctly. I get output that "TITLE" should FAIL, but it PASS. How do I make it to where it should PASS?

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9282)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

💔 Test failed - linux-rel

@KiChjang
Copy link
Member

KiChjang commented Jan 13, 2016

@bors-servo retry

  • ./mach test-wpt timed out
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

Testing commit e684a20 with merge ee5aead...

bors-servo added a commit that referenced this pull request Jan 13, 2016
Making create attribute lowercase

This should fix #9252. I need some help getting the test case to work correctly. I get output that "TITLE" should FAIL, but it PASS. How do I make it to where it should PASS?

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9282)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2016

@bors-servo bors-servo merged commit e684a20 into servo:master Jan 13, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

6 participants
You can’t perform that action at this time.