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

Update nestedAppAuth.json to include E2E tests #2652

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

baljesingh
Copy link
Contributor

Description

Add E2E test for the nested app auth flow i.e. send NAA payload to the NAA bridge from the parent iframe.

Main changes in the PR:

  1. Update nestedAppAuth.json to include E2E tests

Validation

Validation performed:

  1. Locally test it with web host SDK.

Unit Tests added:

Unit tests are required for all changes. If no unit tests were added as part of this change, please explain why they aren't necessary.

Not applicable

End-to-end tests added:

Yes

@baljesingh baljesingh requested a review from a team as a code owner December 6, 2024 14:02
@baljesingh baljesingh force-pushed the baljesingh/AddE2ETestForWeb branch from 5c9076d to 5f7ef26 Compare January 14, 2025 21:24
@baljesingh baljesingh force-pushed the baljesingh/AddE2ETestForWeb branch from 5f7ef26 to 7d6954c Compare February 6, 2025 18:44
Copy link
Contributor

github-actions bot commented Feb 6, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
packages/teams-js/dist/esm/packages/teams-js/src/index.js 182.11 KB (0%) 3.7 s (0%) 995 ms (+359.83% 🔺) 4.7 s

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

Successfully merging this pull request may close these issues.

1 participant