Closed Bug 1146612 Opened 9 years ago Closed 9 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
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

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: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
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: