Merge pull request #509 from unexpectedjs/v11

V11 (Major)

Author: papandreou

.travis.yml

@@ -4,20 +4,26 @@ env:
   - TARGET=travis
 
 node_js:
-  - "0.10"
-  - "0.12"
-  - "4"
-  - "5"
   - "6"
   - "7"
+  - "8.4.0"
   - "9"
   - "10"
 
 matrix:
   include:
-  - name: Travis main 8.4.0
+  - name: Lint (8.4.0)
     node_js: "8.4.0"
-    env: TARGET=travis-main
+    env: TARGET=lint
+  - name: Jest (8.4.0)
+    node_js: "8.4.0"
+    env: TARGET=test-jest
+  - name: Jasmine (8.4.0)
+    node_js: "8.4.0"
+    env: TARGET=test-jasmine
+  - name: Coverage (8.4.0)
+    node_js: "8.4.0"
+    env: TARGET=travis-coverage
   - name: BrowserStack IE11
     node_js: "8.4.0"
     env: TARGET=test-browserstack-ie11

Makefile

@@ -1,6 +1,8 @@
 REPORTER = dot
 
 TARGETS ?= unexpected.js unexpected.js.map
+.PHONY: unexpected.js
+.SECONDARY: unexpected.js.map
 
 CHEWBACCA_THRESHOLD ?= 25
 
