Merge pull request #24 from ekonstantinidis/jest-tests

Initialize Tests (Jest)

Author: ekonstantinidis

.gitignore

@@ -2,5 +2,7 @@
 npm-debug.log
 
 build/
+coverage/
 node_modules/
 Gitify.app/
+

.travis.yml

@@ -1,10 +1,18 @@
 language: node_js
 
 node_js:
-- '0.12'
+- '0.10.36'
+
+addons:
+  code_climate:
+    repo_token: 7c19339cc50fdc0dceb2acc8e022e939d845e1676eee84ca9115c5a9b6796992
 
 install:
 - npm install
+- npm install -g codeclimate-test-reporter
 
 script:
 - npm test
+
+after_script:
+  - codeclimate < ./coverage/lcov.info

README.md

@@ -1,10 +1,17 @@
-Gitify [![Build Status](https://travis-ci.org/ekonstantinidis/gitify.svg?branch=master)](https://travis-ci.org/ekonstantinidis/gitify)
-==========
-GitHub Notifications on your menu bar.
+# Gitify
+[![travis][travis-image]][travis-url]
+[![cc-coverage][cc-coverage-image]][cc-coverage-url]
+
+[travis-image]: https://travis-ci.org/ekonstantinidis/gitify.svg?branch=master
+[travis-url]: https://travis-ci.org/ekonstantinidis/gitify
+[cc-coverage-image]: https://codeclimate.com/github/ekonstantinidis/gitify/badges/coverage.svg
+[cc-coverage-url]: https://codeclimate.com/github/ekonstantinidis/gitify/coverage
+
+### GitHub Notifications on your menu bar.
 
 ![Gitify](images/press.png)
 
-### Installation
+### Download
 You can download Gitify from the [releases](https://github.com/ekonstantinidis/gitify/releases) page. Currently only supports OS X.
 
 ### Prerequisites

package.json

@@ -4,13 +4,88 @@
   "description": "GitHub Notifications on your menu bar.",
   "main": "main.js",
   "scripts": {
-    "build-js": "npm run mkdir -p build/js && browserify -t reactify src/js/app.js -o build/js/app.js",
+    "build-js": "mkdir -p build/js && browserify -t reactify src/js/app.js -o build/js/app.js",
     "build": "npm install && mkdir -p build/js && grunt build && npm run build-js",
     "watch-js": "watchify -t reactify src/js/app.js -o build/js/app.js -v",
     "watch": "grunt build && npm build && npm run watch-js | grunt watch",
     "start": "electron .",
     "dist": "rm -rf Gitify.app/ && electron-packager . Gitify --platform=darwin --arch=x64 --version=0.27.1 --icon=images/app-icon.icns --prune --ignore=src",
-    "test": "jsxhint --reporter node_modules/jshint-stylish/stylish.js 'src/**/*.js', 'index.js' --exclude 'Gruntfile.js'"
+    "test": "jsxhint --reporter node_modules/jshint-stylish/stylish.js 'src/**/*.js', 'index.js' --exclude 'Gruntfile.js' && jscs 'src/js/' && jest",
+    "coveralls": "cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js"
+  },
+  "jshintConfig": {
+    "browserify": true,
+    "unused": true,
+    "undef": true,
+    "globals": {
+      "console": false
+    }
+  },
+  "jscsConfig": {
+    "esprima": "esprima-fb",
+    "disallowMultipleVarDecl": "exceptUndefined",
+    "validateQuoteMarks": {
+      "mark": "'",
+      "escape": false
+    },
+    "validateParameterSeparator": ", ",
+    "validateIndentation": 2,
+    "requireSpacesInFunction": {
+      "beforeOpeningRoundBrace": true,
+      "beforeOpeningCurlyBrace": true
+    },
+    "requireSpacesInConditionalExpression": {
+      "afterTest": true,
+      "beforeConsequent": true,
+      "afterConsequent": true,
+      "beforeAlternate": true
+    },
+    "disallowTrailingWhitespace": true,
+    "maximumLineLength": 100,
+    "requireCurlyBraces": true,
+    "disallowMixedSpacesAndTabs": true,
+    "requireSpaceBeforeBinaryOperators": true,
+    "safeContextKeyword": [
+      "self"
+    ],
+    "disallowMultipleLineBreaks": true,
+    "disallowMultipleLineStrings": true,
+    "disallowSpaceAfterObjectKeys": true,
+    "disallowNewlineBeforeBlockStatements": true,
+    "disallowTrailingComma": true,
+    "requireCommaBeforeLineBreak": true,
+    "requireSpaceBetweenArguments": true,
+    "requireSpaceBeforeBlockStatements": true,
+    "requireSpaceBeforeObjectValues": true,
+    "requireSpacesInForStatement": true
+  },
+  "jest": {
+    "scriptPreprocessor": "src/js/__tests__/preprocessor.js",
+    "collectCoverage": true,
+    "collectCoverageOnlyFrom": {
+      "src/js/actions/actions.js": true,
+      "src/js/components/login.js": true,
+      "src/js/components/navigation.js": true,
+      "src/js/components/notification.js": true,
+      "src/js/components/notifications.js": true,
+      "src/js/components/repository.js": true,
+      "src/js/components/footer.js": true,
+      "src/js/stores/auth.js": true,
+      "src/js/stores/notifications.js": true
+    },
+    "unmockedModulePathPatterns": [
+      "node_modules/react",
+      "node_modules/react-tools",
+      "node_modules/underscore"
+    ],
+    "testPathDirs": [
+      "src/js/__tests__/"
+    ],
+    "testPathIgnorePatterns": [
+      "src/js/__tests__/preprocessor.js",
+      "src/js/__tests__/__mocks__/",
+      "node_modules/"
+    ]
   },
   "repository": {
     "type": "git",
@@ -36,6 +111,7 @@
     "octicons": "=2.2.0",
     "react": "=0.13.3",
     "react-router": "=0.13.3",
+    "react-tools": "=0.13.3",
     "reactify": "=1.1.1",
     "reflux": "=0.2.7",
     "reloading": "0.0.6",
@@ -46,11 +122,14 @@
   "devDependencies": {
     "electron-packager": "=4.1.2",
     "electron-prebuilt": "=0.27.1",
+    "esprima-fb": "=15001.1.0-dev-harmony-fb",
     "grunt": "=0.4.5",
     "grunt-contrib-clean": "=0.6.0",
     "grunt-contrib-copy": "=0.8.0",
     "grunt-contrib-less": "=1.0.1",
     "grunt-contrib-watch": "=0.6.1",
+    "jest-cli": "=0.4.5",
+    "jscs": "^1.13.1",
     "jshint-stylish": "=1.0.2",
     "jsxhint": "=0.15.0",
     "less": "=2.5.1"

src/js/__tests__/__mocks__/react-router.js

@@ -0,0 +1,25 @@
+// mock of react router used in the tests
+var React = require('react');
+
+function Router (params) {
+  this.name = 'router';
+  this.params = params;
+}
+
+Router.prototype.makeHref = function () {};
+Router.prototype.transitionTo = function () {};
+Router.prototype.isActive = function () {
+  return false;
+};
+
+Router.prototype.getCurrentParams = function () {
+  return this.params;
+};
+
+Router.Link = React.createClass({
+  render: function () {
+    return (<a/>);
+  }
+});
+
+module.exports = Router;

src/js/__tests__/__mocks__/superagent.js

@@ -0,0 +1,81 @@
+/* global jest */
+
+var superagent = jest.genMockFromModule('superagent');
+
+var Response = jest.genMockFunction().mockImplementation(function () {
+  this.status = 200;
+  this.ok = true;
+});
+
+Response.prototype.send = jest.genMockFunction();
+Response.prototype.toError = jest.genMockFunction();
+
+var Request = jest.genMockFunction().mockImplementation(function (method, url) {
+  this.method = method;
+  this.url = url;
+});
+
+Request.prototype.accept = jest.genMockFunction().mockReturnThis();
+Request.prototype.set = jest.genMockFunction().mockReturnThis();
+Request.prototype.send = jest.genMockFunction().mockReturnThis();
+Request.prototype.field = jest.genMockFunction().mockReturnThis();
+Request.prototype.query = jest.genMockFunction().mockReturnThis();
+Request.prototype.end = jest.genMockFunction().mockImplementation(function (callback) {
+
+  if (superagent.mockDelay) {
+    this.delayTimer = setTimeout(callback, 0, superagent.mockError, superagent.mockResponse);
+
+    return;
+  }
+
+  callback(superagent.mockError, superagent.mockResponse);
+});
+
+Request.prototype.abort = jest.genMockFunction().mockImplementation(function () {
+  this.aborted = true;
+
+  if (this.delayTimer) {
+    clearTimeout(this.delayTimer);
+  }
+});
+
+superagent.Request = Request;
+superagent.Response = Response;
+
+superagent.mockResponse = new Response();
+superagent.mockError = null;
+superagent.mockDelay = false;
+
+function __setResponse (status, ok, body, error) {
+  var mockedResponse = jest.genMockFunction().mockImplementation(function () {
+    this.status = status;
+    this.ok = ok;
+    this.body = body;
+  });
+
+  superagent.mockError = {
+    response: {
+      body: error
+    }
+  };
+
+  superagent.mockResponse = new mockedResponse();
+}
+
+module.exports = {
+  Response: Response,
+  Request: Request,
+  post: function () {
+    return new Request();
+  },
+  get: function () {
+    return new Request();
+  },
+  patch: function () {
+    return new Request();
+  },
+  put: function () {
+    return new Request();
+  },
+  __setResponse: __setResponse
+};

src/js/__tests__/basic.js

@@ -0,0 +1,7 @@
+/* global describe, it, expect */
+
+describe('basic', function () {
+  it('adds two and two and gets four', function () {
+    expect(2 + 2).toBe(4);
+  });
+});

src/js/__tests__/components/footer.js

@@ -0,0 +1,42 @@
+/* global jest, describe, beforeEach, it, expect */
+
+jest.dontMock('reflux');
+jest.dontMock('../../actions/actions.js');
+jest.dontMock('../../components/footer.js');
+
+var React = require('react/addons');
+var TestUtils = React.addons.TestUtils;
+
+describe('Test for Footer', function () {
+
+  var Actions, Footer;
+
+  beforeEach(function () {
+    // Mock Electron's window.require
+    // and remote.require('shell')
+    window.require = function () {
+      return {
+        require: function () {
+          return {
+            openExternal: function () {
+              return {};
+            }
+          };
+        }
+      };
+    };
+
+    Actions = require('../../actions/actions.js');
+    Footer = require('../../components/footer.js');
+  });
+
+  it('Should load the footer', function () {
+
+    var instance = TestUtils.renderIntoDocument(<Footer />);
+    expect(instance.openRepoBrowser).toBeDefined();
+
+    instance.openRepoBrowser();
+
+  });
+
+});