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.

Sign up for GitHub

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

Jump to bottom

Mock httpbin for requests tests #1844

Closed
ocelotl opened this issue Jun 12, 2023 · 0 comments · Fixed by #1854
Closed

Mock httpbin for requests tests #1844

ocelotl opened this issue Jun 12, 2023 · 0 comments · Fixed by #1854
Assignees
@ocelotl

Comments

@ocelotl
Copy link
Contributor

ocelotl commented Jun 12, 2023

No description provided.

@ocelotl ocelotl self-assigned this Jun 12, 2023
ocelotl added a commit to ocelotl/opentelemetry-python-contrib that referenced this issue Jun 12, 2023
@ocelotl
Remove use of httpbin
2d4ecd3 
This is done in order to prevent confusion. We are trying to stop using
httpbin.org for our tests. Even when the tests for the requests
instrumentation do not actually perform any request to httpbin.org,
because the test requests are being mocked with httpretty, having the
string httpbin.org in the tests can cause confusion and make the reader
think the tests are actually using httpbin.org.

Fixes open-telemetry#1844
@ocelotl ocelotl mentioned this issue Jun 12, 2023
Closed
ocelotl added a commit to ocelotl/opentelemetry-python-contrib that referenced this issue Jun 12, 2023
@ocelotl
Remove use of httpbin
c78cafb 
This is done in order to prevent confusion. We are trying to stop using
httpbin.org for our tests. Even when the tests for the requests
instrumentation do not actually perform any request to httpbin.org,
because the test requests are being mocked with httpretty, having the
string httpbin.org in the tests can cause confusion and make the reader
think the tests are actually using httpbin.org.

Fixes open-telemetry#1844
ocelotl added a commit to ocelotl/opentelemetry-python-contrib that referenced this issue Jun 12, 2023
@ocelotl
Remove use of httpbin
da23559 
This is done in order to prevent confusion. We are trying to stop using
httpbin.org for our tests. Even when the tests for the requests
instrumentation do not actually perform any request to httpbin.org,
because the test requests are being mocked somehow having the string
httpbin.org in the tests can cause confusion and make the reader
think the tests are actually making external requests to  httpbin.org.

Fixes open-telemetry#1844 open-telemetry#1843 open-telemetry#1845 open-telemetry#1846 open-telemetry#1847 open-telemetry#1848
@ocelotl ocelotl mentioned this issue Jun 12, 2023
Merged
ocelotl added a commit to ocelotl/opentelemetry-python-contrib that referenced this issue Jun 13, 2023
@ocelotl
Remove use of httpbin
5df0403 
This is done in order to prevent confusion. We are trying to stop using
httpbin.org for our tests. Even when the tests for the requests
instrumentation do not actually perform any request to httpbin.org,
because the test requests are being mocked somehow having the string
httpbin.org in the tests can cause confusion and make the reader
think the tests are actually making external requests to  httpbin.org.

Fixes open-telemetry#1844 open-telemetry#1843 open-telemetry#1845 open-telemetry#1846 open-telemetry#1847 open-telemetry#1848
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ocelotl ocelotl

Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@ocelotl