feat: do not use native file chooser when --no-native-window-frame is provided

Contributed on behalf of STMicroelectronics

Signed-off-by: Olaf Lessenich <[email protected]>

Author: xai

examples/playwright/configs/playwright.config.ts

@@ -25,11 +25,7 @@ const config: PlaywrightTestConfig = {
     use: {
         baseURL: 'http://localhost:3000',
         browserName: 'chromium',
-        screenshot: 'only-on-failure',
-        viewport: { width: 1920, height: 1080 },
-        trace: {
-            mode: 'retain-on-failure'
-        }
+        screenshot: 'only-on-failure'
     },
     preserveOutput: 'failures-only',
     reporter: [

examples/playwright/package.json

@@ -18,9 +18,10 @@
     "lint": "eslint -c ./.eslintrc.js --ext .ts ./src",
     "lint:fix": "eslint -c ./.eslintrc.js --ext .ts ./src --fix",
     "watch": "tsc --watch",
-    "ui-tests": "yarn && playwright test --config=./configs/playwright.config.ts",
-    "ui-tests-ci": "yarn && playwright test --config=./configs/playwright.ci.config.ts",
-    "ui-tests-headful": "yarn && playwright test --config=./configs/playwright.headful.config.ts",
+    "rebuild": "yarn --cwd ../browser rebuild && yarn --cwd ../electron rebuild",
+    "ui-tests": "yarn && yarn rebuild && playwright test --config=./configs/playwright.config.ts",
+    "ui-tests-ci": "yarn && yarn rebuild && playwright test --config=./configs/playwright.ci.config.ts",
+    "ui-tests-headful": "yarn && yarn rebuild && playwright test --config=./configs/playwright.headful.config.ts",
     "ui-tests-report-generate": "allure generate ./allure-results --clean -o allure-results/allure-report",
     "ui-tests-report": "yarn ui-tests-report-generate && allure open allure-results/allure-report"
   },

examples/playwright/src/theia-app-loader.ts

@@ -15,8 +15,11 @@
 // *****************************************************************************
 
 import { Page, PlaywrightWorkerArgs, _electron as electron } from '@playwright/test';
+import * as path from 'path';
+import * as fs from 'fs';
 import { TheiaApp, TheiaAppMainPageObjects } from './theia-app';
 import { TheiaWorkspace } from './theia-workspace';
+import { OSUtil } from './util';
 
 export interface TheiaAppFactory<T extends TheiaApp> {
     new(page: Page, initialWorkspace?: TheiaWorkspace, mainPageObjects?: TheiaAppMainPageObjects): T;
@@ -108,7 +111,16 @@ export class ElectronLaunchOptions {
     ) { }
 
     playwrightOptions(workspace?: TheiaWorkspace): object {
-        const executablePath = this.electronAppPath + '/node_modules/.bin/electron';
+        let executablePath = path.normalize(path.join(this.electronAppPath, 'node_modules/.bin/electron'));
+        if (OSUtil.isWindows) {
+            executablePath += '.cmd';
+        }
+        if (!fs.existsSync(executablePath)) {
+            const errorMsg = `executablePath: ${executablePath} does not exist`;
+            console.log(errorMsg);
+            throw new Error(errorMsg);
+        }
+
         const args: string[] = [];
         args.push(this.electronAppPath);
         args.push(...this.additionalArgs);
@@ -119,6 +131,7 @@ export class ElectronLaunchOptions {
         if (workspace) {
             args.push(workspace.path);
         }
+        console.log(`Launching Electron from ${executablePath} with args: ${args.join(' ')}`);
         return { executablePath, args };
     }
 }

packages/filesystem/src/electron-browser/file-dialog/electron-file-dialog-module.ts

@@ -15,10 +15,39 @@
 // *****************************************************************************
 
 import { ContainerModule } from '@theia/core/shared/inversify';
-import { FileDialogService } from '../../browser/file-dialog/file-dialog-service';
+import { DefaultFileDialogService, FileDialogService } from '../../browser/file-dialog/file-dialog-service';
 import { ElectronFileDialogService } from './electron-file-dialog-service';
+import * as electronRemote from '@theia/core/electron-shared/@electron/remote';
+import { LocationListRenderer, LocationListRendererFactory, LocationListRendererOptions } from '../../browser/location';
+import { FileDialogHiddenFilesToggleRenderer, HiddenFilesToggleRendererFactory } from '../../browser/file-dialog/file-dialog-hidden-files-renderer';
+import { FileDialogTree } from '../../browser/file-dialog/file-dialog-tree';
+import { FileDialogTreeFiltersRenderer, FileDialogTreeFiltersRendererFactory, FileDialogTreeFiltersRendererOptions } from '../../browser/file-dialog';
 
 export default new ContainerModule(bind => {
-    bind(ElectronFileDialogService).toSelf().inSingletonScope();
-    bind(FileDialogService).toService(ElectronFileDialogService);
+    const useNativeFileDialogs: boolean = !electronRemote.app.commandLine.hasSwitch('no-native-window-frame');
+    if (useNativeFileDialogs) {
+        bind(ElectronFileDialogService).toSelf().inSingletonScope();
+        bind(FileDialogService).toService(ElectronFileDialogService);
+    } else {
+        bind(DefaultFileDialogService).toSelf().inSingletonScope();
+        bind(FileDialogService).toService(DefaultFileDialogService);
+        bind(LocationListRendererFactory).toFactory(context => (options: LocationListRendererOptions) => {
+            const childContainer = context.container.createChild();
+            childContainer.bind(LocationListRendererOptions).toConstantValue(options);
+            childContainer.bind(LocationListRenderer).toSelf().inSingletonScope();
+            return childContainer.get(LocationListRenderer);
+        });
+        bind(FileDialogTreeFiltersRendererFactory).toFactory(context => (options: FileDialogTreeFiltersRendererOptions) => {
+            const childContainer = context.container.createChild();
+            childContainer.bind(FileDialogTreeFiltersRendererOptions).toConstantValue(options);
+            childContainer.bind(FileDialogTreeFiltersRenderer).toSelf().inSingletonScope();
+            return childContainer.get(FileDialogTreeFiltersRenderer);
+        });
+        bind(HiddenFilesToggleRendererFactory).toFactory(({ container }) => (fileDialogTree: FileDialogTree) => {
+            const child = container.createChild();
+            child.bind(FileDialogTree).toConstantValue(fileDialogTree);
+            child.bind(FileDialogHiddenFilesToggleRenderer).toSelf().inSingletonScope();
+            return child.get(FileDialogHiddenFilesToggleRenderer);
+        });
+    }
 });