Add jasmine & protractor integration tests #82

.npmrc

@@ -0,0 +1 @@
+loglevel = silent

.travis.yml

@@ -1,5 +1,16 @@
+sudo: required
+dist: trusty
+addons:
+  apt:
+    sources:
+      - google-chrome
+    packages:
+      - google-chrome-stable
 language: node_js
 node_js:
   - '6.5'
 script:
-  -  npm run coveralls
+  -  npm run coveralls && npm run test:integration
+before_install:
+  - export DISPLAY=:99.0
+  - sh -e /etc/init.d/xvfb start

CHANGELOG.md

@@ -1,6 +1,7 @@
 # Unreleased
 
 * Add an option to remove duration from execution summary.
+* Add jasmine & protractor integration tests [#82](https://github.com/bcaudan/jasmine-spec-reporter/issues/82)
 
 # 3.1.0
 

README.md

@@ -31,6 +31,7 @@ You can customize the output of the reporter yourself: [see how](docs/customize-
 * install dependencies: `npm install`
 * compile sources: `npm run tsc`
 * launch unit tests: `npm test`
+* launch integration tests: `npm test:integration`
 
 # Contribution
 

examples/node/.npmrc

@@ -0,0 +1 @@
+loglevel = silent

examples/node/spec/helpers/reporter.js

@@ -3,6 +3,9 @@ const SpecReporter = require('jasmine-spec-reporter').SpecReporter;
 jasmine.getEnv().clearReporters();               // remove default reporter logs
 jasmine.getEnv().addReporter(new SpecReporter({  // add jasmine-spec-reporter
   spec: {
-    displayPending: true
+    displayPending: true,
+  },
+  summary: {
+    displayDuration: false,
   }
 }));

examples/protractor/.npmrc

@@ -0,0 +1 @@
+loglevel = silent

examples/protractor/README.md

@@ -37,7 +37,5 @@ You can find an example in this directory:
 
 ```bash
 npm install
-npm run webdriver-update # if needed
-npm run webdriver-start
 npm test
 ```

examples/protractor/package.json

@@ -3,9 +3,8 @@
   "version": "1.0.0",
   "description": "a protractor example",
   "scripts": {
-    "webdriver-update": "node node_modules/protractor/bin/webdriver-manager update",
-    "webdriver-start": "node node_modules/protractor/bin/webdriver-manager start",
-    "test": "node node_modules/protractor/bin/protractor protractor.conf.js"
+    "postinstall": "webdriver-manager update",
+    "test": "protractor protractor.conf.js"
   },
   "author": "Bastien Caudan",
   "license": "Apache-2.0",

examples/protractor/protractor.conf.js

@@ -1,4 +1,6 @@
 let SpecReporter = require('jasmine-spec-reporter').SpecReporter;
+// https://github.com/angular/protractor/issues/1451
+require('protractor/built/logger').Logger.logLevel = 1;
 
 exports.config = {
   framework: 'jasmine2',
@@ -22,6 +24,9 @@ exports.config = {
     jasmine.getEnv().addReporter(new SpecReporter({
       spec: {
         displayStacktrace: true
+      },
+      summary: {
+        displayDuration: false
       }
     }));
   }

examples/typescript/.npmrc

@@ -0,0 +1 @@
+loglevel = silent

package.json

@@ -8,7 +8,9 @@
     "prepublish": "tsc",
     "pretest": "tsc && tsc -p spec/tsconfig.json",
     "test": "jasmine",
+    "test:integration": "npm run pretest && npm run update-examples && jasmine JASMINE_CONFIG_PATH=spec/support/jasmine-integration.json",
     "posttest": "npm run lint",
+    "update-examples": "(cd examples/node && rm -rf node_modules && npm install) && (cd examples/protractor && rm -rf node_modules && npm install)",
     "lint": "tslint -c tslint.json 'src/**/*.ts' && tslint -c tslint.json 'spec/**/*.ts'",
     "coverage": "npm run pretest && rm -rf coverage && istanbul cover jasmine && npm run remap-coverage",
     "remap-coverage": "remap-istanbul -i coverage/coverage.json -o coverage/lcov-ts.info -t lcovonly && remap-istanbul -i coverage/coverage.json -o coverage/html-report -t html",
@@ -40,6 +42,7 @@
     "@types/jasmine": "2.5.40",
     "@types/node": "6.0.58",
     "coveralls": "2.11.15",
+    "diff": "3.2.0",
     "istanbul": "0.4.5",
     "jasmine": "2.5.2",
     "jasmine-core": "2.5.2",

spec/integration/integration.spec.ts

@@ -0,0 +1,40 @@
+import { exec } from "child_process";
+import { readFileSync } from "fs";
+const JsDiff = require("diff");
+
+// https://github.com/jasmine/jasmine-npm/issues/85
+process.env.JASMINE_CONFIG_PATH = "spec/support/jasmine.json";
+const TIMEOUT_INCREASED = 120000;
+
+const filter = (diff) => {
+    const value = (element) => {
+        return element.value;
+    };
+    const added = diff.filter((element) => {
+        return element.added === true;
+    }).map(value);
+    const removed = diff.filter((element) => {
+        return element.removed === true;
+    }).map(value);
+    return { added, removed };
+};
+
+describe("Integration", () => {
+    it("with jasmine-npm should be ok", (done) => {
+        exec("cd examples/node && npm test", (error, stdout) => {
+            const expected = readFileSync("spec/resources/node-example.out", { encoding: "utf-8" });
+            const { added, removed } = filter(JsDiff.diffLines(expected, stdout));
+            expect(added).toEqual(removed);
+            done();
+        });
+    }, TIMEOUT_INCREASED);
+
+    it("with protractor should be ok", (done) => {
+        exec("cd examples/protractor && npm test", (error, stdout) => {
+            const expected = readFileSync("spec/resources/node-protractor.out", { encoding: "utf-8" });
+            const { added, removed } = filter(JsDiff.diffLines(expected, stdout));
+            expect(added).toEqual(removed);
+            done();
+        });
+    }, TIMEOUT_INCREASED);
+});

spec/resources/node-example.out

@@ -0,0 +1,56 @@
+Spec started
+
+  first suite
+    ✓ should be ok
+    * should be pending
+    ✗ should failed
+      - Expected true to be false.
+    ✓ should be ok
+
+  second suite
+    * should be pending
+    ✓ should be ok
+
+    first child suite
+
+      first grandchild suite
+        ✗ should failed
+          - Expected true to be false.
+          - Expected true to be false.
+        ✗ should failed
+          - Expected true to be false.
+        ✓ should be ok
+
+      second grandchild suite
+        ✗ should failed
+          - Expected true to be false.
+        ✓ should be ok
+
+**************************************************
+*                    Failures                    *
+**************************************************
+
+1) first suite should failed
+  - Expected true to be false.
+
+2) second suite first child suite first grandchild suite should failed
+  - Expected true to be false.
+  - Expected true to be false.
+
+3) second suite first child suite first grandchild suite should failed
+  - Expected true to be false.
+
+4) second suite first child suite second grandchild suite should failed
+  - Expected true to be false.
+
+**************************************************
+*                    Pending                     *
+**************************************************
+
+1) first suite should be pending
+  will work soon
+
+2) second suite should be pending
+  Temporarily disabled with xit
+
+Executed 9 of 11 specs (4 FAILED) (2 PENDING).

spec/resources/node-protractor.out

@@ -0,0 +1,6 @@
+Spec started
+
+  angularjs homepage
+    ✓ should greet the named user
+
+Executed 1 of 1 spec SUCCESS.

spec/support/jasmine-integration.json

@@ -0,0 +1,9 @@
+{
+  "spec_dir": "spec",
+  "spec_files": [
+    "integration/**/*.spec.js"
+  ],
+  "helpers": [
+    "helpers/**/*.js"
+  ]
+}

spec/support/jasmine.json

@@ -1,8 +1,7 @@
 {
   "spec_dir": "spec",
   "spec_files": [
-    "**/*.spec.js",
-    "**/*.spec.coffee"
+    "unit/**/*.spec.js"
   ],
   "helpers": [
     "helpers/**/*.js"

spec/tsconfig.json

@@ -4,7 +4,6 @@
     "target": "es5"
   },
   "include": [
-    "./**/*.ts",
-    "../typings/**/*.ts"
+    "./**/*.ts"
   ]
 }

spec/configuration-parser.spec.ts → spec/unit/configuration-parser.spec.ts

@@ -1,5 +1,5 @@
-import { ConfigurationParser } from "../built/configuration-parser";
-import { TestProcessor } from "./helpers/test-processor";
+import { ConfigurationParser } from "../../built/configuration-parser";
+import { TestProcessor } from "../helpers/test-processor";
 
 describe("Configuration parser", () => {
     it("should have a default configuration", () => {