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

Refactoring app notification tests #2444

Merged
merged 27 commits into from
Aug 22, 2022

Conversation

pmpurifoy
Copy link
Contributor

@pmpurifoy pmpurifoy commented Apr 27, 2022

This PR refactors the AppNotification tests into TAEF. Similar to the PushNotificationTests refactor PR, all the unit tests that use to exist in the ToastNotificationTestApp are moving to TAEF. Now error codes will be seen in the TAEF process.

This runs each test scenario under 4 diffrent envs:

  • packaged
  • unpackaged
  • packaged + SelfContained
  • unpackaged + SelfContained

@pmpurifoy pmpurifoy marked this pull request as ready for review April 27, 2022 19:25
@pmpurifoy
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@loneursid loneursid added the area-Notifications Toast notification, badges, Live Tiles, push notifications label Apr 28, 2022
@pmpurifoy pmpurifoy changed the title User/purifoypaul/refactoring app notification tests Refactoring app notification tests Apr 28, 2022
@pmpurifoy
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@pmpurifoy
Copy link
Contributor Author

/azp run TransportPackage-Foundation-PR

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pmpurifoy
Copy link
Contributor Author

/azp run TransportPackage-Foundation-PR

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pmpurifoy
Copy link
Contributor Author

/azp run TransportPackage-Foundation-PR

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pmpurifoy
Copy link
Contributor Author

/azp run TransportPackage-Foundation-PR

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pmpurifoy
Copy link
Contributor Author

/azp run TransportPackage-Foundation-PR

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pmpurifoy
Copy link
Contributor Author

/azp run TransportPackage-Foundation-PR

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pmpurifoy
Copy link
Contributor Author

/azp run TransportPackage-Foundation-PR

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pmpurifoy
Copy link
Contributor Author

/azp run TransportPackage-Foundation-PR

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@pmpurifoy pmpurifoy enabled auto-merge (squash) August 22, 2022 16:20
@pmpurifoy pmpurifoy merged commit b36ff7e into develop Aug 22, 2022
@pmpurifoy pmpurifoy deleted the user/purifoypaul/RefactoringAppNotificationTests branch August 22, 2022 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Notifications Toast notification, badges, Live Tiles, push notifications needs-triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants