Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@akomisarek
Copy link
Contributor

JIRA link (if applicable)

https://tools.hmcts.net/jira/browse/BPS-1256

Change description

Adds new /o/userinfo pac

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[X] No

Adam Komisarek added 3 commits September 15, 2020 14:07
* the fix removes broken PACT, which doesn't have appropriate state in IDAM
* the PACT consumer is bumped so charset=UTF-8 is optional in the application/json portion
params.put("username", "[email protected]");
params.put("password", "Password12");

return builder.given("I have obtained an access_token as a user",params)

Choose a reason for hiding this comment

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

formatting (space after ',')

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@Test
@PactTestFor(pactMethod = "executeGetUserInfo")
void verifyUserInfo() {
UserInfo actualUserInfo = idamClient.getUserInfo("Bearer eyJ0eXAiOiJKV1QiLCJraWQiOiJiL082T3ZWdeRre");

Choose a reason for hiding this comment

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

perhaps this token/header could be a constant

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Base automatically changed from dev/fix-idam-content-type-verification-failure to master September 15, 2020 15:31
@akomisarek akomisarek merged commit 4d51397 into master Sep 15, 2020
@akomisarek akomisarek deleted the dev/add-new-user-info-pact branch September 15, 2020 15:52
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