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

Add test base class with local HTTP server. #2101

Merged
merged 4 commits into from
Sep 20, 2021

Conversation

Oberon00
Copy link
Member

@Oberon00 Oberon00 commented Sep 6, 2021

Description

This is a new test base class that starts a localhost HTTP server. It is used for tests in this contrib PR: open-telemetry/opentelemetry-python-contrib#661

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

This is a test utility. It has not been tested by itself.

Does This PR Require a Contrib Repo Change?

No, but this change is required for a contrib PR: open-telemetry/opentelemetry-python-contrib#661

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • [ ] Unit tests have been added
  • [ ] Documentation has been updated

@lzchen lzchen added the Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary label Sep 16, 2021
@Oberon00
Copy link
Member Author

Thank you for reviewing & approving this PR. The dependent open-telemetry/opentelemetry-python-contrib#661 would also need some 😃

@lzchen lzchen merged commit d9c22a8 into open-telemetry:main Sep 20, 2021
@Oberon00 Oberon00 deleted the http-test branch September 20, 2021 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approve Public API check This label shows that the public symbols added or changed in a PR are strictly necessary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants