Closed Bug 1089031 Opened 10 years ago Closed 10 years ago

Invalid mochitests skipped in dom/encoding/test/mochitest.ini

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: jgilbert, Assigned: vaibhav1994)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

> 16:07:15     INFO -  Warning: test_BOMEncoding.js from manifest /builds/slave/test/build/tests/mochitest/tests/dom/encoding/test/mochitest.ini is not a valid test
> 16:07:15     INFO -  Warning: test_TextDecoder.js from manifest /builds/slave/test/build/tests/mochitest/tests/dom/encoding/test/mochitest.ini is not a valid test
> 16:07:15     INFO -  Warning: test_TextEncoder.js from manifest /builds/slave/test/build/tests/mochitest/tests/dom/encoding/test/mochitest.ini is not a valid test
Comment on attachment 8514349 [details] [diff] [review]
manifest.patch

Try push is green and warnings not present for the above tests. :jmaher, adding you as the reviewer.
Attachment #8514349 - Flags: review?(jmaher)
Comment on attachment 8514349 [details] [diff] [review]
manifest.patch

Review of attachment 8514349 [details] [diff] [review]:
-----------------------------------------------------------------

awesome, thanks!
Attachment #8514349 - Flags: review?(jmaher) → review+
Assignee: nobody → vaibhavmagarwal
Keywords: checkin-needed
needs rebasing i guess

Hunk #1 FAILED at 3
1 out of 1 hunks FAILED -- saving rejects to file dom/encoding/test/mochitest.ini.rej
patch failed, unable to continue (try -v)
patch failed, rejects left in working dir
errors during apply, please fix and refresh manifest4.patch
Keywords: checkin-needed
Attached patch manifest4.patch (deleted) — Splinter Review
Fixed for bitrot.
Attachment #8514349 - Attachment is obsolete: true
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/8eafdf1f34ec
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: