Skip to content

Fix a DeprecationWarning from aiohttp in TestPetPhotoView - #836

Merged
p1c2u merged 1 commit into
python-openapi:masterfrom
musicinmybrain:aiohttp-3.9.4
Apr 24, 2024
Merged

Fix a DeprecationWarning from aiohttp in TestPetPhotoView#836
p1c2u merged 1 commit into
python-openapi:masterfrom
musicinmybrain:aiohttp-3.9.4

Conversation

@musicinmybrain

Copy link
Copy Markdown
Contributor

The warning says, “In v4, passing bytes will no longer create a file field. Please explicitly use the filename parameter or pass a BytesIO object.” This commit implements the latter approach.

The warning says, “In v4, passing bytes will no longer create a file
field. Please explicitly use the filename parameter or pass a BytesIO
object.” This commit implements the latter approach.
@codecov

codecovBot commented Apr 22, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.28%. Comparing base (fb0ea95) to head (72c9dca).
Report is 23 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #836 +/- ##
==========================================
+ Coverage 90.26% 90.28% +0.02% 
==========================================
Files 147 148 +1 Lines 4302 4313 +11 Branches 524 527 +3 ==========================================
+ Hits 3883 3894 +11 
Misses 317 317 Partials 102 102 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@musicinmybrain

Copy link
Copy Markdown
ContributorAuthor

I failed to reference the bug that this fixes: #835

@p1c2up1c2u left a comment

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.

LGTM

@p1c2u

Copy link
Copy Markdown
Collaborator

@musicinmybrain thanks for the contribution

@p1c2u
p1c2u merged commit 9f42906 into python-openapi:masterApr 24, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@musicinmybrain@p1c2u