From 3252bf12526088a9ec9934ba2355c59d4814f298 Mon Sep 17 00:00:00 2001 From: Robin Ward Date: Tue, 28 Apr 2015 19:24:32 -0400 Subject: [PATCH] FIX: Desktop Notifications break Qunit tests We should figure out why they can't run without the conditional in a browser. --- .../initializers/subscribe-user-notifications.js.es6 | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/assets/javascripts/discourse/initializers/subscribe-user-notifications.js.es6 b/app/assets/javascripts/discourse/initializers/subscribe-user-notifications.js.es6 index 7ee8088b5e5..aef79209832 100644 --- a/app/assets/javascripts/discourse/initializers/subscribe-user-notifications.js.es6 +++ b/app/assets/javascripts/discourse/initializers/subscribe-user-notifications.js.es6 @@ -56,7 +56,9 @@ export default { }); }); - initDesktopNotifications(bus); + if (!Ember.testing) { + initDesktopNotifications(bus); + } } } };