diff --git a/tests/test_permissions.py b/tests/test_permissions.py index b8728d922a1e5aad38db333a96bb984842d086bb..6e042040424d31682b019d77d5cb2d8e114e3609 100644 --- a/tests/test_permissions.py +++ b/tests/test_permissions.py @@ -223,7 +223,7 @@ def test_query(): unique=True).id == exp.id assert db.execute_query( "FIND TestExperiment WHICH HAS A TestConductor WHICH has a TestFirstName=Daniel", - unique = True).id == exp.id + unique=True).id == exp.id '''success''' grant_permission(person, "RETRIEVE:*") @@ -233,7 +233,7 @@ def test_query(): switch_to_test_user() assert db.execute_query( "FIND TestExperiment WHICH HAS A TestConductor WHICH has a TestFirstName=Daniel", - unique = True).id == exp.id + unique=True).id == exp.id '''failure - dan''' deny_permission(dan, "RETRIEVE:*") @@ -244,13 +244,13 @@ def test_query(): with raises(db.EmptyUniqueQueryError): db.execute_query( "FIND TestExperiment WHICH HAS A TestConductor WHICH has a TestFirstName=Daniel", - unique = True) + unique=True) '''... but works without the which clause''' - assert db.execute_query("FIND TestExperiment", unique = True).id == exp.id + assert db.execute_query("FIND TestExperiment", unique=True).id == exp.id '''and with the id''' assert db.execute_query( "FIND TestExperiment WHICH HAS A TestConductor=" + str(dan.id), - unique = True).id == exp.id + unique=True).id == exp.id '''failure - exp''' grant_permission(dan, "RETRIEVE:*") @@ -260,15 +260,15 @@ def test_query(): with raises(db.EmptyUniqueQueryError): db.execute_query( "FIND TestExperiment WHICH HAS A TestConductor=TestDaniel", - unique = True) + unique=True) with raises(db.EmptyUniqueQueryError): - db.execute_query("FIND TestExperiment", unique = True) + db.execute_query("FIND TestExperiment", unique=True) @ with_setup(setup, teardown) def test_update_acl(): - p=db.Property(name = "TestProperty", datatype = db.TEXT).insert() + p = db.Property(name="TestProperty", datatype=db.TEXT).insert() assert_true(p.is_valid()) assert_is_none(p.acl) p.retrieve_acl() @@ -278,24 +278,24 @@ def test_update_acl(): "USE:AS_DATA_TYPE" in p.acl.get_permissions_for_user( db.get_config().get("Connection", "username"))) p.acl.deny( - username = db.get_config().get("Connection", "username"), - permission = "USE:AS_DATA_TYPE") + username=db.get_config().get("Connection", "username"), + permission="USE:AS_DATA_TYPE") assert_false( "USE:AS_DATA_TYPE" in p.acl.get_permissions_for_user( db.get_config().get("Connection", "username"))) '''Success''' - p.update(flags = {"ACL": None}) + p.update(flags={"ACL": None}) assert_true(p.is_valid()) assert_is_not_none(p.acl) assert_false( "USE:AS_DATA_TYPE" in p.acl.get_permissions_for_user( db.get_config().get("Connection", "username"))) - p2=db.execute_query( + p2 = db.execute_query( "FIND TestProperty", - unique = True, - flags = { + unique=True, + flags={ "ACL": None}) assert_true(p2.is_valid()) assert_is_not_none(p2.acl) @@ -310,8 +310,8 @@ def test_update_acl(): "EDIT:ACL" in p2.acl.get_permissions_for_user( db.get_config().get("Connection", "username"))) p2.acl.deny( - username = db.get_config().get("Connection", "username"), - permission = "EDIT:ACL") + username=db.get_config().get("Connection", "username"), + permission="EDIT:ACL") assert_false( "EDIT:ACL" in p2.acl.get_permissions_for_user( db.get_config().get("Connection", "username")))