Closed
Bug 778213
Opened 12 years ago
Closed 12 years ago
error: TEST FAILED: test-xhr.testResponseHeaders (failure)
Categories
(Add-on SDK Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: KWierso, Unassigned)
References
Details
Attachments
(1 file)
error: TEST FAILED: test-xhr.testResponseHeaders (failure) error: fail: XHR's headers are valid ("Content-Type: text/plain\r\n" != "Content-Type: text/plain\n") info: Traceback (most recent call last): File "resource://2fac4b91-34cf-493c-a315-0702e1623235-at-jetpack/api-utils/lib/xhr.js", line 121, in null self._orsc.apply(self, arguments); File "resource://2fac4b91-34cf-493c-a315-0702e1623235-at-jetpack/api-utils/tests/test-xhr.js", line 62, in null "XHR's headers are valid"); File "resource://2fac4b91-34cf-493c-a315-0702e1623235-at-jetpack/api-utils/lib/unit-test.js", line 164, in assertEqual this.fail(message); File "resource://2fac4b91-34cf-493c-a315-0702e1623235-at-jetpack/api-utils/lib/unit-test.js", line 71, in fail this.console.trace(); This started showing up after Will pushed bug 754111, which looks to me like a docs-only change... All m-c test runs are now orange from this.
Reporter | ||
Comment 1•12 years ago
|
||
Looks like this was a change on Firefox's side.
Reporter | ||
Comment 2•12 years ago
|
||
Attachment #646664 -
Flags: review?(poirot.alex)
Comment 3•12 years ago
|
||
Comment on attachment 646664 [details]
Link to pull request 514
Can you then push a new changeset in jetpack-location.txt?
There is many bugs being fixed on master and still failing on m-c tree.
Attachment #646664 -
Flags: review?(poirot.alex) → review+
Reporter | ||
Comment 4•12 years ago
|
||
Can do, though it probably won't be until later today at the earliest.
Comment 5•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/ac8c41c91cdbd65df00c5e30059c7d71773b1213 Fix bug 778213 by changing the expected headers https://github.com/mozilla/addon-sdk/commit/12922dab9c916e132aecb40ea35f5432087bf722 Merge pull request #514 from KWierso/bug778213 Fix bug 778213 by changing the expected xhr headers
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 6•12 years ago
|
||
(In reply to Alexandre Poirot (:ochameau) from comment #3) > There is many bugs being fixed on master and still failing on m-c tree. http://hg.mozilla.org/integration/mozilla-inbound/rev/9448db2a1708
Status: RESOLVED → UNCONFIRMED
Ever confirmed: false
Resolution: FIXED → ---
Reporter | ||
Comment 7•12 years ago
|
||
Not quite sure how it reset the status/resolutions...
Status: UNCONFIRMED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•