Merge pull request #90 from poetic/location-type-errors

Added check to tell the user if the locationType is not set to hash

index.js

@@ -81,5 +81,11 @@ module.exports = {
     }
 
     return tree;
+  },
+
+  config: function(env, baseConfig) {
+    if (env !== 'test' && baseConfig.locationType !== 'hash') {
+      throw new Error('ember-cli-cordova: You must specify the locationType as \'hash\' in your environment.js');
+    }
   }
 };

node-tests/unit/addon-test.js

@@ -0,0 +1,21 @@
+var addon = require('../../index');
+
+function expectWithConfig(baseConfig, env) {
+  return expect(addon.config.bind(addon, env || 'development', baseConfig));
+}
+
+describe('Addon', function() {
+  describe('config', function() {
+    describe('validates location type', function() {
+      it('should throw Error', function() {
+        expectWithConfig({locationType: 'auto'}).to.throw(Error);
+      });
+      it('should not throw an error', function() {
+        expectWithConfig({locationType: 'hash'}).to.not.throw(Error);
+      });
+      it('should not throw an error on test', function() {
+        expectWithConfig({locationType: 'auto'}, 'test').to.not.throw(Error);
+      });
+    });
+  });
+});

package.json

@@ -6,7 +6,7 @@
   "repository": "https://github.com/poetic/ember-cli-cordova.git",
   "main": "index.js",
   "scripts": {
-    "node-test": "mocha --require node-tests/helpers/_helper.js --reporter spec node-tests/**/*-test.js",
+    "node-test": "mocha --require node-tests/helpers/_helper.js --reporter spec node-tests/**/*-test.js node-tests/**/**/*-test.js",
     "ember-test": "ember test",
     "test": "npm run node-test && npm run ember-test"
   },

tests/dummy/config/environment.js

@@ -5,7 +5,7 @@ module.exports = function(environment) {
     modulePrefix: 'dummy',
     environment: environment,
     baseURL: '/',
-    locationType: 'auto',
+    locationType: 'hash',
     EmberENV: {
       FEATURES: {
         // Here you can enable experimental features on an ember canary build