Skip to content

Enable tests for new aad implementation.#17823

Merged
chenrujun merged 1 commit intoAzure:masterfrom
chenrujun:Enable-tests-for-new-aad-implementation
Nov 26, 2020
Merged

Enable tests for new aad implementation.#17823
chenrujun merged 1 commit intoAzure:masterfrom
chenrujun:Enable-tests-for-new-aad-implementation

Conversation

@chenrujun
Copy link

Enable tests for new aad implementation.

@ghost ghost added the azure-spring All azure-spring related issues label Nov 26, 2020
@chenrujun
Copy link
Author

/azp run java - spring - ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@saragluna saragluna added the azure-spring-aad Spring active directory related issues. label Nov 26, 2020
Copy link
Member

@yiliuTo yiliuTo left a comment

Choose a reason for hiding this comment

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

LGTM

@chenrujun chenrujun merged commit 3986906 into Azure:master Nov 26, 2020
@chenrujun chenrujun deleted the Enable-tests-for-new-aad-implementation branch November 26, 2020 06:48
import org.springframework.security.oauth2.client.OAuth2AuthorizedClient;
import org.springframework.security.oauth2.client.registration.ClientRegistration;
import org.springframework.security.oauth2.client.web.OAuth2AuthorizedClientRepository;
import org.springframework.security.oauth2.core.AbstractOAuth2Token;
Copy link
Member

Choose a reason for hiding this comment

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

Is this test class more like a unit test instead of an integration test?

Copy link
Author

Choose a reason for hiding this comment

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

I created a PR: #17857

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

azure-spring All azure-spring related issues azure-spring-aad Spring active directory related issues.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants