fix code review comments

microsoft · Jun 18, 2018 · ac5ee79 · ac5ee79
1 parent 0bd432e
commit ac5ee79
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/client/unittests/nosetest/services/argsService.ts b/src/client/unittests/nosetest/services/argsService.ts
@@ -53,7 +53,7 @@ export class ArgumentsService implements IArgumentsService {
         if (Array.isArray(argumentToRemoveOrFilter)) {
             argumentToRemoveOrFilter.forEach(item => {
                 if (OptionsWithArguments.indexOf(item) >= 0) {
-                    optionsWithoutArgsToRemove.push(item);
+                    optionsWithArgsToRemove.push(item);
                 }
                 if (OptionsWithoutArguments.indexOf(item) >= 0) {
                     optionsWithoutArgsToRemove.push(item);

diff --git a/src/client/unittests/pytest/services/argsService.ts b/src/client/unittests/pytest/services/argsService.ts
@@ -55,7 +55,7 @@ export class ArgumentsService implements IArgumentsService {
         if (Array.isArray(argumentToRemoveOrFilter)) {
             argumentToRemoveOrFilter.forEach(item => {
                 if (OptionsWithArguments.indexOf(item) >= 0) {
-                    optionsWithoutArgsToRemove.push(item);
+                    optionsWithArgsToRemove.push(item);
                 }
                 if (OptionsWithoutArguments.indexOf(item) >= 0) {
                     optionsWithoutArgsToRemove.push(item);

diff --git a/src/client/unittests/unittest/services/argsService.ts b/src/client/unittests/unittest/services/argsService.ts
@@ -38,7 +38,7 @@ export class ArgumentsService implements IArgumentsService {
         if (Array.isArray(argumentToRemoveOrFilter)) {
             argumentToRemoveOrFilter.forEach(item => {
                 if (OptionsWithArguments.indexOf(item) >= 0) {
-                    optionsWithoutArgsToRemove.push(item);
+                    optionsWithArgsToRemove.push(item);
                 }
                 if (OptionsWithoutArguments.indexOf(item) >= 0) {
                     optionsWithoutArgsToRemove.push(item);