Skip to content

Replace httpretty with responses#48

Merged
blearandy merged 2 commits intodevelopfrom
tests/responses
Aug 18, 2025
Merged

Replace httpretty with responses#48
blearandy merged 2 commits intodevelopfrom
tests/responses

Conversation

@NjM3MjY5NzAgNzA3MzA3
Copy link
Copy Markdown
Collaborator

httpretty is not maintained and no wheel is available requiring it be built, replace it with responses. Remove httpretty and mock.
Use HTTPStatus.value.
Replace remove_None_fields_from_dict, was leaving empty lists and dicts, with remove_empty_fields_from_dict.
Remove days option from csr, this option is invalid and not included in the csr.

httpretty is not maintained and no wheel is available requiring it be built, replace it with responses.
Remove httpretty and mock.
Use HTTPStatus.value.
Replace remove_None_fields_from_dict, was leaving empty lists and dicts, with remove_empty_fields_from_dict
Remove days option from csr, this option is invalid and not included in the csr.

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
@NjM3MjY5NzAgNzA3MzA3 NjM3MjY5NzAgNzA3MzA3 self-assigned this Aug 12, 2025
@NjM3MjY5NzAgNzA3MzA3 NjM3MjY5NzAgNzA3MzA3 added the enhancement New feature or request label Aug 12, 2025
@NjM3MjY5NzAgNzA3MzA3 NjM3MjY5NzAgNzA3MzA3 marked this pull request as draft August 12, 2025 23:17
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting. But guess it's probably better to use https on these anyway

Copy link
Copy Markdown
Collaborator

@blearandy blearandy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Man. You've been busy. Really appreciate looking at all this and updating it!

Get the local timezone where test is running.
Add tzdata for Windows.

Signed-off-by: NjM3MjY5NzAgNzA3MzA3 <102704081+NjM3MjY5NzAgNzA3MzA3@users.noreply.github.com>
@NjM3MjY5NzAgNzA3MzA3 NjM3MjY5NzAgNzA3MzA3 marked this pull request as ready for review August 15, 2025 18:45
@blearandy blearandy merged commit 9da4f6c into develop Aug 18, 2025
13 checks passed
@blearandy blearandy deleted the tests/responses branch August 18, 2025 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants