Skip to content

Commit 42286fb

Browse files
Fix signature helper editor tests (#8006)
Fix signature test; add makefile helper arg
1 parent 2ff52fb commit 42286fb

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

Makefile

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,8 @@ run-desktop: install build ## Start the desktop app
113113
###############################################################################
114114
# TEST
115115

116+
PW_ARGS ?=
117+
116118
E2E_GREP ?=
117119
E2E_WORKERS ?=
118120
E2E_FAILURES ?= 1
@@ -138,17 +140,17 @@ test-e2e: test-e2e-$(TARGET)
138140
.PHONY: test-e2e-web
139141
test-e2e-web: install build ## Run the web e2e tests
140142
ifdef E2E_GREP
141-
npm run test:e2e:web -- --headed --grep="$(E2E_GREP)" --max-failures=$(E2E_FAILURES)
143+
npm run test:e2e:web -- --headed --grep="$(E2E_GREP)" --max-failures=$(E2E_FAILURES) $(PW_ARGS)
142144
else
143-
npm run test:e2e:web -- --headed --workers='100%'
145+
npm run test:e2e:web -- --headed --workers='100%' $(PW_ARGS)
144146
endif
145147

146148
.PHONY: test-e2e-desktop
147149
test-e2e-desktop: install build ## Run the desktop e2e tests
148150
ifdef E2E_GREP
149-
npm run test:e2e:desktop -- --grep="$(E2E_GREP)" --max-failures=$(E2E_FAILURES)
151+
npm run test:e2e:desktop -- --grep="$(E2E_GREP)" --max-failures=$(E2E_FAILURES) $(PW_ARGS)
150152
else
151-
npm run test:e2e:desktop -- --workers='100%'
153+
npm run test:e2e:desktop -- --workers='100%' $(PW_ARGS)
152154
endif
153155

154156
.PHONY: test-snapshots

e2e/playwright/editor-tests.spec.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -625,6 +625,11 @@ a1 = startSketchOn(offsetPlane(XY, offset = 10))
625625

626626
await scene.connectionEstablished()
627627

628+
// Wait for highlighting to kick in, a good proxy that the LSP is ready.
629+
await expect
630+
.poll(() => page.evaluate(() => document.querySelector('.ͼu') !== null))
631+
.toBe(true)
632+
628633
// Expect the signature help to NOT be visible
629634
await expect(page.locator('.cm-signature-tooltip')).not.toBeVisible()
630635

0 commit comments

Comments
 (0)