Skip to content

Commit

Permalink
chore: Adapt lancher's tests to the upstream changes
Browse files Browse the repository at this point in the history
Signed-off-by: Roman Nikitenko <rnikiten@redhat.com>

rh-pre-commit.version: 2.2.0
rh-pre-commit.check-secrets: ENABLED
  • Loading branch information
RomanNikitenko committed Oct 3, 2024
1 parent f3a55ca commit 8f9b309
Show file tree
Hide file tree
Showing 8 changed files with 37 additions and 37 deletions.
File renamed without changes.
8 changes: 4 additions & 4 deletions launcher/tests/devworkspace-id.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,11 @@ import { env } from 'process';

import { DevWorkspaceId } from '../src/devworkspace-id';

const ORIGIN_WORKBENCH_WEB_MAIN = `
const ORIGIN_WORKBENCH = `
some code, some code, a mask to be replaced https://{{che-cluster}}.{{host}}/{{namespace}}/{{workspace-name}}/{{port}}/, some code
`;

const NEW_WORKBENCH_WEB_MAIN = `
const NEW_WORKBENCH = `
some code, some code, a mask to be replaced test-workspace-1234567890, some code
`;

Expand Down Expand Up @@ -49,12 +49,12 @@ describe('Test setting DevWorkspace ID to VS Code', () => {
writeFile: writeFileMock,
});

readFileMock.mockImplementation(() => ORIGIN_WORKBENCH_WEB_MAIN);
readFileMock.mockImplementation(() => ORIGIN_WORKBENCH);

const devWorkspaceId = new DevWorkspaceId();
await devWorkspaceId.configure();

