Test babel-cli --source-maps and errors (#5377)

packages/babel-cli/src/babel/options.js

@@ -162,20 +162,20 @@ export default function parseArgv(args: Array<string>) {
 
   filenames.forEach(function(filename) {
     if (!fs.existsSync(filename)) {
-      errors.push(filename + " doesn't exist");
+      errors.push(filename + " does not exist");
     }
   });
 
   if (commander.outDir && !filenames.length) {
-    errors.push("filenames required for --out-dir");
+    errors.push("--out-dir requires filenames");
   }
 
   if (commander.outFile && commander.outDir) {
-    errors.push("cannot have --out-file and --out-dir");
+    errors.push("--out-file and --out-dir cannot be used together");
   }
 
   if (commander.relative && !commander.outDir) {
-    errors.push("output directory required for --relative");
+    errors.push("--relative requires --out-dir usage");
   }
 
   if (commander.watch) {
@@ -207,7 +207,10 @@ export default function parseArgv(args: Array<string>) {
   }
 
   if (errors.length) {
-    console.error(errors.join(". "));
+    console.error("babel:");
+    errors.forEach(function (e) {
+      console.error("  " + e);
+    });
     process.exit(2);
   }
 

packages/babel-cli/test/fixtures/babel/.stdin --out-file --source-maps/options.json

@@ -1,3 +0,0 @@
-{
-  "args": ["--source-maps", "--out-file", "test.js"]
-}

packages/babel-cli/test/fixtures/babel/.stdin --source-maps inline/options.json

@@ -1,3 +0,0 @@
-{
-  "args": ["--source-maps", "inline"]
-}

packages/babel-cli/test/fixtures/babel/.stdin --source-maps inline/stdout.txt

@@ -1,7 +0,0 @@
-"use strict";
-
-arr.map(function (x) {
-  return x * x;
-});
-
-//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbInN0ZGluIl0sIm5hbWVzIjpbXSwibWFwcGluZ3MiOiI7O0FBQUEsR0FBRyxDQUFDLEdBQUcsQ0FBQyxVQUFBLENBQUM7U0FBSSxDQUFDLEdBQUcsQ0FBQztDQUFBLENBQUMsQ0FBQyIsImZpbGUiOiJzdGRvdXQiLCJzb3VyY2VzQ29udGVudCI6WyJhcnIubWFwKHggPT4geCAqIHgpOyJdfQ==

packages/babel-cli/test/fixtures/babel/dir --out-dir --out-file/in-files/src/bar/bar.js

@@ -0,0 +1,3 @@
+class Test {
+
+}

packages/babel-cli/test/fixtures/babel/dir --out-dir --out-file/in-files/src/foo.js

@@ -0,0 +1 @@
+arr.map(x => x * MULTIPLIER);

packages/babel-cli/test/fixtures/babel/dir --out-dir --out-file/options.json

@@ -0,0 +1,3 @@
+{
+  "args": ["src", "--out-dir", "lib", "--out-file", "compiled.js"]
+}

packages/babel-cli/test/fixtures/babel/dir --out-dir --out-file/stderr.txt

@@ -0,0 +1,2 @@
+babel:
+  --out-file and --out-dir cannot be used together

packages/babel-cli/test/fixtures/babel/dir --out-dir no filenames/options.json

@@ -0,0 +1,3 @@
+{
+  "args": ["--out-dir", "lib"]
+}

packages/babel-cli/test/fixtures/babel/dir --out-dir no filenames/stderr.txt

@@ -0,0 +1,2 @@
+babel:
+  --out-dir requires filenames

packages/babel-cli/test/fixtures/babel/dir --skip-initial-build no --watch/in-files/src/bar/bar.js

@@ -0,0 +1,3 @@
+class Test {
+
+}

packages/babel-cli/test/fixtures/babel/dir --skip-initial-build no --watch/in-files/src/foo.js

@@ -0,0 +1 @@
+arr.map(x => x * MULTIPLIER);

packages/babel-cli/test/fixtures/babel/dir --skip-initial-build no --watch/options.json

@@ -0,0 +1,3 @@
+{
+  "args": ["src", "--skip-initial-build", "--out-dir", "lib"]
+} 

packages/babel-cli/test/fixtures/babel/dir --skip-initial-build no --watch/stderr.txt

@@ -0,0 +1,2 @@
+babel:
+  --skip-initial-build requires --watch

packages/babel-cli/test/fixtures/babel/dir --watch no --out-dir no --out-file/in-files/src/bar/bar.js

@@ -0,0 +1,3 @@
+class Test {
+
+}

packages/babel-cli/test/fixtures/babel/dir --watch no --out-dir no --out-file/in-files/src/foo.js

@@ -0,0 +1 @@
+arr.map(x => x * MULTIPLIER);

packages/babel-cli/test/fixtures/babel/dir --watch no --out-dir no --out-file/options.json

@@ -0,0 +1,3 @@
+{
+  "args": ["src", "--watch"]
+}

packages/babel-cli/test/fixtures/babel/dir --watch no --out-dir no --out-file/stderr.txt

@@ -0,0 +1,2 @@
+babel:
+  --watch requires --out-file or --out-dir

packages/babel-cli/test/fixtures/babel/error --out-dir --out-file/stderr.txt

@@ -1 +1 @@
-filenames required for --out-dir
+--out-dir requires filenames

packages/babel-cli/test/fixtures/babel/error --out-dir/stderr.txt

@@ -1 +1 @@
-cannot have --out-file and --out-dir
+--out-file and --out-dir cannot be used together

packages/babel-cli/test/fixtures/babel/error --relative/options.json

@@ -1,4 +1,4 @@
 {
-  "args": ["--relative"],
+  "args": ["--relative", "--no-babelrc"],
   "stderrContains": true
 }

packages/babel-cli/test/fixtures/babel/error --relative/stderr.txt

@@ -1 +1 @@
-output directory required for --relative
+--relative requires --out-dir usage

packages/babel-cli/test/fixtures/babel/error wrong filename/stderr.txt

@@ -1 +1 @@
-foo.json doesn't exist
+foo.json does not exist

packages/babel-cli/test/fixtures/babel/filename --watch no filenames/in-files/script.js

@@ -0,0 +1 @@
+arr.map(x => x * MULTIPLIER);

packages/babel-cli/test/fixtures/babel/filename --watch no filenames/options.json

@@ -0,0 +1,3 @@
+{
+  "args": ["--watch", "--out-file", "out.js", "--no-babelrc"]
+}

packages/babel-cli/test/fixtures/babel/filename --watch no filenames/stderr.txt

@@ -0,0 +1,2 @@
+babel:
+  --watch requires filenames

packages/babel-cli/test/fixtures/babel/filename does not exist/options.json

@@ -0,0 +1,3 @@
+{
+  "args": ["index.js"]
+}

packages/babel-cli/test/fixtures/babel/filename does not exist/stderr.txt

@@ -0,0 +1,2 @@
+babel:
+  index.js does not exist

packages/babel-cli/test/fixtures/babel/stdin --out-file --source-maps/options.json

@@ -0,0 +1,3 @@
+{
+  "args": ["--source-maps", "--out-file", "test.js", "--no-babelrc"]
+}

packages/babel-cli/test/fixtures/babel/stdin --out-file --source-maps/out-files/test.js.map

@@ -1 +1 @@
-{"version":3,"sources":["stdin"],"names":[],"mappings":";;AAAA,GAAG,CAAC,GAAG,CAAC,UAAA,CAAC;SAAI,CAAC,GAAG,CAAC;CAAA,CAAC,CAAC","file":"test.js","sourcesContent":["arr.map(x => x * x);"]}
+{"version":3,"sources":["stdin"],"names":[],"mappings":";;AAAA,GAAG,CAAC,GAAJ,CAAQ,UAAA,CAAC;AAAA,SAAI,CAAC,GAAG,CAAR;AAAA,CAAT","file":"test.js","sourcesContent":["arr.map(x => x * x);"]}

packages/babel-cli/test/fixtures/babel/stdin --source-maps inline/options.json

@@ -0,0 +1,3 @@
+{
+  "args": ["--source-maps", "inline", "--no-babelrc"]
+}

packages/babel-cli/test/fixtures/babel/stdin --source-maps inline/stdout.txt

@@ -0,0 +1,8 @@
+"use strict";
+
+arr.map(function (x) {
+  return x * x;
+});
+//# sourceMappingURL=data:application/json;charset=utf-8;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbInN0ZGluIl0sIm5hbWVzIjpbImFyciIsIm1hcCIsIngiXSwibWFwcGluZ3MiOiI7O0FBQUFBLEdBQUcsQ0FBQ0MsR0FBSixDQUFRLFVBQUFDLENBQUM7QUFBQSxTQUFJQSxDQUFDLEdBQUdBLENBQVI7QUFBQSxDQUFUIiwic291cmNlc0NvbnRlbnQiOlsiYXJyLm1hcCh4ID0+IHggKiB4KTsiXX0=
+
+//# sourceMappingURL=data:application/json;charset=utf-8;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbXSwibmFtZXMiOltdLCJtYXBwaW5ncyI6IiIsImZpbGUiOiJzdGRvdXQifQ==