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

Wait for request when testing session revocation #24048

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

jonkoops
Copy link
Contributor

Closes #24022

@jonkoops jonkoops requested a review from a team as a code owner October 17, 2023 09:11
@ghost ghost added the team/ui label Oct 17, 2023
@cypress
Copy link

cypress bot commented Oct 17, 2023

2 flaky tests on run #9431 ↗︎

0 527 48 0 Flakiness 2

Details:

Merge 13b2ac4 into 631bf09...
Project: Keycloak Admin UI Commit: bfc4fd08cb ℹ️
Status: Passed Duration: 11:30 💡
Started: Oct 17, 2023 9:20 AM Ended: Oct 17, 2023 9:31 AM
Flakiness  clients_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Clients test > Accessibility tests for clients > Check a11y violations on load/ clients list tab Test Replay Output Screenshots
Flakiness  realm_settings_general_tab_test.spec.ts • 1 flaky test • chrome

View Output Video

Test Artifacts
Realm settings general tab tests > Test all general tab switches Test Replay Output Screenshots

Review all test suite changes for PR #24048 ↗︎

@jonkoops jonkoops enabled auto-merge (squash) October 17, 2023 09:33
Copy link
Contributor

@edewit edewit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jonkoops jonkoops merged commit 0844b57 into keycloak:main Oct 17, 2023
52 checks passed
@jonkoops jonkoops deleted the fix-sessions-test branch October 17, 2023 09:44
@jonkoops
Copy link
Contributor Author

Like the refactor, don't really like the use of https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Classes/Private_class_fields

I prefer to use features of JavaScript where possible, the private keyword is mostly there because TypeScript implemented it before it became a JavaScript feature. I think the dev lead of the TypeScript project puts it well.

jonkoops added a commit to jonkoops/keycloak that referenced this pull request Nov 28, 2023
Signed-off-by: Jon Koops <jonkoops@gmail.com>
jonkoops added a commit that referenced this pull request Nov 28, 2023
Signed-off-by: Jon Koops <jonkoops@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sessions revocation check if notBefore saved test fails
2 participants