Merge pull request #4280 from Turbo87/blueprint-tests

Update "blueprint-test-helpers" and "ember-cli"

.npmignore

@@ -16,8 +16,9 @@
 .watchmanconfig
 bower.json
 ember-cli-build.js
-testem.json
+testem.js
 
 *.gem
 *.gemspec
 **/*.rb
+node-tests/

node-tests/fixtures/addon/package/package.json

@@ -1,49 +1,20 @@
 {
   "name": "my-addon",
   "version": "0.0.0",
-  "description": "The default blueprint for ember-cli addons.",
-  "directories": {
-    "doc": "doc",
-    "test": "tests"
-  },
-  "scripts": {
-    "build": "ember build",
-    "start": "ember server",
-    "test": "ember try:testall"
-  },
-  "repository": "",
+  "description": "Addon fixture package for ember-cli-blueprint-test-helpers",
   "engines": {
     "node": ">= 0.10.0"
   },
-  "author": "",
-  "license": "MIT",
   "devDependencies": {
-    "broccoli-asset-rev": "^2.1.2",
-    "ember-ajax": "0.6.2",
-    "ember-cli": "1.13.12",
-    "ember-cli-app-version": "^1.0.0",
-    "ember-cli-content-security-policy": "0.4.0",
-    "ember-cli-dependency-checker": "^1.1.0",
-    "ember-cli-htmlbars-inline-precompile": "^0.3.1",
-    "ember-cli-inject-live-reload": "^1.3.1",
-    "ember-cli-qunit": "^1.0.1",
-    "ember-cli-release": "0.2.3",
-    "ember-cli-sri": "^1.0.3",
-    "ember-cli-uglify": "^1.2.0",
-    "ember-data": "2.0.0",
-    "ember-disable-proxy-controllers": "^1.0.0",
-    "ember-export-application-global": "^1.0.4",
-    "ember-resolver": "^2.0.2",
-    "ember-disable-prototype-extensions": "^1.0.0",
-    "ember-try": "~0.0.8",
+    "ember-cli": "*",
     "ember-data": "*"
   },
   "keywords": [
     "ember-addon"
   ],
   "dependencies": {
-    "ember-cli-htmlbars": "^1.0.0",
-    "ember-cli-babel": "^5.1.3"
+    "ember-cli-htmlbars": "*",
+    "ember-cli-babel": "*"
   },
   "ember-addon": {
     "configPath": "tests/dummy/config"

node-tests/fixtures/app/package/package.json

@@ -1,43 +1,13 @@
 {
   "name": "my-app",
   "version": "0.0.0",
-  "description": "Small description for nested-project goes here",
+  "description": "App fixture package for ember-cli-blueprint-test-helpers",
   "private": true,
-  "directories": {
-    "doc": "doc",
-    "test": "tests"
-  },
-  "scripts": {
-    "build": "ember build",
-    "start": "ember server",
-    "test": "ember test"
-  },
-  "repository": "",
   "engines": {
     "node": ">= 0.10.0"
   },
-  "author": "",
-  "license": "MIT",
   "devDependencies": {
-    "broccoli-asset-rev": "^2.1.2",
-    "ember-ajax": "0.6.2",
-    "ember-cli": "1.13.8",
-    "ember-cli-app-version": "^1.0.0",
-    "ember-cli-babel": "^5.1.3",
-    "ember-cli-content-security-policy": "0.4.0",
-    "ember-cli-dependency-checker": "^1.1.0",
-    "ember-cli-htmlbars": "^1.0.0",
-    "ember-cli-htmlbars-inline-precompile": "^0.3.1",
-    "ember-cli-inject-live-reload": "^1.3.1",
-    "ember-cli-legacy-blueprints": "*",
-    "ember-cli-qunit": "^1.0.1",
-    "ember-cli-release": "0.2.3",
-    "ember-cli-sri": "^1.0.3",
-    "ember-cli-uglify": "^1.2.0",
-    "ember-data": "2.0.0",
-    "ember-disable-proxy-controllers": "^1.0.0",
-    "ember-export-application-global": "^1.0.4",
-    "ember-resolver": "^2.0.2",
+    "ember-cli": "*",
     "ember-data": "*"
   }
 }

node-tests/nodetest-runner.js

@@ -2,27 +2,21 @@
 
 var glob = require('glob');
 var Mocha = require('mocha');
-var Promise = require('ember-cli/lib/ext/promise');
+var RSVP = require('rsvp');
 var rimraf = require('rimraf');
 var mochaOnlyDetector = require('mocha-only-detector');
 
-process.env.EMBER_DATA_SKIP_VERSION_CHECKING_DO_NOT_USE_THIS_ENV_VARIABLE = true;
-
-if (process.env.EOLNEWLINE) {
-  require('os').EOL = '\n';
-}
-
 rimraf.sync('.node_modules-tmp');
 rimraf.sync('.bower_components-tmp');
 
 var root = 'node-tests/{blueprints,acceptance,unit}';
-var _checkOnlyInTests = Promise.denodeify(mochaOnlyDetector.checkFolder.bind(null, root + '/**/*{-test}.js'));
+var _checkOnlyInTests = RSVP.denodeify(mochaOnlyDetector.checkFolder.bind(null, root + '/**/*{-test}.js'));
 var optionOrFile = process.argv[2];
 var mocha = new Mocha({
   timeout: 5000,
   reporter: 'spec'
 });
