fix: should only read pkg if argv.typescript not pass

eggjs · Apr 4, 2018 · e0e7fee · e0e7fee
1 parent 4b798d9
commit e0e7fee
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 8 deletions.
diff --git a/lib/command.js b/lib/command.js
@@ -19,6 +19,7 @@ class Command extends BaseCommand {
         description: 'whether enable typescript support, will load `ts-node/register` etc',
         type: 'boolean',
         alias: 'ts',
+        default: undefined,
       },
     };
   }
@@ -33,14 +34,16 @@ class Command extends BaseCommand {
     // remove unuse args
     argv.$0 = undefined;
 
-    // read `egg.typescript` from package.json
-    let baseDir = argv._[0] || argv.baseDir || cwd;
-    if (!path.isAbsolute(baseDir)) baseDir = path.join(cwd, baseDir);
-    const pkgFile = path.join(baseDir, 'package.json');
-    if (fs.existsSync(pkgFile)) {
-      const pkgInfo = require(pkgFile);
-      if (pkgInfo && pkgInfo.egg && pkgInfo.egg.typescript) {
-        argv.typescript = true;
+    // read `egg.typescript` from package.json if not pass argv
+    if (argv.typescript === undefined) {
+      let baseDir = argv._[0] || argv.baseDir || cwd;
+      if (!path.isAbsolute(baseDir)) baseDir = path.join(cwd, baseDir);
+      const pkgFile = path.join(baseDir, 'package.json');
+      if (fs.existsSync(pkgFile)) {
+        const pkgInfo = require(pkgFile);
+        if (pkgInfo && pkgInfo.egg && pkgInfo.egg.typescript === true) {
+          argv.typescript = true;
+        }
       }
     }
 

diff --git a/test/fixtures/example-ts-pkg/agent.js b/test/fixtures/example-ts-pkg/agent.js
@@ -0,0 +1,6 @@
+'use strict';
+
+
+module.exports = agent => {
+  console.log(`agent.options.typescript = ${agent.options.typescript}`);
+};
diff --git a/test/ts.test.js b/test/ts.test.js
@@ -90,6 +90,15 @@ describe('test/ts.test.js', () => {
         .end();
     });
 
+    it('should fail start app with --no-ts', () => {
+      return coffee.fork(eggBin, [ 'dev', '--no-ts' ], { cwd })
+        // .debug()
+        .expect('stdout', /agent.options.typescript = false/)
+        .expect('stdout', /started/)
+        .expect('code', 0)
+        .end();
+    });
+
     it('should start app with relative path', () => {
       return coffee.fork(eggBin, [ 'dev', './example-ts-pkg' ], { cwd: path.dirname(cwd) })
         // .debug()