expect(readFileMock).toBeCalledTimes(1);
expect(writeFileMock).toBeCalledWith('out/vs/workbench/workbench.web.main.internal.js', NEW_WORKBENCH_WEB_MAIN);
expect(writeFileMock).toBeCalledWith('out/vs/code/browser/workbench/workbench.js', NEW_WORKBENCH);
});
});
36 changes: 18 additions & 18 deletions launcher/tests/openvsix-registry.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -66,29 +66,29 @@ describe('Test Configuring of OpenVSIX registry:', () => {
// it must be cut when forming a registry URL
env.OPENVSX_REGISTRY_URL = 'https://test-openvsx.org/';

const fileWorkbenchWebMain = await fs.readFile(path.resolve(__dirname, '_data', 'workbench.web.main.internal.js'));
const fileWorkbench = await fs.readFile(path.resolve(__dirname, '_data', 'workbench.js'));

const fileWorkbenchWebMainExpected = await fs.readFile(
path.resolve(__dirname, '_data', 'workbench.web.main.test-openvsix-registry.js')
const fileWorkbenchExpected = await fs.readFile(
path.resolve(__dirname, '_data', 'workbench.test-openvsix-registry.js')
);

let savedProductJson;
let savedWorkbenchWebMain;
let savedWorkbench;

Object.assign(fs, {
readFile: async (file: string) => {
if ('product.json' === file) {
return ORIGINAL_PRODUCT_JSON;
} else if ('out/vs/workbench/workbench.web.main.internal.js' === file) {
return fileWorkbenchWebMain;
} else if ('out/vs/code/browser/workbench/workbench.js' === file) {
return fileWorkbench;
}
},

writeFile: async (file: string, data: string) => {
if ('product.json' === file) {
savedProductJson = data;
} else if ('out/vs/workbench/workbench.web.main.internal.js' === file) {
savedWorkbenchWebMain = data;
} else if ('out/vs/code/browser/workbench/workbench.js' === file) {
savedWorkbench = data;
}
},
});
Expand All @@ -98,7 +98,7 @@ describe('Test Configuring of OpenVSIX registry:', () => {
await openVSIXRegistry.configure();

expect(savedProductJson).toBe(TEST_APPLY_OPENVSIX_REISTRY);
expect(savedWorkbenchWebMain).toBe(fileWorkbenchWebMainExpected);
expect(savedWorkbench).toBe(fileWorkbenchExpected);
});

test('should proceed with CHE_PLUGIN_REGISTRY_URL if OPENVSX_REGISTRY_URL is empty', async () => {
Expand All @@ -107,29 +107,29 @@ describe('Test Configuring of OpenVSIX registry:', () => {
// it must be cut when forming a registry URL
env.CHE_PLUGIN_REGISTRY_URL = 'https://che-dogfooding.apps.che-dev.x6e0.p1.openshiftapps.com/plugin-registry/v3/';

const fileWorkbenchWebMain = await fs.readFile(path.resolve(__dirname, '_data', 'workbench.web.main.internal.js'));
const fileWorkbench = await fs.readFile(path.resolve(__dirname, '_data', 'workbench.js'));

const fileWorkbenchWebMainExpected = await fs.readFile(
path.resolve(__dirname, '_data', 'workbench.web.main.test-che-plugin-registry.js')
const fileWorkbenchExpected = await fs.readFile(
path.resolve(__dirname, '_data', 'workbench.test-che-plugin-registry.js')
);

let savedProductJson;
let savedWorkbenchWebMain;
let savedWorkbench;

Object.assign(fs, {
readFile: async (file: string) => {
if ('product.json' === file) {
return ORIGINAL_PRODUCT_JSON;
} else if ('out/vs/workbench/workbench.web.main.internal.js' === file) {
return fileWorkbenchWebMain;
} else if ('out/vs/code/browser/workbench/workbench.js' === file) {
return fileWorkbench;
}
},

writeFile: async (file: string, data: string) => {
if ('product.json' === file) {
savedProductJson = data;
} else if ('out/vs/workbench/workbench.web.main.internal.js' === file) {
savedWorkbenchWebMain = data;
} else if ('out/vs/code/browser/workbench/workbench.js' === file) {
savedWorkbench = data;
}
},
});
Expand All @@ -139,7 +139,7 @@ describe('Test Configuring of OpenVSIX registry:', () => {
await openVSIXRegistry.configure();

expect(savedProductJson).toBe(TEST_APPLY_CHE_PLUGIN_REISTRY);
expect(savedWorkbenchWebMain).toBe(fileWorkbenchWebMainExpected);
expect(savedWorkbench).toBe(fileWorkbenchExpected);
});

test('should skip if OPENVSX_REGISTRY_URL is empty but CHE_PLUGIN_REGISTRY_URL is not set', async () => {
Expand Down
22 changes: 11 additions & 11 deletions launcher/tests/webview-resources.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,11 @@ describe('Test Configuring of WebView static resources:', () => {

// the second path tests the functionality of the WebviewResources module

// load "out/vs/workbench/workbench.web.main.internal.js"
const fileWorkbenchWebMain = await fs.readFile(path.resolve(__dirname, '_data', 'workbench.web.main.internal.js'));
// load "out/vs/code/browser/workbench/workbench.js"
const fileWorkbench = await fs.readFile(path.resolve(__dirname, '_data', 'workbench.js'));

const fileWorkbenchWebMainExpected = await fs.readFile(
path.resolve(__dirname, '_data', 'workbench.web.main.test-webview-resources.js')
const fileWorkbenchExpected = await fs.readFile(
path.resolve(__dirname, '_data', 'workbench.test-webview-resources.js')
);

// load "out/vs/workbench/api/node/extensionHostProcess.js"
Expand All @@ -63,8 +63,8 @@ describe('Test Configuring of WebView static resources:', () => {
case 'product.json':
return fileProductJSON;

case 'out/vs/workbench/workbench.web.main.internal.js':
return fileWorkbenchWebMain;
case 'out/vs/code/browser/workbench/workbench.js':
return fileWorkbench;

case 'out/vs/workbench/api/node/extensionHostProcess.js':
return fileExtensionHostProcess;
Expand All @@ -73,14 +73,14 @@ describe('Test Configuring of WebView static resources:', () => {
return undefined;
});

let gotFileWorkbenchWebMain;
let gotFileWorkbench;
let gotFileExtensionHostProcess;

const writeFileMock = jest.fn();
writeFileMock.mockImplementation(async (fileName: string, data: string) => {
switch (fileName) {
case 'out/vs/workbench/workbench.web.main.internal.js':
gotFileWorkbenchWebMain = data;
case 'out/vs/code/browser/workbench/workbench.js':
gotFileWorkbench = data;

case 'out/vs/workbench/api/node/extensionHostProcess.js':
gotFileExtensionHostProcess = data;
Expand All @@ -97,7 +97,7 @@ describe('Test Configuring of WebView static resources:', () => {
expect(updateMock).toHaveBeenCalledTimes(2);

expect(updateMock).toHaveBeenCalledWith(
'out/vs/workbench/workbench.web.main.internal.js',
'out/vs/code/browser/workbench/workbench.js',
'https://{{uuid}}.vscode-cdn.net/insider/ef65ac1ba57f57f2a3961bfe94aa20481caca4c6/out/vs/workbench/contrib/webview/browser/pre/',
'https://che-dogfooding.apps.che-dev.x6e0.p1.openshiftapps.com/vgulyy/che-code-multiroot/3100/oss-dev/static/out/vs/workbench/contrib/webview/browser/pre/'
);
Expand All @@ -108,7 +108,7 @@ describe('Test Configuring of WebView static resources:', () => {
'https://che-dogfooding.apps.che-dev.x6e0.p1.openshiftapps.com/vgulyy/che-code-multiroot/3100/oss-dev/static/out/vs/workbench/contrib/webview/browser/pre/'
);

expect(fileWorkbenchWebMainExpected).toBe(gotFileWorkbenchWebMain);
expect(fileWorkbenchExpected).toBe(gotFileWorkbench);
expect(fileExtensionHostProcessExpected).toBe(gotFileExtensionHostProcess);
});
});
8 changes: 4 additions & 4 deletions launcher/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,9 @@
"tests"
],
"exclude": [
"tests/_data/workbench.web.main.internal.js",
"tests/_data/workbench.web.main.test-che-plugin-registry.js",
"tests/_data/workbench.web.main.test-openvsix-registry.js",
"tests/_data/workbench.web.main.test-webview-resources.js"
"tests/_data/workbench.js",
"tests/_data/workbench.test-che-plugin-registry.js",
"tests/_data/workbench.test-openvsix-registry.js",
"tests/_data/workbench.test-webview-resources.js"
]
}

0 comments on commit 8f9b309

Please sign in to comment.