757599 - Settings API: add test for closed lock

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, defect)

Description

[Gregor Wagner [:gwagner]]

Attachment: patch

Comment 1

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 626163 [details] [diff] [review]
patch

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

::: dom/settings/SettingsManager.js
@@ +122,2 @@
>        throw Components.results.NS_ERROR_ABORT;
> +      return;

The return line won't get executed. The 'throw' call will end execution of the function. So just remove the return statement.

Same in all other changes in this file.

::: dom/settings/tests/test_settings_basics.html
@@ +481,5 @@
> +    var lockx = mozSettings.getLock();
> +    var cb = function() {
> +      var reqx = {};
> +      try {
> +        reqx = lockx.set(wifiNetworks0);

Add a ok(false, "should have thrown") test here

@@ +487,5 @@
> +        ok(true, "Caught Exception");
> +        next();
> +      }
> +      reqx.onsuccess = function () {
> +        ok(false, "Shouldn't get here");

instead of doing this, set reqx to null in the beginning, and check that it's still null inside the catch-block before calling next().

@@ +489,5 @@
> +      }
> +      reqx.onsuccess = function () {
> +        ok(false, "Shouldn't get here");
> +      };
> +      reqx.onerror = onFailure;

And remove this line.

@@ +491,5 @@
> +        ok(false, "Shouldn't get here");
> +      };
> +      reqx.onerror = onFailure;
> +    }
> +    window.setTimeout(cb, 1);

Use SimpleTest.executeSoon()

Comment 2

[Gregor Wagner [:gwagner]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/99878dcae029

Comment 3

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/99878dcae029