Makes Twilio credentials optional

Previously, validating that the account sid looked like an account sid
lead to the option being mandatory. This updates the prompt to not worry
if the account sid is an empty string.

It also adds a --skip-credentials option to avoid the tool prompting at
all.

Fixes #3.

Author: philnash

src/cli.js

@@ -19,15 +19,20 @@ function cli(cwd) {
         type: 'string'
       });
       command.options({
-        accountSid: {
+        'account-sid': {
           alias: 'a',
           describe: 'The Account SID for your Twilio account',
           type: 'string'
         },
-        authToken: {
+        'auth-token': {
           alias: 't',
           describe: 'Your Twilio account Auth Token',
           type: 'string'
+        },
+        'skip-credentials': {
+          describe: "Don't ask for Twilio account credentials",
+          type: 'boolean',
+          default: false
         }
       });
     },

src/create-twilio-function/prompt.js

@@ -1,17 +1,21 @@
 const inquirer = require('inquirer');
 
 async function promptForAccountDetails(config) {
+  if (config.skipCredentials) return {};
   const questions = [];
-  if (!config.accountSid) {
+  if (typeof config.accountSid === 'undefined') {
     questions.push({
       type: 'input',
       name: 'accountSid',
       message: 'Twilio Account SID',
-      validate: input =>
-        input.startsWith('AC') ? true : 'An Account SID starts with "AC".'
+      validate: input => {
+        return input.startsWith('AC') || input === ''
+          ? true
+          : 'An Account SID starts with "AC".';
+      }
     });
   }
-  if (!config.authToken) {
+  if (typeof config.authToken === 'undefined') {
     questions.push({
       type: 'password',
       name: 'authToken',

src/create-twilio-function/prompt.test.js

@@ -7,7 +7,7 @@ describe('promptForAccountDetails', () => {
   test(`should ask for an accountSid if not specified`, async () => {
     inquirer.prompt = jest.fn(() =>
       Promise.resolve({
-        accountSid: 'test-sid',
+        accountSid: 'AC1234',
         authToken: 'test-auth-token'
       })
     );
@@ -21,13 +21,12 @@ describe('promptForAccountDetails', () => {
   test(`should ask for an auth if not specified`, async () => {
     inquirer.prompt = jest.fn(() =>
       Promise.resolve({
-        accountSid: 'test-sid',
         authToken: 'test-auth-token'
       })
     );
     await promptForAccountDetails({
       name: 'function-test',
-      accountSid: 'test-sid'
+      accountSid: 'AC1234'
     });
     expect(inquirer.prompt).toHaveBeenCalledTimes(1);
     expect(inquirer.prompt).toHaveBeenCalledWith(expect.any(Array));
@@ -36,16 +35,24 @@ describe('promptForAccountDetails', () => {
   test(`should not prompt if account sid and auth token specified`, async () => {
     inquirer.prompt = jest.fn(() =>
       Promise.resolve({
-        accountSid: 'test-sid',
+        accountSid: 'AC1234',
         authToken: 'test-auth-token'
       })
     );
     await promptForAccountDetails({
       name: 'function-test',
-      accountSid: 'test-sid',
-      authToken: 'test-auth-token'
+      accountSid: 'AC5678',
+      authToken: 'other-test-token'
     });
     expect(inquirer.prompt).toHaveBeenCalledTimes(1);
     expect(inquirer.prompt).toHaveBeenCalledWith([]);
   });
+
+  test(`should not ask for credentials if skip-credentials flag is true`, async () => {
+    inquirer.prompt = jest.fn(() => {});
+    await promptForAccountDetails({
+      skipCredentials: true
+    });
+    expect(inquirer.prompt).not.toHaveBeenCalled();
+  });
 });