feat: add EJS as template engine and force all label to be filled

Author: movrack

package.json

@@ -44,6 +44,7 @@
     "command-line-args": "^5.0.2",
     "dedent": "^0.7.0",
     "deepmerge": "^4.2.2",
+    "ejs": "^3.1.5",
     "glob": "^7.1.3",
     "prompts": "^2.1.0",
     "semver": "^6.1.0"

src/core.js

@@ -1,5 +1,5 @@
 /* eslint-disable no-console, import/no-cycle */
-
+import { render } from 'ejs';
 import { spawn } from 'child_process';
 import deepmerge from 'deepmerge';
 import fs from 'fs';
@@ -95,9 +95,7 @@ export function resetVirtualFiles() {
  */
 export function processTemplate(_fileContent, data = {}) {
   let fileContent = _fileContent;
-  Object.keys(data).forEach(key => {
-    fileContent = fileContent.replace(new RegExp(`<%= ${key} %>`, 'g'), data[key]);
-  });
+  fileContent = render(fileContent, data, { debug: false });
   return fileContent;
 }
 

test/core.test.js

@@ -33,10 +33,12 @@ describe('processTemplate', () => {
     );
   });
 
-  it('ignores <%= keyName %> in source if not provided as data', async () => {
-    expect(processTemplate('prefix <%= name %> suffix', { foo: 'foo' })).to.equal(
-      'prefix <%= name %> suffix',
-    );
+  it('should throw an error if variable is not defined as data for source <%= keyName %> ', async () => {
+    try {
+      processTemplate('prefix <%= name %> suffix', { foo: 'foo' });
+    } catch (e) {
+      expect(e).to.be.an.instanceof(ReferenceError);
+    }
   });
 });