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.

Sign up for GitHub

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

Jump to bottom

Unit tests for OAuth2Manager API #4975

Merged
merged 5 commits into from
Dec 20, 2024
Merged

Unit tests for OAuth2Manager API #4975

merged 5 commits into from
Dec 20, 2024

Conversation

ayushjai30
Copy link
Contributor

@ayushjai30 ayushjai30 commented Dec 16, 2024 •
edited
Loading

Unit tests for OAuth2Manager APIs

@ayushjai30 ayushjai30 force-pushed the user/ayushjaiswal/OAuthAPIUTs branch from 9e8ecfd to e3f2318 Compare December 17, 2024 07:29
@ayushjai30 ayushjai30 force-pushed the user/ayushjaiswal/OAuthAPIUTs branch from e3f2318 to ebf1f2f Compare December 17, 2024 08:40
@ayushjai30
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Pull request contains merge conflicts.

@ayushjai30 ayushjai30 changed the title Unit tests for OAuth2.0 API Unit tests for OAuth2Manager API Dec 17, 2024
@ayushjai30 ayushjai30 marked this pull request as ready for review December 17, 2024 09:27
@ayushjai30 ayushjai30 requested review from akanpatel2206, godlytalias and Saharsh979 and removed request for Saharsh979 December 17, 2024 09:27
@ayushjai30
Copy link
Contributor Author

@ayushjai30 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@ayushjai30 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Microsoft"

@godlytalias godlytalias requested a review from a team December 17, 2024 10:05
@ayushjai30 ayushjai30 force-pushed the user/ayushjaiswal/OAuthAPIUTs branch from ccda8d7 to 6240298 Compare December 17, 2024 11:27
@ayushjai30
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30 ayushjai30 force-pushed the user/ayushjaiswal/OAuthAPIUTs branch from 6240298 to 4b3a746 Compare December 18, 2024 09:36
Hemantxk
Hemantxk reviewed Dec 18, 2024
View reviewed changes
test/OAuth2ManagerTests/OAuth2APITests.cpp Show resolved Hide resolved
WindowsAppRuntime.sln Outdated Show resolved Hide resolved
@ayushjai30
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30 ayushjai30 force-pushed the user/ayushjaiswal/OAuthAPIUTs branch from b015959 to e83a216 Compare December 18, 2024 14:04
@ayushjai30
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30 ayushjai30 force-pushed the user/ayushjaiswal/OAuthAPIUTs branch from 9b8e002 to 22997c9 Compare December 20, 2024 09:32
@ayushjai30
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Pull request contains merge conflicts.

@ayushjai30
Copy link
Contributor Author

/azp run

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30 ayushjai30 merged commit f61910f into main Dec 20, 2024
26 checks passed
@ayushjai30 ayushjai30 deleted the user/ayushjaiswal/OAuthAPIUTs branch December 20, 2024 13:21
codendone pushed a commit that referenced this pull request Jan 10, 2025
@ayushjai30
Unit tests for OAuth2Manager API (#4975)
a6aa974 
* Unit tests for OAuth2.0 API

* incorporated comments

* fix build error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Saharsh979 Saharsh979 Saharsh979 left review comments

@godlytalias godlytalias godlytalias left review comments

@AjitSurana AjitSurana AjitSurana left review comments

@akanpatel2206 akanpatel2206 akanpatel2206 approved these changes

@hiteshkrmsft hiteshkrmsft hiteshkrmsft approved these changes

@Hemantxk Hemantxk Hemantxk approved these changes

Assignees
No one assigned
Labels
needs-triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@ayushjai30 @AjitSurana @godlytalias @Saharsh979 @Hemantxk @hiteshkrmsft @akanpatel2206