globuzma/canaille-globuzma
1
0
Fork 0
forked from Github-Mirrors/canaille
Code Pull requests Activity

unit tests: removed useless try/except in oidc fixtures

Browse source
This commit is contained in:
Éloi Rivard 2022-12-04 13:41:09 +01:00
parent ab905d77e8
commit 73d6e055d3
2 changed files with 6 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
tests/oidc/conftest.py
View file

@ -44,10 +44,7 @@ def client(testclient, other_client, slapd_connection):
c.save() c.save()
yield c yield c
try:
c.delete() c.delete()
except:
pass
@pytest.fixture @pytest.fixture
@ -83,10 +80,7 @@ def other_client(testclient, slapd_connection):
c.save() c.save()
yield c yield c
try:
c.delete() c.delete()
except:
pass
@pytest.fixture @pytest.fixture
@ -108,10 +102,7 @@ def authorization(testclient, user, client, slapd_connection):
) )
a.save() a.save()
yield a yield a
try:
a.delete() a.delete()
except:
pass
@pytest.fixture @pytest.fixture
@ -130,10 +121,7 @@ def token(testclient, client, user, slapd_connection):
) )
t.save() t.save()
yield t yield t
try:
t.delete() t.delete()
except:
pass
@pytest.fixture @pytest.fixture
@ -156,7 +144,4 @@ def consent(testclient, client, user, slapd_connection):
) )
t.save() t.save()
yield t yield t
try:
t.delete() t.delete()
except:
pass

6
tests/oidc/test_client_admin.py
View file

@ -59,6 +59,7 @@ def test_client_add(testclient, logged_admin):
assert [v] == client_value assert [v] == client_value
else: else:
assert v == client_value assert v == client_value
client.delete()
def test_client_edit(testclient, client, logged_admin, other_client): def test_client_edit(testclient, client, logged_admin, other_client):
@ -106,11 +107,6 @@ def test_client_edit(testclient, client, logged_admin, other_client):
else: else:
assert v == client_value assert v == client_value
res.forms["clientadd"].submit(status=302, name="action", value="delete").follow(
status=200
)
assert Client.get(client.client_id) is None
def test_client_edit_preauth(testclient, client, logged_admin, other_client): def test_client_edit_preauth(testclient, client, logged_admin, other_client):
assert not client.preconsent assert not client.preconsent