Skip to content

JCL-337: Use verify endpoint in tests#527

Merged
NSeydoux merged 5 commits intomainfrom
JCL-337_use-verify-endpoint
Jun 15, 2023
Merged

JCL-337: Use verify endpoint in tests#527
NSeydoux merged 5 commits intomainfrom
JCL-337_use-verify-endpoint

Conversation

@NSeydoux
Copy link
Copy Markdown
Contributor

Call verify to check the validity of the issued grant, and the invalidity of the revoked grant.

@NSeydoux NSeydoux force-pushed the JCL-337_use-verify-endpoint branch from 898bfb9 to 56a34be Compare June 14, 2023 12:44
@NSeydoux NSeydoux marked this pull request as ready for review June 14, 2023 19:50
@NSeydoux NSeydoux requested a review from a team as a code owner June 14, 2023 19:50
Copy link
Copy Markdown
Collaborator

@acoburn acoburn left a comment

Choose a reason for hiding this comment

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

LGTM, one minor comment that isn't really required to fix now (more of an FYI)

NSeydoux added 5 commits June 15, 2023 11:39
Call verify to check the validity of the issued grant, and the invalidity of the revoked grant.
The mock is stateful: an access grant is considered valid until a revocation request has been made, and it is then considered invalid.
This test is out of scope for this PR
Enforce support of Java 8 runtime.
@NSeydoux NSeydoux force-pushed the JCL-337_use-verify-endpoint branch from 8260323 to b051b63 Compare June 15, 2023 09:40
@NSeydoux NSeydoux enabled auto-merge (squash) June 15, 2023 09:41
@NSeydoux NSeydoux merged commit c7c87f2 into main Jun 15, 2023
@NSeydoux NSeydoux deleted the JCL-337_use-verify-endpoint branch June 15, 2023 09:49
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