@@ -25,7 +27,6 @@ build/externaltests: externaltests/*
 
 build: build/lib build/test build/externaltests
 
-.PHONY: ${TARGETS}
 ${TARGETS}: build
 	./node_modules/.bin/rollup --config rollup.config.js --sourcemap --format umd --name weknowhow.expect -o unexpected.js build/lib/index.js
 
@@ -50,15 +51,12 @@ test-jest:
 ifeq ($(MODERN_NODE), true)
 	./node_modules/.bin/jest
 else
-	./node_modules/.bin/jest -c test/jest.es5.config.json
+	./node_modules/.bin/jest --rootDir . -c test/jest.es5.config.json
 endif
 
-test-jest-if-supported-node-version:
-	@node-version-gte-6 && make test-jest || echo Skipping, jest is unsupported with node $(shell node --version)
-
 .PHONY: test
 test: test-sources
-	@./node_modules/.bin/mocha --opts $(MOCHA_OPTS) $(TEST_SOURCES) $(TEST_SOURCE_MARKDOWN)
+	@./node_modules/.bin/mocha --opts $(MOCHA_OPTS) $(TEST_SOURCES) $(TEST_SOURCES_MARKDOWN)
 
 nyc-includes:
 ifeq ($(MODERN_NODE), true)
@@ -84,12 +82,12 @@ test-chrome-headless: ${TARGETS}
 test-browserstack-%: ${TARGETS}
 	@./node_modules/.bin/karma start --browsers=$* --single-run
 
-.PHONY: travis-main
-travis-main: clean lint test test-jasmine test-jest coverage
+.PHONY: travis-coverage
+travis-coverage: clean coverage
 	-<coverage/lcov.info ./node_modules/coveralls/bin/coveralls.js
 
 .PHONY: travis
-travis: test test-jest-if-supported-node-version
+travis: test
 
 .PHONY: git-dirty-check
 git-dirty-check:
@@ -117,7 +115,7 @@ changelog: git-dirty-check
 	fi
 
 .PHONY: release-%
-release-%: git-dirty-check lint ${TARGETS} test-chrome-headless test-jasmine test-jest-if-supported-node-version commit-unexpected deploy-site
+release-%: git-dirty-check lint ${TARGETS} test-chrome-headless test-jasmine test-jest commit-unexpected
 	IS_MAKE_RELEASE=yes npm version $*
 	make changelog
 	@echo $* release ready to be publised to NPM

documentation/api/addType.md

@@ -299,32 +299,32 @@ block. The outputs below shows the contrast between setting the
 inlineDiff = true;
 expect(
   {
-    'John Doe': new Person('John Doe', 42),
-    'Jane Doe': new Person('Janie Doe', 24)
+    JohnDoe: new Person('John Doe', 42),
+    JaneDoe: new Person('Janie Doe', 24)
   },
   'to equal',
   {
-    'John Doe': new Person('John Doe', 42),
-    'Jane Doe': new Person('Jane Doe', 24)
+    JohnDoe: new Person('John Doe', 42),
+    JaneDoe: new Person('Jane Doe', 24)
   }
 );
 ```
 
 ```output
 expected
 {
-  'John Doe': new Person('John Doe', 42),
-  'Jane Doe': new Person('Janie Doe', 24)
+  JohnDoe: new Person('John Doe', 42),
+  JaneDoe: new Person('Janie Doe', 24)
 }
 to equal
 {
-  'John Doe': new Person('John Doe', 42),
-  'Jane Doe': new Person('Jane Doe', 24)
+  JohnDoe: new Person('John Doe', 42),
+  JaneDoe: new Person('Jane Doe', 24)
 }
 
 {
-  'John Doe': new Person('John Doe', 42),
-  'Jane Doe': new Person(
+  JohnDoe: new Person('John Doe', 42),
+  JaneDoe: new Person(
     'Janie Doe', // should be 'Jane Doe'
                  // -Janie Doe
                  // +Jane Doe
@@ -337,39 +337,39 @@ to equal
 inlineDiff = false;
 expect(
   {
-    'John Doe': new Person('John Doe', 42),
-    'Jane Doe': new Person('Janie Doe', 24)
+    JohnDoe: new Person('John Doe', 42),
+    JaneDoe: new Person('Janie Doe', 24)
   },
   'to equal',
   {
-    'John Doe': new Person('John Doe', 42),
-    'Jane Doe': new Person('Jane Doe', 24)
+    JohnDoe: new Person('John Doe', 42),
+    JaneDoe: new Person('Jane Doe', 24)
   }
 );
 ```
 
 ```output
 expected
 {
-  'John Doe': new Person('John Doe', 42),
-  'Jane Doe': new Person('Janie Doe', 24)
+  JohnDoe: new Person('John Doe', 42),
+  JaneDoe: new Person('Janie Doe', 24)
 }
 to equal
 {
-  'John Doe': new Person('John Doe', 42),
-  'Jane Doe': new Person('Jane Doe', 24)
+  JohnDoe: new Person('John Doe', 42),
+  JaneDoe: new Person('Jane Doe', 24)
 }
 
 {
-  'John Doe': new Person('John Doe', 42),
-  'Jane Doe': new Person('Janie Doe', 24) // should equal new Person('Jane Doe', 24)
-                                          //
-                                          // new Person(
-                                          //   'Janie Doe', // should be 'Jane Doe'
-                                          //                // -Janie Doe
-                                          //                // +Jane Doe
-                                          //   24
-                                          // )
+  JohnDoe: new Person('John Doe', 42),
+  JaneDoe: new Person('Janie Doe', 24) // should equal new Person('Jane Doe', 24)
+                                       //
+                                       // new Person(
+                                       //   'Janie Doe', // should be 'Jane Doe'
+                                       //                // -Janie Doe
+                                       //                // +Jane Doe
+                                       //   24
+                                       // )
 }
 ```
 

documentation/api/async.md

@@ -99,5 +99,5 @@ If you are using the default error mode, you don't have to specify the
 error message as it is just thrown away.
 
 The diff is a method that will create a custom diff lazily. To get a
-better understanding of the diff method see the [type](/api/addType)
+better understanding of the diff method see the [type](../addType/)
 documentation.

documentation/api/fail.md

@@ -10,7 +10,7 @@ fulfilled. If any of the promises is rejected, the resulting promise will be
 rejected with the same error.
 
 This method is usually used in combination with
-[expect.promise.settle](/api/promise-settle).
+[expect.promise.settle](../promise-settle/).
 
 Let's make an asynchronous assertion that we can use for the examples:
 
@@ -30,7 +30,7 @@ expect.addAssertion('to be a number after a short delay', function(
 });
 ```
 
-See the [promise](/api/promise) documentation for more details on how
+See the [promise](../promise/) documentation for more details on how
 expect.promise works.
 
 The following code snippet creates a promise that is fulfilled when all the

documentation/api/promise-all.md

@@ -27,7 +27,7 @@ expect.addAssertion('to be a number after a short delay', function(
 });
 ```
 
-See the [promise](/api/promise) documentation for more details on how
+See the [promise](../promise/) documentation for more details on how
 expect.promise works.
 
 The following code snippet creates a promise that will be fulfilled when any

documentation/api/promise-any.md

@@ -28,7 +28,7 @@ expect.addAssertion('to be a number after a short delay', function(
 });
 ```
 
-See the [promise](/api/promise) documentation for more details on how
+See the [promise](../promise/) documentation for more details on how
 expect.promise works.
 
 The following code snippet creates a promise that is rejected when any

documentation/api/promise-settle.md

@@ -1,6 +1,6 @@
 # expect.promise(promiseBody)
 
-This method is used inside [addAssertion](/api/addAssertion) to create
+This method is used inside [addAssertion](../addAssertion/) to create
 a promise from the given body function.
 
 Signature:

documentation/api/promise.md

@@ -37,7 +37,7 @@ to extend the instance.
 expect.use(require('unexpected-sinon'));
 ```
 
-Notice that it is usually a good idea to [clone](../clone) the instance before
+Notice that it is usually a good idea to [clone](../clone/) the instance before
 extending it with plugins.
 
 ## Example
@@ -101,4 +101,4 @@ expected [7,13] to contain 27
 ```
 
 For more inspiration you can look at the source for existing plugins.
-See [the plugin page](/plugins/) for a list.
+See [the plugin page](../../plugins/) for a list.