Skip to content

Unit tests for OAuth2Manager API - #4975

Merged
Ayush Jaiswal (ayushjai30) merged 5 commits into
mainfrom
user/ayushjaiswal/OAuthAPIUTs
Dec 20, 2024
Merged

Unit tests for OAuth2Manager API#4975
Ayush Jaiswal (ayushjai30) merged 5 commits into
mainfrom
user/ayushjaiswal/OAuthAPIUTs

Conversation

@ayushjai30

@ayushjai30 Ayush Jaiswal (ayushjai30) commented Dec 16, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

Unit tests for OAuth2Manager APIs

@ayushjai30

Ayush Jaiswal (ayushjai30) commented Dec 17, 2024

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

azure-pipelines Bot commented Dec 17, 2024

Copy link
Copy Markdown
Pull request contains merge conflicts.

@ayushjai30 Ayush Jaiswal (ayushjai30) changed the title Unit tests for OAuth2.0 API Unit tests for OAuth2Manager API Dec 17, 2024
@ayushjai30
Ayush Jaiswal (ayushjai30) marked this pull request as ready for review December 17, 2024 09:27
@ayushjai30

Ayush Jaiswal (ayushjai30) commented Dec 17, 2024

Copy link
Copy Markdown
Contributor Author

Ayush Jaiswal (@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

Ayush Jaiswal (@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
Godly T.Alias (godlytalias) requested a review from a team December 17, 2024 10:05
akanpatel2206
Akanksha Patel (akanpatel2206) reviewed Dec 17, 2024
View reviewed changes
Comment thread test/OAuth2ManagerTests/OAuth2APITests.cpp
@ayushjai30

Ayush Jaiswal (ayushjai30) commented Dec 18, 2024

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

azure-pipelines Bot commented Dec 18, 2024

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Hemantxk
Hemant Kumar (Hemantxk) reviewed Dec 18, 2024
View reviewed changes
Comment thread test/OAuth2ManagerTests/OAuth2APITests.cpp
Comment thread WindowsAppRuntime.sln Outdated
@ayushjai30

Ayush Jaiswal (ayushjai30) commented Dec 18, 2024

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

azure-pipelines Bot commented Dec 18, 2024

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Hemantxk
Hemant Kumar (Hemantxk) reviewed Dec 18, 2024
View reviewed changes
Comment thread test/TestApps/OAuthTestApp/PropertySheet.props Outdated
Hemantxk
Hemant Kumar (Hemantxk) reviewed Dec 18, 2024
View reviewed changes
Comment thread test/TestApps/OAuthTestApp/main.cpp Outdated
Hemantxk
Hemant Kumar (Hemantxk) reviewed Dec 18, 2024
View reviewed changes
Comment thread test/TestApps/OAuthTestApp/readme.txt Outdated
Hemantxk
Hemant Kumar (Hemantxk) reviewed Dec 18, 2024
View reviewed changes
Comment thread test/TestApps/OAuthTestAppPackage/Package.appxmanifest Outdated
godlytalias
Godly T.Alias (godlytalias) reviewed Dec 18, 2024
View reviewed changes
Comment thread test/TestApps/OAuthTestAppPackage/Package.appxmanifest Outdated
AjitSurana
Ajit Surana (AjitSurana) reviewed Dec 19, 2024
View reviewed changes
Comment thread test/OAuth2ManagerTests/OAuth2APITests.h
@ayushjai30

Ayush Jaiswal (ayushjai30) commented Dec 19, 2024

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

azure-pipelines Bot commented Dec 19, 2024

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30

Ayush Jaiswal (ayushjai30) commented Dec 19, 2024

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

azure-pipelines Bot commented Dec 19, 2024

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30

Ayush Jaiswal (ayushjai30) commented Dec 20, 2024

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

azure-pipelines Bot commented Dec 20, 2024

Copy link
Copy Markdown
Pull request contains merge conflicts.

@ayushjai30

Ayush Jaiswal (ayushjai30) commented Dec 20, 2024

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

azure-pipelines Bot commented Dec 20, 2024

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@ayushjai30
Ayush Jaiswal (ayushjai30) deleted the user/ayushjaiswal/OAuthAPIUTs branch December 20, 2024 13:21
Mike Crider (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
This was referenced Dec 18, 2025
Closed
Closed
Closed
This was referenced Jan 14, 2026
Closed
Merged
Closed
This was referenced Jan 26, 2026
Merged
Merged
This was referenced Feb 11, 2026
Closed
Open
@dependabot dependabot Bot mentioned this pull request Mar 19, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AjitSurana Ajit Surana (AjitSurana) AjitSurana left review comments
@godlytalias Godly T.Alias (godlytalias) godlytalias left review comments
@Saharsh979 Saharsh Ananta Jaiswal (Saharsh979) Saharsh979 left review comments
@Hemantxk Hemant Kumar (Hemantxk) Hemantxk approved these changes
@hiteshkrmsft Hitesh Kumar (hiteshkrmsft) hiteshkrmsft approved these changes
@akanpatel2206 Akanksha Patel (akanpatel2206) akanpatel2206 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