Skip to content

Commit

Permalink
Added what should be compensation for phpunit versions
Browse files Browse the repository at this point in the history
  • Loading branch information
amnuts authored and shivammathur committed Jun 12, 2023
1 parent 1b02c00 commit 4630172
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 0 deletions.
18 changes: 18 additions & 0 deletions __tests__/tools.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -540,4 +540,22 @@ describe('Tools tests', () => {
process.env['GITHUB_TOKEN'] = token;
expect(await tools.addTools(tools_csv, '7.4', 'linux')).toContain(script);
});

it.each`
tools_csv | php_version | resolved
${'phpunit'} | ${'8.2'} | ${'/phpunit.phar'}
${'phpunit'} | ${'8.1'} | ${'/phpunit.phar'}
${'phpunit'} | ${'8.0'} | ${'/phpunit-9.6.8.phar'}
${'phpunit'} | ${'7.3'} | ${'/phpunit-9.6.8.phar'}
${'phpunit'} | ${'7.2'} | ${'/phpunit-8.5.33.phar'}
${'phpunit'} | ${'7.1'} | ${'/phpunit-7.5.20.phar'}
${'phpunit'} | ${'7.0'} | ${'/phpunit-6.5.14.phar'}
`(
'checking error: $tools_csv',
async ({tools_csv, php_version, resolved}) => {
expect(await tools.addTools(tools_csv, php_version, 'linux')).toContain(
resolved
);
}
);
});
11 changes: 11 additions & 0 deletions src/tools.ts
Original file line number Diff line number Diff line change
Expand Up @@ -392,6 +392,17 @@ export async function addPhive(data: RS): Promise<string> {
* @param data
*/
export async function addPHPUnitTools(data: RS): Promise<string> {
if (data['version'] == 'latest') {
if (/7\.3|8\.0/.test(data['php_version'])) {
data['version'] = '9.6.8';
} else if (/7\.[2-3]/.test(data['php_version'])) {
data['version'] = '8.5.33';
} else if (/7\.[1-3]/.test(data['php_version'])) {
data['version'] = '7.5.20';
} else if (/7\.[0-2]/.test(data['php_version'])) {
data['version'] = '6.5.14';
}
}
data['url'] = await getPharUrl(data);
return await addArchive(data);
}
Expand Down

0 comments on commit 4630172

Please sign in to comment.