Bug 873397 - [Buri][STK]idle mode text should still display after vie…

…w (tests - mock without timeout)
frsela · Apr 16, 2014 · b6660d4 · b6660d4
1 parent 6c8262a
commit b6660d4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/apps/system/test/unit/icc_worker_test.js b/apps/system/test/unit/icc_worker_test.js
@@ -1,6 +1,7 @@
 'use strict';
 
-/* global MocksHelper, MockNavigatorMozIccManager, MockSystemICC, icc_worker */
+/* global MocksHelper, MockNavigatorMozIccManager, MockSystemICC, icc_worker,
+          MockNotifications */
 
 requireApp('system/test/unit/mock_l10n.js');
 requireApp('system/test/unit/mock_system_icc.js');
@@ -132,6 +133,7 @@ suite('STK (icc_worker) >', function() {
       done();
     };
     launchStkCommand(stkTestCommands.STK_CMD_SET_UP_IDLE_MODE_TEXT);
+    MockNotifications[0].onshow();
   });
 
   test('STK_CMD_REFRESH', function(done) {

diff --git a/shared/test/unit/mocks/mock_notification.js b/shared/test/unit/mocks/mock_notification.js
@@ -1,5 +1,7 @@
 'use strict';
 
+var MockNotifications = [];
+
 /**
  * This mock partly implements a Web Notification contructor, see
  * https://developer.mozilla.org/en-US/docs/Web/API/notification
@@ -12,10 +14,7 @@ function MockNotification(title, options) {
   this.tag = options.tag || undefined;
   this.mEvents = {};
 
-  var self = this;
-  setTimeout(function() {
-    self.onshow();
-  });
+  MockNotifications.push(this);
 }
 
 MockNotification.prototype.close = function() {