#7 Tests for erector-set template object generation

Also added .vscode to .gitignore

Author: gonzofish

.gitignore

@@ -1,5 +1,6 @@
 .idea/
 .nyc_output
+.vscode
 coverage
 *.tgz
 debug.log

test/tools/utilities/files.spec.js

@@ -121,9 +121,118 @@ tap.test('#deleteFolder', (suite) => {
 });
 
 tap.test('#getTemplates', (suite) => {
-    // return a list erector template objects
-    // use .destination if provided
-    // set template to blank if .blank is true
+    const create = (templates) =>
+        files.getTemplates('/root', '/root/current', templates);
+    let mockResolve;
+
+    suite.beforeEach((done) => {
+        mockResolve = sinon.stub(path, 'resolve');
+        mockResolve.callsFake(function() {
+            return Array.prototype.slice.call(arguments)
+                .join('/');
+        });
+
+        done();
+    });
+
+    suite.afterEach((done) => {
+        mockResolve.restore();
+
+        done();
+    });
+
+    suite.test('should return a list of erector template objects', (test) => {
+        test.plan(1);
+
+        const templates = create([
+            { name: 'pizza', overwrite: false },
+            { name: 'broccoli', update: true },
+            { name: 'burger' }
+        ]);
+
+        test.deepEqual(templates, [
+            {
+                check: undefined,
+                destination: '/root/pizza',
+                overwrite: false,
+                template: '/root/current/templates/pizza',
+                update: undefined
+            },
+            {
+                check: undefined,
+                destination: '/root/broccoli',
+                overwrite: undefined,
+                template: '/root/current/templates/broccoli',
+                update: true
+            },
+            {
+                check: undefined,
+                destination: '/root/burger',
+                overwrite: undefined,
+                template: '/root/current/templates/burger',
+                update: undefined
+            }
+        ]);
+
+        test.end();
+    });
+
+    suite.test('should utilize the destination field if provided', (test) => {
+        test.plan(1);
+
+        const templates = create([
+            { name: 'pizza', destination: '/the/pizzeria/palace' },
+            { name: 'burger' }
+        ]);
+
+        test.deepEqual(templates, [
+            {
+                check: undefined,
+                destination: '/the/pizzeria/palace',
+                overwrite: undefined,
+                template: '/root/current/templates/pizza',
+                update: undefined
+            },
+            {
+                check: undefined,
+                destination: '/root/burger',
+                overwrite: undefined,
+                template: '/root/current/templates/burger',
+                update: undefined
+            }
+        ]);
+
+        test.end();
+    });
+
+    suite.test('should provide a blank template if .blank is truthy', (test) => {
+        test.plan(1);
+
+        const templates = create([
+            { name: 'pizza' },
+            { blank: true, name: 'broccoli' }
+        ]);
+
+        test.deepEqual(templates, [
+            {
+                check: undefined,
+                destination: '/root/pizza',
+                overwrite: undefined,
+                template: '/root/current/templates/pizza',
+                update: undefined
+            },
+            {
+                check: undefined,
+                destination: '/root/broccoli',
+                overwrite: undefined,
+                template: undefined,
+                update: undefined
+            }
+        ]);
+
+        test.end();
+    });
+
     suite.end();
 });
 

tools/utilities/files.js

@@ -23,7 +23,7 @@ const removePath = (folder, file) => {
 exports.getTemplates = (rootDir, directory, filenames) => filenames.map((filename) => ({
     check: filename.check,
     destination: filename.destination || path.resolve(rootDir, filename.name),
-    template: filename.blank ? undefined : [directory, 'templates', filename.name].join(path.sep),
+    template: filename.blank ? undefined : path.resolve(directory, 'templates', filename.name),
     update: filename.update,
     overwrite: filename.overwrite
 }));