Fix copying of static files

models.json

@@ -41,6 +41,11 @@
         "config.*.json"
       ],
       "relations": {
+        "package": {
+          "type": "hasOne",
+          "model": "PackageDefinition",
+          "foreignKey": "componentName"
+        },
         "models": {
           "type": "hasMany",
           "model": "ModelDefinition",

models/workspace-entity.js

@@ -54,9 +54,13 @@ WorkspaceEntity.allFromCache = function(cache) {
     .map(this.getFromCache.bind(this, cache));
 }
 
-WorkspaceEntity.getPath = function(app, obj) {
+WorkspaceEntity.getPath = function(componentName, obj) {
   if(obj.configFile) return obj.configFile;
-  return path.join(app, this.settings.defaultConfigFile);
+  return path.join(componentName, this.settings.defaultConfigFile);
+}
+
+WorkspaceEntity.getDir = function(componentName, obj) {
+  return path.dirname(WorkspaceEntity.getPath(componentName, obj));
 }
 
 WorkspaceEntity.getConfigFile = function(componentName, obj) {

models/workspace.js

@@ -125,17 +125,13 @@ Workspace.addComponent = function(options, cb) {
   steps.push(function(cb) {
     fs.exists(fileTemplatesDir, function(exists) {
       if(exists) {
-        steps.push(copyTemplateFiles);
-        cb();
+        ncp(fileTemplatesDir, dest, cb);
       } else {
         cb();
       }
     });
   });
 
-  function copyTemplateFiles(cb) {
-    ncp(fileTemplatesDir, dest, cb);
-  }
   function setComponentName(obj) {
     if(Array.isArray(obj)) {
       obj.forEach(function(item) {

templates/api-server/component.js

@@ -21,4 +21,4 @@ template.package = {
   "optionalDependencies": {
     "loopback-explorer": "^1.1.0"
   }
-};
+};

templates/rest/template/boot/authentication.js

@@ -1,4 +1,4 @@
 var rest = require('../rest');
 
 // enable authentication
-server.enableAuth();
+rest.enableAuth();

templates/rest/template/rest.js

@@ -8,4 +8,4 @@ var server = module.exports = loopback();
 boot(server, __dirname);
 
 // middleware
-server.use(loopback.rest());
+server.use(loopback.rest());

templates/server/template/server.js

@@ -42,4 +42,4 @@ app.start = function() {
 // start the server if `$ node server.js`
 if (require.main === module) {
   app.start();
-}
+}

test/component-definition.js

@@ -0,0 +1,8 @@
+var async = require('async');
+var app = require('../app');
+var ComponentDefinition = app.models.ComponentDefinition;
+var ConfigFile = app.models.ConfigFile;
+
+describe('ComponentDefinition', function() {
+
+});

test/support.js

@@ -12,6 +12,10 @@ expectFileExists = function (file) {
   assert(fs.existsSync(file), file + ' does not exist');
 }
 
+getPath = function(relativePath) {
+  return ConfigFile.toAbsolutePath(relativePath);
+}
+
 expectValueInJSONFile = function(file, propertyPath, val) {
   var contents = fs.readFileSync(file, 'utf8');
   var obj = JSON.parse(contents);

test/workspace.js

@@ -3,6 +3,7 @@ var app = require('../app');
 var TestDataBuilder = require('loopback-testing').TestDataBuilder;
 var Workspace = app.models.Workspace;
 var ConfigFile = app.models.ConfigFile;
+var ComponentDefinition = app.models.ComponentDefinition;
 
 describe('Workspace', function() {
   describe('Workspace.getAvailableTemplates(callback)', function() {
@@ -16,6 +17,19 @@ describe('Workspace', function() {
     });
   });
 
+  describe('Workspace.addComponent(options, cb)', function () {
+    beforeEach(givenEmptySandbox);
+    beforeEach(function(done) {
+      Workspace.addComponent({
+        template: 'rest'
+      }, done);
+    });
+    it('should add the static component files', function () {
+      expectFileExists(getPath('rest/rest.js'));
+      expectFileExists(getPath('rest/boot/authentication.js'));
+    });
+  });
+
   describe('Workspace.createFromTemplate(templateName, callback)', function() {
     beforeEach(givenBasicWorkspace);
     beforeEach(findAllEntities);
@@ -46,13 +60,23 @@ describe('Workspace', function() {
       expect(dataSourceNames).to.contain('mail');
       expect(dataSourceNames).to.contain('db');
     });
+
+    it('should create a runnable set of components', function (done) {
+      ComponentDefinition.findOne({
+        where: {
+          name: '.'
+        }
+      }, function(err, component) {
+        if(err) return done(err);
+        done();
+      });
+    });
   });
 
   describe('Workspace.listUseableConnectors(cb)', function () {
     it('should return a list of connectors in package.json');
   });
 
-
   describe('project.listAvailableConnectors(cb)', function() {
     before(function(done) {
       Workspace.listAvailableConnectors(function(err, list) {