Skip to content

Editor e2e PageObject#649

Open
DavidSouther wants to merge 2 commits into
mainfrom
monaco_page
Open

Editor e2e PageObject#649
DavidSouther wants to merge 2 commits into
mainfrom
monaco_page

Conversation

@DavidSouther

Copy link
Copy Markdown
Collaborator

No description provided.

@DavidSouther DavidSouther force-pushed the monaco_page branch 2 times, most recently from 8fbbbda to ec5bbf7 Compare April 26, 2026 17:19
@mmatsumoto1026

Copy link
Copy Markdown
Contributor

After executing chipPage.editor.write in the test "toggleMonaco re-enables Monaco editor for tests that need it" of editor-panel.spec.ts, the code text in the Monaco editor written by key press simulator will break as shown in below.

CIP nd
	IN, b;
	  outot;

		PARTS:
		   Nandaa = ot=);
		       Not(n=,out=ut);
		       }

The method this.page.keyboard.type(text) in Monaco may triggers several automated features like auto-completion.

With using this.page.keyboard.insertText(text), We can avoid the individual key-event logic for most editor side-effects, allowing the entire block of text to be inserted cleanly into the Monaco.

@DavidSouther DavidSouther force-pushed the main branch 6 times, most recently from 9e0a7cc to 8ce609b Compare May 16, 2026 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants