Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wip #658

Open
wants to merge 37 commits into
base: main
Choose a base branch
from
Open

wip #658

wants to merge 37 commits into from

Conversation

TateB
Copy link
Collaborator

@TateB TateB commented Mar 4, 2024

No description provided.

@TateB TateB requested a review from sugh01 as a code owner March 4, 2024 02:49
Copy link

cloudflare-pages bot commented Mar 4, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: a359a47
Status: ✅  Deploy successful!
Preview URL: https://82788846.ens-app-v3.pages.dev
Branch Preview URL: https://tate-playwright-optimise.ens-app-v3.pages.dev

View logs

Copy link

sonarcloud bot commented Mar 5, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

Copy link

github-actions bot commented Mar 5, 2024

Playwright test results

failed  6 failed
passed  181 passed
flaky  6 flaky
skipped  6 skipped

Details

stats  199 tests across 58 suites
duration  6 minutes, 13 seconds
commit  a359a47

Failed tests

stateless-serial → stateless/serial/deleteSubname.spec.ts → should not allow parent owner to delete if PCC is expired
stateless-serial → stateless/serial/extendNames.spec.ts → should be able to extend a single unwrapped name in grace period from profile
stateless-serial → stateless/serial/extendNames.spec.ts → should be able to extend a single unwrapped name in grace period from profile 2
stateful → stateful/profile.spec.ts → Profile → should allow user to connect
stateless-serial → stateless/serial/registerName.spec.ts → normal registration → should allow normal registration
stateless-serial → stateless/serial/setPrimary.spec.ts → profile → should allow setting primary name from name with encrypted label

Flaky tests

stateless-parallel → stateless/parallel/ownership.2LD.spec.ts → Unwrapped 2LD - Owner Only → Edit Roles
stateless-parallel → stateless/parallel/ownership.3LD.spec.ts → Wrapped 3LD, Unwrapped 2LD - Manager only → Expiry Section, Extend & Set Reminder
stateful → stateful/profile.spec.ts → Profile → should show a warning if name is not supported
stateful → stateful/profile.spec.ts → Profile → should load emoji domain pages
stateless-serial → stateless/serial/profileEditor.spec.ts → migrations → should force a name on the old registry to go to update registry
stateless-parallel → stateless/parallel/permissions.spec.ts → Permissions → should show parent not locked warning

Skipped tests

stateful → stateful/dnsclaim.spec.ts → Import DNSSEC name → should not allow the user to proceed if they have not set the correct TXT record - offchain
stateless-serial → stateless/serial/registerName.spec.ts → normal registration → should not direct to the registration page on search, and show all records from registration
stateless-serial → stateless/serial/registerName.spec.ts → normal registration → should allow registering a non-primary name
stateless-parallel → stateless/parallel/settings.spec.ts → Transactions → should add a failed transaction to the transaction list, and show the corresponding notification
stateless-parallel → stateless/parallel/settings.spec.ts → Transactions → should add a pending transaction to the transaction list, and show the corresponding notification once confirmed
stateless-parallel → stateless/parallel/settings.spec.ts → Transactions → should clear transactions when clear is pressed

Base automatically changed from viem-test to main March 14, 2024 20:53
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.

None yet

1 participant