-var testFiles = glob.sync(root + '**/*-test.js');
+var testFiles = glob.sync(root + '/**/*-test.js');
 /*var jshintPosition = testFiles.indexOf('tests/unit/jshint-test.js');
 var jshint = testFiles.splice(jshintPosition, 1);
 
@@ -62,7 +56,7 @@ function ciVerificationStep() {
   if (process.env.CI === 'true') {
     return checkOnlyInTests();
   } else {
-    return Promise.resolve();
+    return RSVP.Promise.resolve();
   }
 }
 

package.json

@@ -54,17 +54,17 @@
     "broccoli-uglify-sourcemap": "^1.0.1",
     "broccoli-yuidoc": "^2.1.0",
     "ember-ajax": "0.7.1",
-    "ember-cli": "2.3.0",
+    "ember-cli": "2.4.1",
     "ember-cli-app-version": "^1.0.0",
-    "ember-cli-blueprint-test-helpers": "^0.9.0",
+    "ember-cli-blueprint-test-helpers": "0.10.2",
     "ember-cli-dependency-checker": "^1.2.0",
     "ember-cli-htmlbars": "^1.0.3",
     "ember-cli-htmlbars-inline-precompile": "^0.2.0",
     "ember-cli-inject-live-reload": "^1.3.1",
     "ember-cli-internal-test-helpers": "^0.8.1",
     "ember-cli-qunit": "^1.2.1",
     "ember-cli-release": "0.2.3",
-    "ember-cli-sri": "^2.0.0",
+    "ember-cli-sri": "^2.1.0",
     "ember-cli-uglify": "^1.2.0",
     "ember-dev": "emberjs/ember-dev#bcfb9c3487ec2fd58b932394a15ce16fd9cf7eed",
     "ember-disable-prototype-extensions": "^1.1.0",
@@ -76,12 +76,12 @@
     "ember-try": "^0.1.2",
     "ember-watson": "^0.7.0",
     "github": "^0.2.4",
-    "glob": "^5.0.13",
+    "glob": "5.0.13",
     "loader.js": "^4.0.0",
-    "mocha": "^2.3.4",
+    "mocha": "2.4.5",
     "mocha-only-detector": "0.0.2",
-    "rimraf": "^2.3.2",
-    "rsvp": "^3.1.0"
+    "rimraf": "2.5.2",
+    "rsvp": "3.2.1"
   },
   "peerDependencies": {
     "ember-inflector": "^1.9.4"

testem.json → testem.js

@@ -1,4 +1,5 @@
-{
+/*jshint node:true*/
+module.exports = {
   "framework": "qunit",
   "test_page": "tests/index.html?hidepassed",
   "disable_watching": true,
@@ -10,4 +11,4 @@
     "PhantomJS",
     "Chrome"
   ]
-}
+};

tests/helpers/module-for-acceptance.js

@@ -13,11 +13,11 @@ export default function(name, options = {}) {
     },
 
     afterEach() {
-      destroyApp(this.application);
-
       if (options.afterEach) {
         options.afterEach.apply(this, arguments);
       }
+
+      destroyApp(this.application);
     }
   });
 }

tests/index.html

@@ -21,10 +21,10 @@
     {{content-for "body"}}
     {{content-for "test-body"}}
 
+    <script src="testem.js" integrity=""></script>
     <script src="assets/vendor.js"></script>
     <script src="assets/test-support.js"></script>
     <script src="assets/dummy.js"></script>
-    <script src="testem.js" integrity=""></script>
     <script src="assets/tests.js"></script>
     <script src="assets/test-loader.js"></script>