fix: container test should not run app vulns scan by default

Mattlk13 · Oct 6, 2020 · b62c1c0 · b62c1c0
1 parent 2fae1cb
commit b62c1c0
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 28 deletions.
diff --git a/src/cli/modes.ts b/src/cli/modes.ts
@@ -19,7 +19,6 @@ const modes: Record<string, ModeData> = {
     config: (args): [] => {
       args['docker'] = true;
       args['experimental'] = true;
-      args['app-vulns'] = args.json ? false : true;
 
       return args;
     },

diff --git a/test/modes.spec.ts b/test/modes.spec.ts
@@ -118,7 +118,6 @@ describe('when have a valid mode and command', () => {
       _: [],
       docker: true,
       experimental: true,
-      'app-vulns': true,
       'package-manager': 'pip',
     };
     const cliCommand = 'container';
@@ -160,7 +159,6 @@ describe('when have a valid mode, command and exists a command alias', () => {
       _: [],
       docker: true,
       experimental: true,
-      'app-vulns': true,
       'package-manager': 'pip',
     };
     const cliCommand = 'container';
@@ -176,31 +174,6 @@ describe('when have a valid mode, command and exists a command alias', () => {
     expect(cliArgs['docker']).toBeTruthy();
     expect(cliArgs['experimental']).toBeTruthy();
   });
-
-  it('"container test" should set docker option and not app-vulns and test command', () => {
-    const expectedCommand = 't';
-    const expectedArgs = {
-      _: [],
-      json: true,
-      docker: true,
-      experimental: true,
-      'app-vulns': false,
-      'package-manager': 'pip',
-    };
-    const cliCommand = 'container';
-    const cliArgs = {
-      _: ['t'],
-      json: true,
-      'package-manager': 'pip',
-    };
-
-    const command = parseMode(cliCommand, cliArgs);
-
-    expect(command).toBe(expectedCommand);
-    expect(cliArgs).toEqual(expectedArgs);
-    expect(cliArgs['docker']).toBeTruthy();
-    expect(cliArgs['experimental']).toBeTruthy();
-  });
 });
 
 describe('when have a valid mode and not allowed command', () => {