Skip to content

Update integration tests to 8f2fba6cfc5f582e776615359113d9ff37218b3d - #1717

Merged
Difegue merged 1 commit into
Difegue:devfrom
psilabs-dev:integration-test-update/2026-09-07
Sep 10, 2026
Merged

Difegue merged 1 commit into
Difegue:devfrom
psilabs-dev:integration-test-update/2026-09-07

Conversation

@psilabs-dev

Copy link
Copy Markdown
Contributor

Primarily fix some playwright test methodologies in preparation for upcoming UI changes. Does not introduce new tests, though hardens almost all existing ones in the following ways:

  1. error toasts are banned
  2. spinners are banned
  3. any playwright test that's testing some kind of scope no longer runs API calls through that scope, it will run user actions instead (i.e. try to more faithfully conduct a true UI test)

Additionally, previous integration test cases kept adding Playwright boilerplate code, so I've moved all of that under a context manager, which incidentally also closed off browser/page closure leaks, and forgotten OK assertions.

psilabs-dev added a commit to psilabs-dev/LANraragi that referenced this pull request Sep 8, 2026
psilabs-dev added a commit to psilabs-dev/LANraragi that referenced this pull request Sep 8, 2026
…f37218b3d Difegue#1717)

# Conflicts:
#	.github/workflows/push-continuous-integration.yml
@psilabs-dev
psilabs-dev marked this pull request as ready for review September 10, 2026 07:35
@Difegue
Difegue merged commit a60f6df into Difegue:dev Sep 10, 2026
4 checks passed
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