-
Notifications
You must be signed in to change notification settings - Fork 827
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
integrationTest: Add IT for user_token grant variants #2194
Conversation
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/184429062 The labels on this github issue will be updated when the story is started. |
Can we get a description on what this is addressing? |
created it to better understand what #2193 changes, so if we proceed with it, this test should be merged |
uaa/src/test/java/org/cloudfoundry/identity/uaa/integration/feature/UserTokenGrantIT.java
Show resolved
Hide resolved
uaa/src/test/java/org/cloudfoundry/identity/uaa/integration/feature/UserTokenGrantIT.java
Show resolved
Hide resolved
uaa/src/test/java/org/cloudfoundry/identity/uaa/integration/feature/UserTokenGrantIT.java
Outdated
Show resolved
Hide resolved
The grant is similar to refresh but always without a secret. The 4 examples are for the discussion about issue #2193
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few more minor comments. I'll go ahead and approve so you can take a look and then move ahead.
uaa/src/test/java/org/cloudfoundry/identity/uaa/integration/feature/UserTokenGrantIT.java
Outdated
Show resolved
Hide resolved
uaa/src/test/java/org/cloudfoundry/identity/uaa/integration/feature/UserTokenGrantIT.java
Outdated
Show resolved
Hide resolved
* Add IT for user_token grant * add more examples for user_token grant The grant is similar to refresh but always without a secret. The 4 examples are for the discussion about issue #2193 * refactor test cases * test renamed * review Co-authored-by: Bruce Ricard <bricard@vmware.com>
User Token Grant Type is proprietary grant type. Until now we dont have pure integration tests. therefore add with this PR one.
https://docs.cloudfoundry.org/api/uaa/version/76.18.0/index.html#user-token-grant
In PR
#2193
I noticed that no integration test has to be change but the PR is a restriction of current behavior
Because of the other PR I named the tests according to client types, e.g. public and/or confidential. A public client is a client without any secret or with an empty secret (like cf client). A confidential client is a client with a secure secret.