test: remove test coverage around --key flag

these tests were duplicative and felt out of scope of the command tests. will add proper top-level coverage for this!

also, couldn't figure out a good way to add coverage in commands without running into weird flakiness and race conditions.

__tests__/cmds/categories/create.test.ts

@@ -25,19 +25,6 @@ describe('rdme categories:create', () => {
 
   afterEach(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should error if no title provided', () => {
     return expect(run(['--key', key])).rejects.toThrow('Missing 1 required arg:\ntitle');
   });

__tests__/cmds/categories/index.test.ts

@@ -25,19 +25,6 @@ describe('rdme categories', () => {
 
   afterEach(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should return all categories for a single page', async () => {
     const getMock = getAPIMockWithVersionHeader(version)
       .persist()

__tests__/cmds/changelogs/index.test.ts

@@ -33,19 +33,6 @@ describe('rdme changelogs', () => {
 
   afterAll(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should error if no path provided', () => {
     return expect(run(['--key', key])).rejects.toThrow('Missing 1 required arg:\npath');
   });

__tests__/cmds/changelogs/single.test.ts

@@ -33,19 +33,6 @@ describe('rdme changelogs (single)', () => {
 
   afterAll(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should error if no file path provided', () => {
     return expect(run(['--key', key])).rejects.toThrow('Missing 1 required arg:\npath');
   });

__tests__/cmds/custompages/index.test.ts

@@ -33,19 +33,6 @@ describe('rdme custompages', () => {
 
   afterAll(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should error if no path provided', () => {
     return expect(run(['--key', key])).rejects.toThrow('Missing 1 required arg:\npath');
   });

__tests__/cmds/custompages/single.test.ts

@@ -33,19 +33,6 @@ describe('rdme custompages (single)', () => {
 
   afterAll(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should error if no file path provided', () => {
     return expect(run(['--key', key])).rejects.toThrow('Missing 1 required arg:\npath');
   });

__tests__/cmds/docs/index.test.ts

@@ -40,13 +40,6 @@ describe('rdme docs', () => {
 
   afterAll(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
   it('should successfully log in user via prompts if API key is not provided', async () => {
     const email = '[email protected]';
     const password = 'pass123';
@@ -623,10 +616,6 @@ describe('rdme docs', () => {
 
     afterEach(afterGHAEnv);
 
-    it('should error in CI if no API key provided', () => {
-      return expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    });
-
     it('should sync new docs directory with correct headers', async () => {
       const slug = 'new-doc';
       const id = '1234';

__tests__/cmds/docs/prune.test.ts

@@ -28,19 +28,6 @@ describe('rdme docs:prune', () => {
 
   afterAll(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should error if no folder provided', () => {
     return expect(run(['--key', key, '--version', version])).rejects.rejects.toThrow('Missing 1 required arg:\nfolder');
   });

__tests__/cmds/docs/single.test.ts

@@ -37,13 +37,6 @@ describe('rdme docs (single)', () => {
 
   afterAll(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
   it('should error if no file path provided', () => {
     return expect(run(['--key', key, '--version', version])).rejects.toThrow('Missing 1 required arg:\npath');
   });
@@ -357,10 +350,6 @@ describe('rdme docs (single)', () => {
 
     afterEach(afterGHAEnv);
 
-    it('should error in CI if no API key provided', () => {
-      return expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    });
-
     it('should sync new doc with correct headers', async () => {
       const slug = 'new-doc';
       const id = '1234';

__tests__/cmds/openapi/index.test.ts

@@ -686,11 +686,6 @@ describe('rdme openapi', () => {
   });
 
   describe('error handling', () => {
-    it('should prompt for login if no API key provided', () => {
-      prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-      return expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    });
-
     it('should error if `--create` and `--update` flags are passed simultaneously', () => {
       return expect(run(['--key', key, '--create', '--update'])).rejects.toThrow(
         '--update=true cannot also be provided when using --create',
@@ -1196,10 +1191,6 @@ describe('rdme openapi', () => {
 
     afterEach(afterGHAEnv);
 
-    it('should error in CI if no API key provided', () => {
-      return expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    });
-
     it('should error out if multiple possible spec matches were found', () => {
       return expect(run(['--key', key, '--version', version])).rejects.toStrictEqual(
         new Error('Multiple API definitions found in current directory. Please specify file.'),

__tests__/cmds/versions/create.test.ts

@@ -25,19 +25,6 @@ describe('rdme versions:create', () => {
 
   afterEach(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should error if no version provided', () => {
     return expect(run(['--key', key])).rejects.toThrow('Missing 1 required arg:\nversion');
   });

__tests__/cmds/versions/delete.test.ts

@@ -26,19 +26,6 @@ describe('rdme versions:delete', () => {
 
   afterEach(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should delete a specific version', async () => {
     const mockRequest = getAPIMock()
       .delete(`/api/v1/version/${version}`)

__tests__/cmds/versions/index.test.ts

@@ -47,19 +47,6 @@ describe('rdme versions', () => {
 
   afterEach(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should make a request to get a list of existing versions', async () => {
     const mockRequest = getAPIMock()
       .get('/api/v1/version')

__tests__/cmds/versions/update.test.ts

@@ -26,19 +26,6 @@ describe('rdme versions:update', () => {
 
   afterEach(() => nock.cleanAll());
 
-  it('should prompt for login if no API key provided', async () => {
-    const consoleInfoSpy = vi.spyOn(console, 'info').mockImplementation(() => {});
-    prompts.inject(['this-is-not-an-email', 'password', 'subdomain']);
-    await expect(run()).rejects.toStrictEqual(new Error('You must provide a valid email address.'));
-    consoleInfoSpy.mockRestore();
-  });
-
-  it('should error in CI if no API key provided', async () => {
-    process.env.TEST_RDME_CI = 'true';
-    await expect(run()).rejects.toStrictEqual(new Error('No project API key provided. Please use `--key`.'));
-    delete process.env.TEST_RDME_CI;
-  });
-
   it('should update a specific version object', async () => {
     const versionToChange = '1.1.0';
     const renamedVersion = '1.1.0-update';