858118 - [system] fix and reenable unit tests for update manager

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Comment 1

[Julien Wajsberg [:julienw]]

Attachment: patch v1

see also https://github.com/mozilla-b2g/gaia/pull/9016

- fix updatable tests and use the new mocks helper
- reenable update tests without any more change as I couldn't make them fail. Feel
free to disable again if this intermittently fails after all.
- fix lockscreen test for various firefox versions
---
 apps/system/test/unit/lockscreen_test.js     |   31 +++++++++-------
 apps/system/test/unit/mocks_helper.js        |   13 ++++++-
 apps/system/test/unit/setup.js               |    5 +++
 apps/system/test/unit/updatable_test.js      |   50 ++++++++++++++------------
 apps/system/test/unit/update_manager_test.js |    8 -----
 5 files changed, 63 insertions(+), 44 deletions(-)
 create mode 100644 apps/system/test/unit/setup.js

Comment 2

[Etienne Segonzac (:etienne)]

Comment on attachment 733901 [details] [diff] [review]
patch v1

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

r=me with the nit (singular)

let's give travis some work :)

::: apps/system/test/unit/updatable_test.js
@@ +64,5 @@
>      mockApp = new MockApp();
>      subject = new AppUpdatable(mockApp);
> +
> +    // keeping this line here, we might need to use it again in the future
> +    //subject._mgmt = MockAppsMgmt;

we have a pretty solid test-running env right now, it works well against it so let's remove the dead code/dead comment.

::: apps/system/test/unit/update_manager_test.js
@@ +36,5 @@
>    if (! window[mockName]) {
>      window[mockName] = null;
>    }
>  });
>  

(note that from my tests these tests are indeed deterministicly green now)

Comment 3

[Julien Wajsberg [:julienw]]

master: 3f9979fe26842d90c01e040f57edb9ff59580766

Comment 4

[Julien Wajsberg [:julienw]]

Attachment: patch v2

carrying r=etienne