Skip to content

Commit 4210a47

Browse files
authored
fix: add support for ${workspaceFolder} in --configuration argument (#411)
1 parent 9182549 commit 4210a47

2 files changed

Lines changed: 13 additions & 1 deletion

File tree

packages/phpunit/src/Configuration/Configuration.test.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,4 +59,13 @@ describe('Configuration Test', () => {
5959

6060
expect(await configuration.getConfigurationFile()).toBeUndefined();
6161
});
62+
63+
it('get phpunit.xml with ${workspaceFolder} in --configuration arg', async () => {
64+
const root = phpUnitProject('');
65+
const configuration = new Configuration({
66+
args: [`--configuration=\${workspaceFolder}/phpunit.xml`],
67+
});
68+
69+
expect(await configuration.getConfigurationFile(root)).toEqual(join(root, 'phpunit.xml'));
70+
});
6271
});

packages/phpunit/src/Configuration/Configuration.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import { join } from 'node:path';
2+
import { VAR_WORKSPACE_FOLDER } from '../constants';
23
import { checkFileExists, findAsyncSequential, parseArguments } from '../utils';
34
import { BinaryDetector } from './BinaryDetector';
45

@@ -50,7 +51,9 @@ export abstract class BaseConfiguration implements IConfiguration {
5051
parameter.startsWith('--configuration'),
5152
);
5253
if (configuration) {
53-
const configurationFile = configuration.replace('--configuration=', '');
54+
const configurationFile = configuration
55+
.replace('--configuration=', '')
56+
.replace(VAR_WORKSPACE_FOLDER, root);
5457
files = [configurationFile, join(root, configurationFile), ...files];
5558
}
5659

0 commit comments

Comments
 (0)