Report an error when using both --offline and new-package

jfmengels · Oct 17, 2023 · b49d8d1 · b49d8d1
1 parent 24de02d
commit b49d8d1
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 0 deletions.
diff --git a/lib/options.js b/lib/options.js
@@ -552,6 +552,22 @@ Otherwise, I recommend you try to gain network access and initialize your config
       )
     );
   }
+
+  if (args.offline && subcommand === 'new-package') {
+    reportErrorAndExit(
+      new ErrorMessage.CustomError(
+        'COMMAND REQUIRES NETWORK ACCESS',
+        wrap(
+          `I can't use ${chalk.yellow('new-package')} in ${chalk.cyan(
+            'offline'
+          )} mode, as I need network access to perform a number of steps.
+
+I recommend you try to gain network access and try again.`,
+          80
+        )
+      )
+    );
+  }
 }
 
 function parseGitHubAuth(subcommand, gitHubAuth) {

diff --git a/test/flags.test.js b/test/flags.test.js
@@ -125,3 +125,8 @@ test('Using both --template and --offline (init)', async () => {
     const output = await TestCli.runAndExpectError('init --template jfmengels/elm-review-unused/example --offline');
     expect(output).toMatchFile(testName('template-and-offline-init'));
 });
+
+test('Using both new-package and --offline', async () => {
+  const output = await TestCli.runAndExpectError('new-package --offline');
+  expect(output).toMatchFile(testName('offline-new-package'));
+});
diff --git a/test/snapshots/flags/offline-new-package.txt b/test/snapshots/flags/offline-new-package.txt
@@ -0,0 +1,7 @@
+-- COMMAND REQUIRES NETWORK ACCESS ---------------------------------------------
+
+I can't use new-package in offline mode, as I need network access to perform a
+number of steps.
+
+I recommend you try to gain network access and try again.
+