Merge pull request #62 from JakeSidSmith/total-coverage

Total coverage

Author: ekonstantinidis

src/js/__tests__/stores/sound-notification.js

@@ -74,4 +74,12 @@ describe('Tests for SoundNotificationStore', function () {
 
   });
 
+  it('Should reopen gitify when a notification is clicked', function () {
+    var nativeNotification = SoundNotificationStore.newNotification('Test', 'Hello, world!');
+
+    expect(nativeNotification.onclick.toString().indexOf('reopen-window') >= 0).toBe(true);
+
+    nativeNotification.onclick();
+  });
+
 });

src/js/stores/sound-notification.js

@@ -17,19 +17,24 @@ var SoundNotificationStore = Reflux.createStore({
     audio.play();
   },
 
+  newNotification: function (title, body) {
+    var nativeNotification = new Notification(title, {
+      body: body
+    });
+    nativeNotification.onclick = function () {
+      ipc.sendChannel('reopen-window');
+    };
+    return nativeNotification;
+  },
+
   showNotification: function (countNew, response, latestNotification) {
     var title = (countNew == 1 ?
       'Gitify - ' + latestNotification.full_name :
       'Gitify');
     var body = (countNew == 1 ?
       latestNotification.subject :
       'You\'ve got ' + countNew + ' notifications.');
-    var nativeNotification = new Notification(title, {
-      body: body
-    });
-    nativeNotification.onclick = function () {
-      ipc.sendChannel('reopen-window');
-    };
+    this.newNotification(title, body);
   },
 
   onIsNewNotification: function (response) {