feat: auto-prefix selectors based upon configuration

addon/ng2/blueprints/component/files/__path__/__name__.component.ts

@@ -3,7 +3,7 @@ import {RouteConfig, ROUTER_DIRECTIVES, ROUTER_PROVIDERS} from 'angular2/router'
 
 @Component({
   moduleId: __moduleName,
-  selector: '<%= dasherizedModuleName %>',<% if(inlineTemplate) { %>
+  selector: '<%= selector %>',<% if(inlineTemplate) { %>
   template: `
     <p>
       <%= dasherizedModuleName %> Works!

addon/ng2/blueprints/component/index.js

@@ -1,37 +1,38 @@
 var path = require('path');
+var stringUtils = require('ember-cli-string-utils');
+var chalk = require('chalk');
 var Blueprint = require('ember-cli/lib/models/blueprint');
 var dynamicPathParser = require('../../utilities/dynamic-path-parser');
 var addBarrelRegistration = require('../../utilities/barrel-management');
 var getFiles = Blueprint.prototype.files;
 
-function validateName(name) {
-  if (name.indexOf('-') >= 0) {
-    return true;
-  } else if (name === name.toUpperCase()) {
-    return false;
-  } else if (name === name.toLowerCase()) {
-    return false;
-  }
-  return true;
-}
-
 module.exports = {
   description: '',
 
   availableOptions: [
     { name: 'flat', type: Boolean, default: false },
     { name: 'route', type: Boolean, default: false },
     { name: 'inline-template', type: Boolean, default: false, aliases: ['it'] },
-    { name: 'inline-style', type: Boolean, default: false, aliases: ['is'] }
+    { name: 'inline-style', type: Boolean, default: false, aliases: ['is'] },
+    { name: 'prefix', type: Boolean, default: true }
   ],
 
   normalizeEntityName: function (entityName) {
     var parsedPath = dynamicPathParser(this.project, entityName);
 
     this.dynamicPath = parsedPath;
 
-    if (!validateName(parsedPath.name)) {
-      throw 'Names must contain a dash either include a dash or multiCase name. (i.e. multiCase -> multi-case)';
+    var defaultPrefix = '';
+    if (this.project.ngConfig && 
+        this.project.ngConfig.defaults &&
+        this.project.ngConfig.defaults.prefix) {
+      defaultPrefix = this.project.ngConfig.defaults.prefix + '-';
+    }
+    var prefix = this.options.prefix ? defaultPrefix : '';
+    this.selector = stringUtils.dasherize(prefix + parsedPath.name);
+
+    if (this.selector.indexOf('-') === -1) {
+      this._writeStatusToUI(chalk.yellow, 'WARNING', 'selectors should contain a dash');
     }
 
     return parsedPath.name;
@@ -49,7 +50,8 @@ module.exports = {
       route: options.route,
       styleExt: this.styleExt,
       isLazyRoute: !!options.isLazyRoute,
-      isAppComponent: !!options.isAppComponent
+      isAppComponent: !!options.isAppComponent,
+      selector: this.selector
     };
   },
 

addon/ng2/blueprints/ng2/files/angular-cli.json

@@ -17,5 +17,8 @@
     "karma": {
       "config": "karma.conf.js"
     }
+  },
+  "defaults": {
+    "prefix": "app"
   }
 }

addon/ng2/blueprints/route/index.js

@@ -105,7 +105,8 @@ module.exports = {
     { name: 'default', type: Boolean, default: false },
     { name: 'lazy', type: Boolean, default: true },
     { name: 'inline-template', type: Boolean, default: false, aliases: ['it'] },
-    { name: 'inline-style', type: Boolean, default: false, aliases: ['is'] }
+    { name: 'inline-style', type: Boolean, default: false, aliases: ['is'] },
+    { name: 'prefix', type: Boolean, default: true }
   ],
 
   beforeInstall: function(options) {

lib/config/schema.json

@@ -76,6 +76,15 @@
         }
       },
       "additionalProperties": false
+    },
+    "defaults": {
+      "type": "object",
+      "properties": {
+        "prefix": {
+          "type": "string"
+        }
+      },
+      "additionalProperties": false
     }
   },
   "additionalProperties": false

tests/acceptance/generate-component.spec.js

@@ -139,17 +139,23 @@ describe('Acceptance: ng generate component', function () {
     });
   });
 
-  it('ng generate component mycomp will fail: no dashes in name', () => {
+  it('ng generate component mycomp will prefix selector', () => {
     return ng(['generate', 'component', 'mycomp'])
-      .then((exitCode) => {
-        expect(exitCode).to.equal(1);
+      .then(() => {
+        var testPath = path.join(root, 'tmp', 'foo', 'src', 'client', 'app', 'mycomp', 'mycomp.component.ts');
+        expect(existsSync(testPath)).to.equal(true);
+        var contents = fs.readFileSync(testPath, 'utf8');
+        expect(contents.indexOf('selector: \'app-mycomp\'') === -1).to.equal(false);
       });
   });
 
-  it('ng generate component MYCOMP will fail: no dashes in name', () => {
-    return ng(['generate', 'component', 'MYCOMP'])
-      .then((exitCode) => {
-        expect(exitCode).to.equal(1);
+  it('ng generate component mycomp --no-prefix will not prefix selector', () => {
+    return ng(['generate', 'component', 'mycomp', '--no-prefix'])
+      .then(() => {
+        var testPath = path.join(root, 'tmp', 'foo', 'src', 'client', 'app', 'mycomp', 'mycomp.component.ts');
+        expect(existsSync(testPath)).to.equal(true);
+        var contents = fs.readFileSync(testPath, 'utf8');
+        expect(contents.indexOf('selector: \'mycomp\'') === -1).to.equal(false);
       });
   });
 
@@ -174,4 +180,7 @@ describe('Acceptance: ng generate component', function () {
       expect(existsSync(testPath)).to.equal(false);
     });
   });
+
+  it('should ', () => {
+  });
 });