Add "Too many arguments" message to info and improve code coverage (#…

…2235)

__tests__/commands/info.js

@@ -1,5 +1,6 @@
 /* @flow */
 
+import * as reporters from '../../src/reporters/index.js';
 import {run as info} from '../../src/cli/commands/info.js';
 import {BufferReporter} from '../../src/reporters/index.js';
 import Config from '../../src/config.js';
@@ -47,7 +48,7 @@ const expectedKeys = [
 ];
 
 test.concurrent('without arguments and in directory containing a valid package file', (): Promise<void> => {
-  return runInfo(['.'], {}, 'local',
+  return runInfo([], {}, 'local',
     (config, output): ?Promise<void> => {
       const actualKeys = Object.keys(output);
       expectedKeys.forEach((key) => expect(actualKeys).toContain(key));
@@ -127,3 +128,21 @@ test.concurrent('with two arguments and second argument path containing non-exis
     },
   );
 });
+
+test.concurrent('reports error on invalid package names', (): Promise<void> => {
+  const reporter = new reporters.ConsoleReporter({});
+  return runInfo(['YARN.invalid.package.name.YARN'], {}, '',
+    (config, output): ?Promise<void> => {
+      expect(output).toContain(reporter.lang('infoFail', 2));
+    },
+  );
+});
+
+test.concurrent('reports error with too many arguments', (): Promise<void> => {
+  const reporter = new reporters.ConsoleReporter({});
+  return runInfo(['yarn', 'version', 'extra.invalid.arg'], {}, '',
+    (config, output): ?Promise<void> => {
+      expect(output).toContain(reporter.lang('tooManyArguments', 2));
+    },
+  );
+});

src/cli/commands/info.js

@@ -43,6 +43,7 @@ export async function run(
  args: Array<string>,
 ): Promise<void> {
   if (args.length > 2) {
+    reporter.error(reporter.lang('tooManyArguments', 2));
     return;
   }