PEP8: fix tests for membership

datarequest.py

@@ -889,7 +889,7 @@ def file_write_and_lock(ctx, coll_path, filename, data, readers):
         msi.set_acl(ctx, "default", "read", reader, file_path)
 
     # Revoke temporary write permission (unless read permissions were set on the invoking user)
-    if not user.full_name(ctx) in readers:
+    if user.full_name(ctx) not in readers:
         msi.set_acl(ctx, "default", "null", user.full_name(ctx), file_path)
     # If invoking user is request owner, set read permission for this user on the collection again,
     # else revoke individual user permissions on collection entirely (invoking users will still have

deposit.py

@@ -70,7 +70,7 @@ def api_deposit_copy_data_package(ctx, reference, deposit_group):
 
     # Check if user has write access to research folder.
     # Only normal user has write access.
-    if not groups.user_role(ctx, user_full_name, group_name) in ['normal', 'manager']:
+    if groups.user_role(ctx, user_full_name, group_name) not in ['normal', 'manager']:
         return api.Error('NoWriteAccessTargetCollection', 'Not permitted to write in selected folder')
 
     # Register to delayed rule queue.

vault.py

@@ -225,7 +225,7 @@ def api_vault_copy_to_research(ctx, coll_origin, coll_target):
 
     # Check if user has write access to research folder.
     # Only normal user has write access.
-    if not groups.user_role(ctx, user_full_name, group_name) in ['normal', 'manager']:
+    if groups.user_role(ctx, user_full_name, group_name) not in ['normal', 'manager']:
         return api.Error('NoWriteAccessTargetCollection', 'Not permitted to write in selected folder')
 
     # Register to delayed rule queue.