testMultipleLoads.js is failing due to new HTTP Cache v2

RESOLVED FIXED

Status

Testing Graveyard
Mozmill
RESOLVED FIXED
4 years ago
2 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

Bug Flags:
in-testsuite +

Details

(Whiteboard: [mozmill-2.0.7])

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
The mentioned test is currently failing on most branches due to the new Cache v2 implementation. We actually never updated the Mutt test:

ERROR | Test Failure | {
  "exception": {
    "message": "Component returned failure code: 0x80004001 (NS_ERROR_NOT_IMPLEMENTED) [nsICacheService.evictEntries]", 
    "lineNumber": 28, 
    "name": "NS_ERROR_NOT_IMPLEMENTED"
  }
}

We should fix this test, so the full Mutt testrun will pass.
(Assignee)

Comment 1

4 years ago
Created attachment 8488212 [details] [diff] [review]
http cache v1

Simple fix with a fallback to the old cache.
Attachment #8488212 - Flags: review?(ahalberstadt)
(Assignee)

Updated

4 years ago
Whiteboard: [mozmill-2.0.7]
Attachment #8488212 - Flags: review?(ahalberstadt) → review+
(Assignee)

Comment 2

4 years ago
https://github.com/mozilla/mozmill/commit/5cd7c99b6ce7e5dc7181d0746fe2a71e6d7005dd (hotfix-2.0)
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
(Assignee)

Comment 3

4 years ago
Just to note, this test is not present anymore on master. So no landing is necessary on that branch.
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.