Add pytest-xdist's -n to the list of supported options for pytest (#6565

)

* Add pytest-xdist's n to the supported options list
* Better comment
* Add some tests
* Add some more tests

news/2 Fixes/6293.md

@@ -0,0 +1 @@
+Add pytest-xdist's -n option to the list of supported pytest options.

src/client/testing/pytest/services/argsService.ts

@@ -8,6 +8,7 @@ import { IServiceContainer } from '../../../ioc/types';
 import { IArgumentsHelper, IArgumentsService, TestFilter } from '../../types';
 
 const OptionsWithArguments = ['-c', '-k', '-m', '-o', '-p', '-r', '-W',
+    '-n', // -n is a pytest-xdist option
     '--assert', '--basetemp', '--capture', '--color', '--confcutdir',
     '--cov', '--cov-config', '--cov-fail-under', '--cov-report',
     '--deselect', '--dist', '--doctest-glob',

src/test/testing/pytest/pytest.argsService.unit.test.ts

@@ -90,4 +90,23 @@ suite('ArgsService: pytest', () => {
         expect(testDirs[1]).to.equal(dir);
         expect(testDirs[2]).to.equal(dir2);
     });
+    test('Test getting the list of known options for pytest', () => {
+        const knownOptions = argumentsService.getKnownOptions();
+        expect(knownOptions.withArgs.length).to.not.equal(0);
+        expect(knownOptions.withoutArgs.length).to.not.equal(0);
+    });
+    test('Test calling ArgumentsService.getOptionValue with the option followed by the value', () => {
+        const knownOptionsWithValues = argumentsService.getKnownOptions().withArgs;
+        knownOptionsWithValues.forEach(option => {
+            const args = ['--foo', '--bar', 'arg1', option, 'value1'];
+            expect(argumentsService.getOptionValue(args, option)).to.deep.equal('value1');
+        });
+    });
+    test('Test calling ArgumentsService.getOptionValue with the inline option syntax', () => {
+        const knownOptionsWithValues = argumentsService.getKnownOptions().withArgs;
+        knownOptionsWithValues.forEach(option => {
+            const args = ['--foo', '--bar', 'arg1', `${option}=value1`];
+            expect(argumentsService.getOptionValue(args, option)).to.deep.equal('value1');
+        });
+    });
 });