This commit is contained in:
Alexandre Aubin 2022-12-22 05:05:43 +01:00
parent 4775b40b95
commit ed77bcc29a

View file

@ -75,7 +75,7 @@ def test_provision_dummy():
assert not os.path.exists(dummyfile) assert not os.path.exists(dummyfile)
AppResourceManager("testapp", current=current, wanted=wanted).apply( AppResourceManager("testapp", current=current, wanted=wanted).apply(
rollback_if_failure=False rollback_and_raise_exception_if_failure=False
) )
assert open(dummyfile).read().strip() == "foo" assert open(dummyfile).read().strip() == "foo"
@ -89,7 +89,7 @@ def test_deprovision_dummy():
assert open(dummyfile).read().strip() == "foo" assert open(dummyfile).read().strip() == "foo"
AppResourceManager("testapp", current=current, wanted=wanted).apply( AppResourceManager("testapp", current=current, wanted=wanted).apply(
rollback_if_failure=False rollback_and_raise_exception_if_failure=False
) )
assert not os.path.exists(dummyfile) assert not os.path.exists(dummyfile)
@ -101,7 +101,7 @@ def test_provision_dummy_nondefaultvalue():
assert not os.path.exists(dummyfile) assert not os.path.exists(dummyfile)
AppResourceManager("testapp", current=current, wanted=wanted).apply( AppResourceManager("testapp", current=current, wanted=wanted).apply(
rollback_if_failure=False rollback_and_raise_exception_if_failure=False
) )
assert open(dummyfile).read().strip() == "bar" assert open(dummyfile).read().strip() == "bar"
@ -115,7 +115,7 @@ def test_update_dummy():
assert open(dummyfile).read().strip() == "foo" assert open(dummyfile).read().strip() == "foo"
AppResourceManager("testapp", current=current, wanted=wanted).apply( AppResourceManager("testapp", current=current, wanted=wanted).apply(
rollback_if_failure=False rollback_and_raise_exception_if_failure=False
) )
assert open(dummyfile).read().strip() == "bar" assert open(dummyfile).read().strip() == "bar"
@ -130,7 +130,7 @@ def test_update_dummy_fail():
assert open(dummyfile).read().strip() == "foo" assert open(dummyfile).read().strip() == "foo"
with pytest.raises(Exception): with pytest.raises(Exception):
AppResourceManager("testapp", current=current, wanted=wanted).apply( AppResourceManager("testapp", current=current, wanted=wanted).apply(
rollback_if_failure=False rollback_and_raise_exception_if_failure=False
) )
assert open(dummyfile).read().strip() == "forbiddenvalue" assert open(dummyfile).read().strip() == "forbiddenvalue"
@ -145,7 +145,7 @@ def test_update_dummy_failwithrollback():
assert open(dummyfile).read().strip() == "foo" assert open(dummyfile).read().strip() == "foo"
with pytest.raises(Exception): with pytest.raises(Exception):
AppResourceManager("testapp", current=current, wanted=wanted).apply( AppResourceManager("testapp", current=current, wanted=wanted).apply(
rollback_if_failure=True rollback_and_raise_exception_if_failure=True
) )
assert open(dummyfile).read().strip() == "foo" assert open(dummyfile).read().strip() == "foo"
@ -397,9 +397,7 @@ def test_resource_permissions():
res = user_permission_list(full=True)["permissions"] res = user_permission_list(full=True)["permissions"]
# FIXME FIXME FIXME : this is the current behavior but assert res["testapp.admin"]["url"] == "/adminpanel"
# it is NOT okay. c.f. comment in the code
assert res["testapp.admin"]["url"] == "/admin" # should be '/adminpanel'
r(conf, "testapp").deprovision() r(conf, "testapp").deprovision()