Add tests to ensure that deleteDatabase sets undefined success result

RESOLVED FIXED in mozilla15

Status

()

Core
DOM: IndexedDB
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: khuey, Assigned: khuey)

Tracking

Trunk
mozilla15
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Comment hidden (empty)
Created attachment 624892 [details] [diff] [review]
Part 1: Run deleteDatabase test in window and component scopes

This is just moving code around.
Assignee: nobody → khuey
Status: NEW → ASSIGNED
Attachment #624892 - Flags: review?(jonas)
Created attachment 624894 [details] [diff] [review]
Part 2: Test explicitly for undefined.
Attachment #624894 - Flags: review?(jonas)
Attachment #624892 - Flags: review?(jonas) → review+
Attachment #624894 - Flags: review?(jonas) → review+
https://hg.mozilla.org/mozilla-central/rev/1143cf39246f
https://hg.mozilla.org/mozilla-central/rev/38c9289bdf6d
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
OS: Windows 7 → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Summary: deleteDatabase should set undefined success result → Add tests to ensure that deleteDatabase sets undefined success result
Target Milestone: --- → mozilla15
Version: unspecified → Trunk

Comment 4

5 years ago
Comment on attachment 624894 [details] [diff] [review]
Part 2: Test explicitly for undefined.

>-  is(event.target.result, null, "event should have no result");
>+  ok(event.target.result === undefined, "event should have no result");
ise(event.target.result, undefined, "event should have undefined result");
No, is does ==, not ===.
The 'ise' instead of 'is' was not a typo; 'ise' does use ===
Sounds like someone needs to update https://developer.mozilla.org/en/Mochitest#Test_functions then.
You need to log in before you can comment on or make changes to this bug.