Paul Purifoy

Address argument encoding nits

16

This PR addresses the nits listed in from the previously closed PR: https://github.com/microsoft/WindowsAppSDK/pull/2805. Add sample to display format: https://github.com/microsoft/WindowsAppSDK/pull/2805#discussion_r938203444 Can we return empty string in value for arguments? https://github.com/microsoft/WindowsAppSDK/pull/2805#discussion_r938209869 Tokenizing:...

### Describe the bug Testing argument decoding has to be done manually to verify behavior. Filing an issue to fill this gap in the future. ### Steps to reproduce the...

### Describe the bug Argument does not match the old API it was based on: https://docs.microsoft.com/en-us/uwp/api/Windows.UI.Notifications.ToastActivatedEventArgs?view=winrt-22621. It should be renamed to Arguments instead. ### Steps to reproduce the bug Read...

### Describe the bug Adding a Clone() function would allow developers to post multiple AppNotifications with slight changes in the XML easier. ### Steps to reproduce the bug A feature...

Refactoring app notification tests

2

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...

### Describe the bug Consider adding an async version of some of the AppNotificationManager APIs like Show. It would be valuable to for developer to know when an AppNotification was...

This PR refactors the activation tests for a more maintainable unit test in NotificationActivation scenarios.

GetActivatedEventArgs throws when activated by App/PushNotifications without calling Register

2

### Describe the bug An app activated by App/PushNotifications but does not call App/PushNotificationManager.Default.Register, will throw on GetActivatedEventArgs. Instead, the app shouldn't throw and should return null args. ### Steps...

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

### Describe the bug If a developer tries to register a foreground handler using PushReceived after they called Register, it returns ERROR_NOT_FOUND. This error code is not very telling of...