More tests for NotificationsStore

Author: Emmanouil Konstantinidis

src/js/__tests__/stores/notifications.js

@@ -91,7 +91,43 @@ describe('Tests for NotificationsStore', function () {
     jest.runAllTimers();
 
     var repository = NotificationsStore._notifications[0][0].repository;
+    var subject = NotificationsStore._notifications[0][0].subject;
     expect(repository.full_name).toBe('octocat/Hello-World');
+    expect(subject.title).toBe('Greetings');
+    expect(NotificationsStore.trigger).toHaveBeenCalled();
+
+  });
+
+  it('should get 0(zero) notifications from the GitHub API', function () {
+
+    spyOn(NotificationsStore, 'trigger');
+
+    var response = [];
+
+    var superagent = require('superagent');
+    superagent.__setResponse(200, 'ok', response, false);
+
+    Actions.getNotifications();
+
+    jest.runAllTimers();
+
+    expect(NotificationsStore._notifications.length).toBe(0);
+    expect(NotificationsStore.trigger).toHaveBeenCalled();
+
+  });
+
+  it('should FAIL to create a booking via the API', function () {
+
+    spyOn(NotificationsStore, 'trigger');
+    spyOn(NotificationsStore, 'onGetNotificationsFailed');
+
+    var superagent = require('superagent');
+    superagent.__setResponse(400, false);
+
+    Actions.getNotifications();
+
+    jest.runAllTimers();
+
     expect(NotificationsStore.trigger).toHaveBeenCalled();
 
   });

src/js/stores/notifications.js

@@ -23,7 +23,6 @@ var NotificationsStore = Reflux.createStore({
 
   onGetNotifications: function () {
     var self = this;
-    var tokens = AuthStore.authStatus();
 
     apiRequests
       .getAuth('https://api.github.com/notifications')
@@ -45,16 +44,17 @@ var NotificationsStore = Reflux.createStore({
     });
 
     var array = [];
-    _.map(groupedNotifications, function (obj, i) {
+    _.map(groupedNotifications, function (obj) {
       array.push(obj);
     });
 
     this._notifications = array;
     this.trigger(this._notifications);
   },
 
-  onGetNotificationsFailed: function (error) {
-    console.log('Errored.' + error);
+  onGetNotificationsFailed: function () {
+    this._notifications = [];
+    this.trigger(this._notifications);
   }
 
 });