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

Bump pytest from 7.4.4 to 8.1.1 #8223

Merged
merged 3 commits into from
Apr 19, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions requirements/constraints.txt
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ pip-tools==7.4.1
# via -r requirements/dev.in
platformdirs==2.4.0
# via virtualenv
pluggy==1.0.0
pluggy==1.4.0
# via pytest
pre-commit==3.5.0
# via -r requirements/lint.in
Expand Down Expand Up @@ -164,7 +164,7 @@ pyproject-hooks==1.0.0
# via
# build
# pip-tools
pytest==7.4.4
pytest==8.1.1
# via
# -r requirements/lint.in
# -r requirements/test.in
Expand Down
4 changes: 2 additions & 2 deletions requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ pip-tools==7.4.1
# via -r requirements/dev.in
platformdirs==3.10.0
# via virtualenv
pluggy==1.2.0
pluggy==1.4.0
# via pytest
pre-commit==3.5.0
# via -r requirements/lint.in
Expand All @@ -155,7 +155,7 @@ pyproject-hooks==1.0.0
# via
# build
# pip-tools
pytest==7.4.4
pytest==8.1.1
# via
# -r requirements/lint.in
# -r requirements/test.in
Expand Down
4 changes: 2 additions & 2 deletions requirements/lint.txt
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,11 @@ packaging==23.1
# via pytest
platformdirs==3.10.0
# via virtualenv
pluggy==1.2.0
pluggy==1.4.0
# via pytest
pre-commit==3.5.0
# via -r requirements/lint.in
pytest==7.4.4
pytest==8.1.1
# via -r requirements/lint.in
pyyaml==6.0.1
# via pre-commit
Expand Down
4 changes: 2 additions & 2 deletions requirements/test.txt
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ packaging==23.1
# via
# gunicorn
# pytest
pluggy==1.2.0
pluggy==1.4.0
# via pytest
proxy-py==2.4.4rc4
# via -r requirements/test.in
Expand All @@ -77,7 +77,7 @@ pydantic==2.2.0
# via python-on-whales
pydantic-core==2.6.0
# via pydantic
pytest==7.4.4
pytest==8.1.1
# via
# -r requirements/test.in
# pytest-cov
Expand Down
12 changes: 6 additions & 6 deletions tests/test_pytest_plugin.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@ def test_aiohttp_plugin(testdir) -> None:

from aiohttp import web

value = web.AppKey('value', str)


async def hello(request):
return web.Response(body=b'Hello, world')
Expand Down Expand Up @@ -75,10 +77,10 @@ async def test_noop() -> None:
async def previous(request):
if request.method == 'POST':
with pytest.deprecated_call(): # FIXME: this isn't actually called
request.app['value'] = (await request.post())['value']
request.app[value] = (await request.post())['value']
return web.Response(body=b'thanks for the data')
else:
v = request.app.get('value', 'unknown')
v = request.app.get(value, 'unknown')
return web.Response(body='value: {}'.format(v).encode())


Expand All @@ -98,7 +100,7 @@ async def test_set_value(cli) -> None:
assert resp.status == 200
text = await resp.text()
assert text == 'thanks for the data'
assert cli.server.app['value'] == 'foo'
assert cli.server.app[value] == 'foo'


async def test_get_value(cli) -> None:
Expand All @@ -107,7 +109,7 @@ async def test_get_value(cli) -> None:
text = await resp.text()
assert text == 'value: unknown'
with pytest.warns(DeprecationWarning):
cli.server.app['value'] = 'bar'
cli.server.app[value] = 'bar'
resp = await cli.get('/')
assert resp.status == 200
text = await resp.text()
Expand All @@ -119,7 +121,6 @@ def test_noncoro() -> None:


async def test_failed_to_create_client(aiohttp_client) -> None:

def make_app(loop):
raise RuntimeError()

Expand All @@ -142,7 +143,6 @@ async def test_custom_port_test_server(aiohttp_server, aiohttp_unused_port):
port = aiohttp_unused_port()
server = await aiohttp_server(app, port=port)
assert server.port == port

"""
)
testdir.makeconftest(CONFTEST)
Expand Down
Loading