Skip to content

Commit

Permalink
[browser tests] [launch preferences] Adding debug traces to try to un…
Browse files Browse the repository at this point in the history
…derstand what's going on

Signed-off-by: Marc Dumais <marc.dumais@ericsson.com>
  • Loading branch information
marcdumais-work committed Feb 2, 2023
1 parent 34ab392 commit 02ef44e
Show file tree
Hide file tree
Showing 3 changed files with 53 additions and 4 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci-cd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -105,8 +105,8 @@ jobs:
- name: Test (browser)
if: matrix.tests != 'skip' && runner.os == 'Linux'
run: |
yarn browser test
shell: bash
run: yarn browser test

- name: Test (electron)
if: matrix.tests != 'skip' && runner.os == 'Linux'
Expand Down
1 change: 1 addition & 0 deletions dev-packages/cli/src/test-page.ts
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,7 @@ export default async function newTestPage(options: TestPageOptions): Promise<pup
const failures = await page.evaluate(() =>
new Promise<number>(resolve => mocha.run(resolve))
);
console.log('test-page: failures: ' + failures);
if (onDidRun) {
await onDidRun(failures);
}
Expand Down
52 changes: 50 additions & 2 deletions examples/api-tests/src/launch-preferences.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -404,6 +404,7 @@ describe('Launch Preferences', function () {
* @returns {AbstractResourcePreferenceProvider | undefined} The preference provider matching the provided URI.
*/
function findProvider(uri) {
console.log('findProvider(). uri of file to modify: ' + uri.toString());
/**
* @param {PreferenceProvider} provider
* @returns {boolean} whether the provider matches the desired URI.
Expand All @@ -414,18 +415,21 @@ describe('Launch Preferences', function () {
};
for (const provider of userPreferences['providers'].values()) {
if (isMatch(provider) && provider instanceof AbstractResourcePreferenceProvider) {
console.log('provider: userPreferences');
return provider;
}
}
for (const provider of folderPreferences['providers'].values()) {
if (isMatch(provider) && provider instanceof AbstractResourcePreferenceProvider) {
console.log('provider: folderPreferences');
return provider;
}
}
/** @type {PreferenceProvider} */
const workspaceDelegate = workspacePreferences['delegate'];
if (workspaceDelegate !== folderPreferences) {
if (isMatch(workspaceDelegate) && workspaceDelegate instanceof AbstractResourcePreferenceProvider) {
console.log('provider: workspacePreferences');
return workspaceDelegate;
}
}
Expand All @@ -437,6 +441,7 @@ describe('Launch Preferences', function () {
for (const name of ['settings', 'launch']) {
promises.push((async () => {
const uri = rootUri.resolve(configPath + '/' + name + '.json');
console.log('*** deleteWorkspacePreferences : findProvider() call');
const provider = findProvider(uri);
try {
if (provider) {
Expand Down Expand Up @@ -464,12 +469,32 @@ describe('Launch Preferences', function () {
const originalShouldOverwrite = fileResourceResolver['shouldOverwrite'];

before(async () => {
console.log('*** launch-preferences:before() - enter');
// TODO: There's an occasional exception that seems to happen here:
/**
* 1) Launch Preferences
"before all" hook in "Launch Preferences":
Uncaught Error: Uncaught Error: There is no document for file:///home/<user>/theia/examples/browser/package.json
Error: There is no document for file:///home/<user>/theia/examples/browser/package.json
at LinkProviderAdapter.provideLinks (/home/<user>/theia/packages/plugin-ext/lib/plugin/languages/link-provider.js:31:35)
at /home/<user>/theia/packages/plugin-ext/lib/plugin/languages.js:332:97
at LanguagesExtImpl.withAdapter (/home/<user>/theia/packages/plugin-ext/lib/plugin/languages.js:123:20)
at LanguagesExtImpl.$provideDocumentLinks (/home/<user>/theia/packages/plugin-ext/lib/plugin/languages.js:332:21)
at /home/<user>/theia/packages/plugin-ext/lib/common/proxy-handler.js:91:71
at processTicksAndRejections (node:internal/process/task_queues:96:5)
at async RpcProtocol.handleRequest (/home/<user>/theia/packages/core/lib/common/message-rpc/rpc-protocol.js:167:28) (http://127.0.0.1:3000/vendors-node_modules_theia_monaco-editor-core_esm_vs_base_common_severity_js-node_modules_the-68fc42.js:1785)
*
*/
// fail tests if out of async happens
fileResourceResolver['shouldOverwrite'] = async () => (assert.fail('should be in sync'), false);
await deleteWorkspacePreferences();
console.log('*** launch-preferences:before() - end');
});

after(() => {
console.log('*** launch-preferences:after()');
fileResourceResolver['shouldOverwrite'] = originalShouldOverwrite;
});

Expand All @@ -495,6 +520,7 @@ describe('Launch Preferences', function () {
/** @typedef {import('@theia/monaco-editor-core/esm/vs/base/common/lifecycle').IReference<import('@theia/monaco/lib/browser/monaco-editor-model').MonacoEditorModel>} ConfigModelReference */
/** @type {ConfigModelReference[]} */
beforeEach(async () => {
console.log('beforeEach()');
/** @type {Promise<void>[]} */
const promises = [];
/**
Expand All @@ -506,6 +532,7 @@ describe('Launch Preferences', function () {
promises.push((async () => {
try {
const uri = rootUri.resolve(configPath + '/' + name + '.json');
console.log('*** ensureConfigModel : findProvider() call');
const provider = findProvider(uri);
if (provider) {
await provider['doSetPreference']('', [], value);
Expand Down Expand Up @@ -546,8 +573,11 @@ describe('Launch Preferences', function () {
});

testIt('get from rootUri', () => {
console.log(`launch-preferences:get from rootUri: enter. rootUri:\n${rootUri.toString()}`);
/** @type {any} */
const config = preferences.get('launch', undefined, rootUri.toString());
console.log(`config:\n` + JSON.stringify(config));
console.log('vs expected:\n' + JSON.stringify(expectation));
assert.deepStrictEqual(JSON.parse(JSON.stringify(config)), expectation);
});

Expand Down Expand Up @@ -595,11 +625,23 @@ describe('Launch Preferences', function () {
});

testIt('update launch', async () => {
console.log('update launch');

const inspect1 = preferences.inspect('launch');
console.log('*** pre-inspect before updating launch:\n' + JSON.stringify(inspect1));

console.log('validLaunch:\n' + JSON.stringify(validLaunch));
await preferences.set('launch', validLaunch);

const inspect = preferences.inspect('launch');
console.log('inspect:\n' + JSON.stringify(inspect));

const actual = inspect && inspect.workspaceValue;
console.log('actual:\n' + JSON.stringify(actual));

const expected = settingsLaunch && !Array.isArray(settingsLaunch) ? { ...settingsLaunch, ...validLaunch } : validLaunch;
console.log('expected:\n' + JSON.stringify(expected));

assert.deepStrictEqual(actual, expected);
});

Expand Down Expand Up @@ -655,7 +697,8 @@ describe('Launch Preferences', function () {
}
}
expected = expected || settingsLaunch;
assert.deepStrictEqual(actual && actual.workspaceValue, expected);
console.log('**_**_**_** \nactual: ' + JSON.stringify(actual && actual.workspaceValue) + '\nexpected: ' + JSON.stringify(expected));
assert.deepStrictEqual(actual && actual.workspaceValue, expected, '\nactual: ' + JSON.stringify(actual && actual.workspaceValue) + '\nexpected: ' + JSON.stringify(expected));
});

if ((launch && !Array.isArray(launch)) || (settingsLaunch && !Array.isArray(settingsLaunch))) {
Expand All @@ -664,6 +707,11 @@ describe('Launch Preferences', function () {

const actual = preferences.inspect('launch');
const actualWorkspaceValue = actual && actual.workspaceValue;
let actual_ws = "";
if (actual && actual.workspaceValue) {
actual_ws= JSON.stringify(actual.workspaceValue);
}
console.log('++++++++++++ actualWorkspaceValue = actual && actual.workspaceValue : \n+++ actual: ' + JSON.stringify(actual) + '\n+++ actual.workspaceValue: ' + actual_ws);

let expected = undefined;
if (launch) {
Expand All @@ -685,7 +733,7 @@ describe('Launch Preferences', function () {
expected = _settingsLaunch;
}
}

console.log('**_**_**_** \nactual: ' + JSON.stringify(actualWorkspaceValue) + '\nexpected: ' + JSON.stringify(expected));
assert.deepStrictEqual(actualWorkspaceValue, expected);
});
}
Expand Down

0 comments on commit 02ef44e

Please sign in to comment.