Knobs for unit-testing extensions (#10178)

Converted `test-extensions-unit.bat` to javascript to add a small QoL improvement that I think people will find useful: specifying which extension(s) you want to run tests for instead of commenting out lines of a script.

In addition to existing behavior being mostly preserved (node test-extensions-unit.js defaults to running all unit tests), you can specify one or more:

```
node test-extensions-unit.js sql-database-projects
node test-extensions-unit.js sql-database-projects schema-compare
```
This commit is contained in:
Benjin Dubishar
2020-05-04 13:07:52 -07:00
committed by GitHub
parent 3e5b7fd3e6
commit 5498c1c3a9
3 changed files with 131 additions and 5 deletions

View File

@@ -11,12 +11,12 @@ echo %VSCODEEXTENSIONSDIR%
:: Figure out which Electron to use for running tests
if "%INTEGRATION_TEST_ELECTRON_PATH%"=="" (
:: Run out of sources: no need to compile as code.sh takes care of it
:: Run out of sources: no need to compile as code.bat takes care of it
set INTEGRATION_TEST_ELECTRON_PATH=.\scripts\code.bat
echo "Running integration tests out of sources."
echo "Running unit tests out of sources."
) else (
:: Run from a built: need to compile all test extensions
:: Run from a build: need to compile all test extensions
call yarn gulp compile-extension:admin-tool-ext-win
call yarn gulp compile-extension:agent
call yarn gulp compile-extension:azurecore
@@ -29,12 +29,12 @@ if "%INTEGRATION_TEST_ELECTRON_PATH%"=="" (
call yarn gulp compile-extension:machine-learning
call yarn gulp compile-extension:sql-database-projects
echo "Running integration tests with '%INTEGRATION_TEST_ELECTRON_PATH%' as build."
echo "Running unit tests with '%INTEGRATION_TEST_ELECTRON_PATH%' as build."
)
:: Default to only running stable tests if test grep isn't set
if "%ADS_TEST_GREP%" == "" (
echo Running stable tests only
echo "Running stable tests only"
set ADS_TEST_GREP=@UNSTABLE@
SET ADS_TEST_INVERT_GREP=1
)

View File

@@ -0,0 +1,72 @@
/*---------------------------------------------------------------------------------------------
* Copyright (c) Microsoft Corporation. All rights reserved.
* Licensed under the Source EULA. See License.txt in the project root for license information.
*--------------------------------------------------------------------------------------------*/
const execSync = require('child_process').execSync;
const tmp = require('tmp');
const fs = require('fs-extra');
const path = require('path');
const os = require('os');
const extensionList = [
'admin-tool-ext-win',
'agent',
'azurecore',
'cms',
'dacpac',
'schema-compare',
//'mssql',
'notebook',
'resource-deployment',
'machine-learning-services',
'sql-database-projects'];
let argv = require('yargs')
.command('$0 [extensions...]')
.choices('extensions', extensionList)
.default('extensions', extensionList)
.strict().help().wrap(null).version(false).argv;
// set up environment
const VSCODEUSERDATADIR = tmp.dirSync({ prefix: 'adsuser' }).name;
const VSCODEEXTENSIONSDIR = tmp.dirSync({ prefix: 'adsext' }).name;
console.log(VSCODEUSERDATADIR);
console.log(VSCODEEXTENSIONSDIR);
if (!process.env.INTEGRATION_TEST_ELECTRON_PATH) {
process.env.INTEGRATION_TEST_ELECTRON_PATH = path.join(__dirname, '..', 'scripts', os.platform() === 'win32' ? 'code.bat' : 'code.sh');
console.log('Running unit tests out of sources.');
}
else {
for (const ext of argv.extensions) {
console.log(execSync(`yarn gulp compile-extension:${ext}`, { encoding: 'utf-8' }));
}
console.log(`Running unit tests with '${process.env.INTEGRATION_TEST_ELECTRON_PATH}' as build.`);
}
if (!process.env.ADS_TEST_GREP) {
console.log('Running stable tests only');
process.env.ADS_TEST_GREP = '@UNSTABLE@';
process.env.ADS_TEST_INVERT_GREP = 1;
}
// execute tests
for (const ext of argv.extensions) {
console.log('*'.repeat(ext.length + 23));
console.log(`*** starting ${ext} tests ***`);
console.log('*'.repeat(ext.length + 23));
const command = `${process.env.INTEGRATION_TEST_ELECTRON_PATH} --extensionDevelopmentPath=${path.join(__dirname, '..', 'extensions', ext)} --extensionTestsPath=${path.join(__dirname, '..', 'extensions', ext, 'out', 'test')} --user-data-dir=${VSCODEUSERDATADIR} --extensions-dir=${VSCODEEXTENSIONSDIR} --remote-debugging-port=9222 --disable-telemetry --disable-crash-reporter --disable-updates --nogpu`;
console.log(execSync(command, {stdio: 'inherit'}));
}
// clean up
fs.remove(VSCODEUSERDATADIR, { recursive: true });
fs.remove(VSCODEEXTENSIONSDIR, { recursive: true });