Skip to content

[Communication][Email] Separating Email test resources + Adding PPE live tests - #29190

Merged
yogeshmo merged 6 commits into
mainfrom
email/add-ppe-livetests
Mar 16, 2023
Merged

[Communication][Email] Separating Email test resources + Adding PPE live tests#29190
yogeshmo merged 6 commits into
mainfrom
email/add-ppe-livetests

Conversation

@yogeshmo

@yogeshmo yogeshmo commented Mar 7, 2023

Copy link
Copy Markdown
Contributor
  • Adding PPE SDK live tests for the email communication package.
  • Follow up to Adding email live tests resources to test-resources.json azure-sdk-for-js#24175. That PR added email resource creation to a common resources template shared by all communication tests. However, only communication-email package needs the email resource for its test pipeline. So this PR adds an email-specific resource template, while reverting changes to the common template. This also includes changes to the common file used for the live testing pipeline.

@ghost ghost added the Communication label Mar 7, 2023
@yogeshmo

yogeshmo commented Mar 7, 2023

Copy link
Copy Markdown
Contributor Author

/azp run prepare-pipelines

@azure-pipelines

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

@azure-sdk

azure-sdk commented Mar 8, 2023

Copy link
Copy Markdown
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-communication-phonenumbers

@yogeshmo yogeshmo changed the title [Communication][Email] Adding PPE live tests [Communication][Email] Separating Email test resources + Adding PPE live tests Mar 15, 2023
@yogeshmo

Copy link
Copy Markdown
Contributor Author

/azp run prepare-pipelines

@yogeshmo
yogeshmo marked this pull request as ready for review March 15, 2023 00:26
@azure-pipelines

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

@AikoBB

AikoBB commented Mar 15, 2023

Copy link
Copy Markdown
Member

/azp run python - azure-communication-identity - tests

@azure-pipelines

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

@yogeshmo
yogeshmo enabled auto-merge (squash) March 16, 2023 21:07
@yogeshmo
yogeshmo merged commit aec6d92 into main Mar 16, 2023
@yogeshmo
yogeshmo deleted the email/add-ppe-livetests branch March 16, 2023 21:58
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.

6 participants