755509 - Add tests to ensure that deleteDatabase sets undefined success result

Status: RESOLVED FIXED

(Core :: Storage: IndexedDB, defect)

Comment 1

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Attachment: Part 1: Run deleteDatabase test in window and component scopes

This is just moving code around.

Comment 2

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Attachment: Part 2: Test explicitly for undefined.

Comment 3

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

https://hg.mozilla.org/mozilla-central/rev/1143cf39246f
https://hg.mozilla.org/mozilla-central/rev/38c9289bdf6d

Comment 4

[[e-mail]]

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");

Comment 5

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

No, is does ==, not ===.

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

The 'ise' instead of 'is' was not a typo; 'ise' does use ===

Comment 7

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Sounds like someone needs to update https://developer.mozilla.org/en/Mochitest#Test_functions then.