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

test: add tests on keycloak authenticator logic [do not merge] #761

Closed
wants to merge 22 commits into from

Conversation

patlo-iog
Copy link
Contributor

@patlo-iog patlo-iog commented Oct 19, 2023

Overview

Do not merge until these 2 PRs are merged and conflicts are resolved

Checklist

My PR contains...

  • No code changes (changes to documentation, CI, metadata, etc.)
  • Bug fixes (non-breaking change which fixes an issue)
  • Improvements (misc. changes to existing features)
  • Features (non-breaking change which adds functionality)

My changes...

  • are breaking changes
  • are not breaking changes
  • If yes to above: I have updated the documentation accordingly

Documentation

  • My changes do not require a change to the project documentation
  • My changes require a change to the project documentation
  • If yes to above: I have updated the documentation accordingly

Tests

  • My changes can not or do not need to be tested
  • My changes can and should be tested by unit and/or integration tests
  • If yes to above: I have added tests to cover my changes
  • If yes to above: I have taken care to cover edge cases in my tests

yshyn-iohk and others added 20 commits October 12, 2023 19:00
Signed-off-by: Pat Losoponkul <[email protected]>
Signed-off-by: Pat Losoponkul <[email protected]>
Signed-off-by: Pat Losoponkul <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Oct 19, 2023

Atala PRISM Test Results

  82 files  +  3    82 suites  +3   15m 10s ⏱️ +4s
692 tests +12  672 ✔️ +3  11 💤 ±0  9 +9 
698 runs  +12  678 ✔️ +3  11 💤 ±0  9 +9 

For more details on these failures, see this check.

Results for commit a0f6648. ± Comparison against base commit 5558a9e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 19, 2023

Integration Test Results

11 files  ±0  11 suites  ±0   4s ⏱️ -1s
25 tests ±0  25 ✔️ ±0  0 💤 ±0  0 ±0 
37 runs  ±0  37 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 7f02810. ± Comparison against base commit 3e7534f.

♻️ This comment has been updated with latest results.

@patlo-iog patlo-iog changed the title test: add tests on keycloak authenticator logic [do not merged] test: add tests on keycloak authenticator logic [do not merge] Oct 19, 2023
@patlo-iog
Copy link
Contributor Author

Closing this PR since it is already part of this PR #755

@patlo-iog patlo-iog closed this Nov 6, 2023
@patlo-iog patlo-iog deleted the test/keycloak-authz-test branch November 6, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants