Skip to content

Conversation

@brettlangdon
Copy link
Member

Description

Backport of #4907 to 1.7

The user agent used in testing may change as the version of requests being used changes.

This change ensure that we do not consider the http.useragent tag value in the snapshots for the sanic tests.

Checklist

Reviewer Checklist

  • Title is accurate.
  • Description motivates each change.
  • No unnecessary changes were introduced in this PR.
  • Avoid breaking API changes unless absolutely necessary.
  • Tests provided or description of manual testing performed is included in the code or PR.
  • Release note has been added and follows the library release note guidelines, or else changelog/no-changelog label added.
  • All relevant GitHub issues are correctly linked.
  • Backports are identified and tagged with Mergifyio.

…pshots (#4907)

## Description
The user agent used in testing may change as the version of `requests`
being used changes.

This change ensure that we do not consider the `http.useragent` tag
value in the snapshots for the sanic tests.

## Checklist
- [ ] Followed the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/contributing.html#Release-Note-Guidelines)
when writing a release note.
- [ ] Add additional sections for `feat` and `fix` pull requests.
- [ ] [Library
documentation](https://github.com/DataDog/dd-trace-py/tree/1.x/docs)
and/or [Datadog's documentation
site](https://github.com/DataDog/documentation/) is updated. Link to doc
PR in description.

## Reviewer Checklist
- [ ] Title is accurate.
- [ ] Description motivates each change.
- [ ] No unnecessary changes were introduced in this PR.
- [ ] Avoid breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes unless absolutely necessary.
- [ ] Tests provided or description of manual testing performed is
included in the code or PR.
- [ ] Release note has been added and follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/contributing.html#Release-Note-Guidelines),
or else `changelog/no-changelog` label added.
- [ ] All relevant GitHub issues are correctly linked.
- [ ] Backports are identified and tagged with Mergifyio.
@brettlangdon brettlangdon added the changelog/no-changelog A changelog entry is not required for this PR. label Jan 14, 2023
@brettlangdon brettlangdon requested a review from a team as a code owner January 14, 2023 01:04
@ZStriker19 ZStriker19 changed the title chore(sanic): ensure sanic tests ignore the http.userganet tag in snapshots [backport #4907 to 1.7] chore(sanic): ensure sanic tests ignore the http.useragent tag in snapshots [backport #4907 to 1.7] Jan 14, 2023
@brettlangdon brettlangdon merged commit 04da476 into 1.7 Jan 14, 2023
@brettlangdon brettlangdon deleted the backport.4907 branch January 14, 2023 03:15
@github-actions github-actions bot added this to the v1.7.1 milestone Jan 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog A changelog entry is not required for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants