Tests for Notifications Component

Author: Emmanouil Konstantinidis

package.json

@@ -64,6 +64,7 @@
     "collectCoverageOnlyFrom": {
       "src/js/actions/actions.js": true,
       "src/js/components/navigation.js": true,
+      "src/js/components/notifications.js": true,
       "src/js/components/footer.js": true,
       "src/js/stores/auth.js": true,
       "src/js/stores/notifications.js": true

src/js/__tests__/components/notifications.js

@@ -0,0 +1,76 @@
+/* global jest, describe, beforeEach, it, expect, spyOn */
+
+jest.dontMock('reflux');
+jest.dontMock('../../actions/actions.js');
+jest.dontMock('../../utils/api-requests');
+jest.dontMock('../../components/notification.js');
+jest.dontMock('../../stores/auth.js');
+jest.dontMock('../../stores/notifications.js');
+
+var React = require('react/addons');
+var TestUtils = React.addons.TestUtils;
+
+describe('Test for Notifications Component', function () {
+
+  var Actions, AuthStore, Notification, NotificationsStore;
+
+  beforeEach(function () {
+    // Mock Electron's window.require
+    // and remote.require('shell')
+    window.require = function () {
+      return {
+        require: function () {
+          return {
+            openExternal: function () {
+              return {};
+            }
+          };
+        }
+      };
+    };
+
+    // Mock localStorage
+    window.localStorage = {
+      item: false,
+      getItem: function () {
+        return this.item;
+      }
+    };
+
+    Actions = require('../../actions/actions.js');
+    AuthStore = require('../../stores/auth.js');
+    Notification = require('../../components/notifications.js');
+    NotificationsStore = require('../../stores/notifications.js');
+  });
+
+  it('Should render the notifications component', function () {
+
+    AuthStore.authStatus = function () {
+      return true;
+    };
+
+    var instance = TestUtils.renderIntoDocument(<Notification />);
+    expect(instance.state.loading).toBeTruthy();
+
+    var response = [[{
+      'repository': {
+        'full_name': 'ekonstantinidis/gitify',
+        'owner': {
+          'avatar_url': 'http://avatar.url'
+        }
+      },
+      'subject': {
+        'type': 'Issue'
+      }
+    }]];
+
+    NotificationsStore.trigger(response);
+    expect(instance.state.notifications.length).toBe(1);
+
+    expect(instance.state.loading).toBeTruthy();
+    instance.completedNotifications();
+    expect(instance.state.loading).toBeFalsy();
+
+  });
+
+});

src/js/components/notifications.js

@@ -34,7 +34,6 @@ var Notifications = React.createClass({
   render: function () {
     var notifications;
     var wrapperClass = 'container-fluid main-container notifications';
-    var self = this;
 
     if (_.isEmpty(this.state.notifications)) {
       wrapperClass += ' all-read';
@@ -47,7 +46,7 @@ var Notifications = React.createClass({
       );
     } else {
       notifications = (
-        this.state.notifications.map(function (obj, i) {
+        this.state.notifications.map(function (obj) {
           var repoFullName = obj[0].repository.full_name;
           return <Repository repo={obj} repoName={repoFullName} key={repoFullName} />;
         })