Closed Bug 1146612 Opened 5 years ago Closed 5 years ago

Add a test to ensure that Cache.put() with an existing request will reorder it in the DB

Categories

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

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: ehsan, Assigned: ehsan)

References

Details

Attachments

(1 file)

No description provided.
Comment on attachment 8582094 [details] [diff] [review]
Add a test to ensure that Cache.put() with an existing request will reorder it in the DB

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

::: dom/cache/test/mochitest/test_cache_put_reorder.js
@@ +23,5 @@
> +}).then(function(r) {
> +  return r.text();
> +}).then(function(body) {
> +  is(body, "overwritten", "The body should be overwritten");
> +  testDone();

I believe we need to delete the cache here to keep from breaking some of the other tests, right?

Once I land the nsIOfflineStorage patch, though, we'll be wiping storage between each test file.  That should avoid needing to do this in the future.
Attachment #8582094 - Flags: review?(bkelly) → review+
https://hg.mozilla.org/mozilla-central/rev/8370190f31b2
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Depends on: 1152622
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.