feat(cli): remove default option for controller (#768)

The CLI no longer has a default ('new') name for the controller, and will prompt for a name until it is given properly.
loopbackio · Dec 1, 2017 · cc41fd6 · cc41fd6
1 parent f67db6c
commit cc41fd6
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 27 deletions.
diff --git a/packages/cli/lib/artifact-generator.js b/packages/cli/lib/artifact-generator.js
@@ -61,9 +61,8 @@ module.exports = class ArtifactGenerator extends Generator {
       {
         type: 'input',
         name: 'name',
-        message: utils.toClassName(this.artifactInfo.type) + ' name:', // capitalization
+        message: utils.toClassName(this.artifactInfo.type) + ' class name:', // capitalization
         when: this.artifactInfo.name === undefined,
-        default: this.artifactInfo.defaultName,
         validate: utils.validateClassName,
       },
     ];

diff --git a/packages/cli/test/controller.js b/packages/cli/test/controller.js
@@ -51,31 +51,6 @@ describe('lb4 controller', () => {
       assert.fileContent(tmpDir + withInputName, /constructor\(\) {}/);
     });
   });
-  describe('without input', () => {
-    let tmpDir;
-    before(() => {
-      return helpers
-        .run(generator)
-        .inTmpDir(dir => {
-          tmpDir = dir;
-          fs.writeFileSync(
-            path.join(tmpDir, 'package.json'),
-            JSON.stringify({
-              keywords: ['loopback'],
-            })
-          );
-        })
-        .withPrompts(noInputProps);
-    });
-    it('writes correct file name', () => {
-      assert.file(tmpDir + noInputName);
-      assert.noFile(tmpDir + templateName);
-    });
-    it('scaffolds correct files', () => {
-      assert.fileContent(tmpDir + noInputName, /class NewController/);
-      assert.fileContent(tmpDir + noInputName, /constructor\(\) {}/);
-    });
-  });
   describe('with arg', () => {
     let tmpDir;
     before(() => {