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.

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

BadgeNotification: Onboarding Unit tests to the pipeline test runs #5140

Merged
merged 2 commits into from
Feb 17, 2025

Conversation

SatwikKrSharma
Copy link
Contributor

@SatwikKrSharma SatwikKrSharma commented Feb 12, 2025

The changes involve onboarding unit tests for the BadgeNotification API to run as part of the pipeline test runs (Checkin, Nightly, and Release).

Testing - In the build pipeline all tests are passing

A microsoft employee must use /azp run to validate using the pipelines below.

WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.

For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

@SatwikKrSharma
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@SatwikKrSharma
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@SatwikKrSharma SatwikKrSharma merged commit 0f71ff3 into main Feb 17, 2025
26 checks passed
@SatwikKrSharma SatwikKrSharma deleted the user/satwiksharma/Onboarding_BadgeUT_To_Pipeline branch February 17, 2025 10:53
@@ -21,7 +21,7 @@
<PublisherDisplayName>Windows APP SDK</PublisherDisplayName>
<Logo>taef.png</Logo>
</Properties>

Copy link
Contributor

Choose a reason for hiding this comment

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

nit : space

@@ -40,7 +40,7 @@
<None Include="packages.config" />
</ItemGroup>
<ItemGroup>
<ResourceCompile Include="AppNotificationTests.rc">
<ResourceCompile Include="BadgeNotificationTests.rc">
Copy link
Contributor

Choose a reason for hiding this comment

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

why is AppNotificationTests , removed

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

Successfully merging this pull request may close these issues.

3 participants