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

[AzureMonitorExporter] Split Integration.Tests project #29703

Merged
merged 6 commits into from
Jul 7, 2022

Conversation

TimothyMothra
Copy link
Contributor

The current "Integration.Tests" project contains two kinds of tests, using either ASP.NET Core Integration tests or the Azure Test Framework.
For easier maintenance, I've split these tests into two separate projects: "Integration.Tests" and "E2E.Tests".

In future PRs, i'll be working on getting the individual tests working again.
New tests will be aligned with the planned Milestones.

Changes

  • New project "E2E.Tests".
    Moved Azure Test Framework tests into this project.
  • Added a README.md to both projects.
  • Normalized namespaces in both projects.
  • Removed unused PackageReferences

image

image

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

@TimothyMothra
Copy link
Contributor Author

To pass the checks, I had to add an empty template to the new README files.
This isn't ideal, if we don't like this we can remove the files

@TimothyMothra TimothyMothra merged commit b62fd9f into main Jul 7, 2022
@TimothyMothra TimothyMothra deleted the tilee/test_projects branch July 7, 2022 22:57
zhihaoxue pushed a commit to zhihaoxue/azure-sdk-for-net that referenced this pull request Jul 27, 2022
* initial commit

* readme

* cleanup

* fix link according to repo guidelines

* azure sdk readme guidelines

* remove readme files
sofiar-msft pushed a commit to sofiar-msft/azure-sdk-for-net that referenced this pull request Dec 7, 2022
* initial commit

* readme

* cleanup

* fix link according to repo guidelines

* azure sdk readme guidelines

* remove readme files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Monitor - Exporter Monitor OpenTelemetry Exporter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants