Skip to content

Commit

Permalink
Merge pull request #11456 from keymanapp/chore/web/e2e-test-migration
Browse files Browse the repository at this point in the history
chore(web): conversion of Web's end-to-end style auto-tests for @web/test-runner use 🏃
  • Loading branch information
jahorton authored May 23, 2024
2 parents feb819e + 2237f8a commit 25e44da
Show file tree
Hide file tree
Showing 20 changed files with 537 additions and 650 deletions.
3 changes: 3 additions & 0 deletions web/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,9 @@
"import": "./build/engine/osk/obj/test-index.js"
}
},
"imports": {
"#recorder": "./build/tools/testing/recorder/obj/index.js"
},
"repository": {
"type": "git",
"url": "git+https://github.com/keymanapp/keyman.git"
Expand Down
2 changes: 1 addition & 1 deletion web/src/app/browser/src/contextManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -227,7 +227,7 @@ export default class ContextManager extends ContextManagerBase<BrowserConfigurat
}
}

public setActiveTarget(target: OutputTarget<any>, sendEvents: boolean) {
public setActiveTarget(target: OutputTarget<any>, sendEvents?: boolean) {
const previousTarget = this.mostRecentTarget;
const originalTarget = this.activeTarget; // may differ, depending on focus state.

Expand Down
2 changes: 1 addition & 1 deletion web/src/app/browser/src/keymanEngine.ts
Original file line number Diff line number Diff line change
Expand Up @@ -546,7 +546,7 @@ export default class KeymanEngine extends KeymanEngineBase<BrowserConfiguration,
* @param {Object|string} e element id or element
* @param {boolean=} setFocus optionally set focus (KMEW-123)
**/
setActiveElement(e: string|HTMLElement, setFocus: boolean) {
setActiveElement(e: string|HTMLElement, setFocus?: boolean) {
if(typeof e == 'string') {
const id = e;
e = document.getElementById(e);
Expand Down
3 changes: 2 additions & 1 deletion web/src/app/browser/src/test-index.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
export { BrowserConfiguration } from './configuration.js';
export { BrowserConfiguration, BrowserInitOptionSpec } from './configuration.js';
export { default as ContextManager, KeyboardCookie } from "./contextManager.js";
export { preprocessKeyboardEvent, default as HardwareEventKeyboard } from './hardwareEventKeyboard.js';

export { default as KeymanEngine } from './keymanEngine.js';
export { default as KeyboardInterface } from './keyboardInterface.js';
export { UIModule } from './uiModuleInterface.js';
4 changes: 0 additions & 4 deletions web/src/test/auto/integrated/CI.conf.cjs

This file was deleted.

126 changes: 0 additions & 126 deletions web/src/test/auto/integrated/base.conf.cjs

This file was deleted.

158 changes: 0 additions & 158 deletions web/src/test/auto/integrated/cases/basics.js

This file was deleted.

Loading

0 comments on commit 25e44da

Please sign in to comment.