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

[Popover]: Skipped unstable test #6472

Closed
alisonailea opened this issue Feb 14, 2023 · 2 comments
Closed

[Popover]: Skipped unstable test #6472

alisonailea opened this issue Feb 14, 2023 · 2 comments
Assignees
Labels
4 - verified Issues that have been released and confirmed resolved. p - high Issue should be addressed in the current milestone, impacts component or core functionality testing Issues related to automated or manual testing.

Comments

@alisonailea
Copy link
Contributor

Test type

unstable test

Which Component(s)

Popover

Unstable Tests

calcite-popover › setFocus › should focus close button

Test error, if applicable

Summary of all failing tests
FAIL src/components/popover/popover.e2e.ts (44.325 s)
● calcite-popover › setFocus › should focus close button

expect(received).toBe(expected) // Object.is equality

Expected: true
Received: false

  224 |         options?.shadowFocusTargetSelector
  225 |       )
> 226 |     ).toBe(true);
      |       ^
  227 |   }
  228 |
  229 |   // wait for next frame before checking focus

PR skipped, if applicable

#6465

Additional Info

No response

@alisonailea alisonailea added testing Issues related to automated or manual testing. p - high Issue should be addressed in the current milestone, impacts component or core functionality 0 - new New issues that need assignment. airtable and removed airtable labels Feb 14, 2023
jcfranco added a commit that referenced this issue Aug 23, 2024
**Related Issue:** #6472

## Summary

✨🧪✨
@jcfranco
Copy link
Member

This was unskipped a while ago and seems to have been stable since. Closing as self-verified!

@jcfranco jcfranco added 3 - installed Issues that have been merged to master branch and are ready for final confirmation. and removed 0 - new New issues that need assignment. labels Dec 14, 2024
@jcfranco jcfranco self-assigned this Dec 14, 2024
Copy link
Contributor

Installed and assigned for verification.

@jcfranco jcfranco assigned jcfranco and unassigned geospatialem and DitwanP Dec 14, 2024
@jcfranco jcfranco added 4 - verified Issues that have been released and confirmed resolved. and removed 3 - installed Issues that have been merged to master branch and are ready for final confirmation. labels Dec 14, 2024
@jcfranco jcfranco added this to the 2024-08-27 - Aug Release milestone Dec 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4 - verified Issues that have been released and confirmed resolved. p - high Issue should be addressed in the current milestone, impacts component or core functionality testing Issues related to automated or manual testing.
Projects
None yet
Development

No branches or pull requests

4 participants