Testing: fix linting issues reported by Ruff

Author: Joseph ORON

tests/test_rse.py

@@ -572,11 +572,10 @@ def test_add_protocol(self, vo, rucio_client):
             rucio_client.add_protocol(protocol_rse, p)
         resp = mgr.get_rse_info(rse=protocol_rse, vo=vo)
         for p in resp['protocols']:
-            assert not (((p['port'] == 19) and (p['domains']['lan']['read'] != 1)) or 
-                    ((p['port'] == 20) and (p['domains']['lan']['read'] != 2)) or 
-                    ((p['port'] == 18) and (p['domains']['lan']['read'] != 1)) or 
-                    ((p['port'] == 17) and (p['domains']['lan']['read'] != 4))), f"Protocol {p['port']} has wrong read value {p['domains']['lan']['read']}. Response: {resp}"
-
+            assert not (((p['port'] == 19) and (p['domains']['lan']['read'] != 1)) or
+                        ((p['port'] == 20) and (p['domains']['lan']['read'] != 2)) or
+                        ((p['port'] == 18) and (p['domains']['lan']['read'] != 1)) or
+                        ((p['port'] == 17) and (p['domains']['lan']['read'] != 4))), f"Protocol {p['port']} has wrong read value {p['domains']['lan']['read']}. Response: {resp}"
 
         rucio_client.delete_protocols(protocol_rse, scheme='MOCK')
         rucio_client.delete_rse(protocol_rse)

tests/test_rule.py

@@ -1296,7 +1296,7 @@ def mktree(scope, account):
         mktree(scope, account)
         rule_ids = add_rule(dids=[{'scope': scope, 'name': 'container1213'}], copies=1, rse_expression=f'{self.rse1}|{self.rse2}|{self.rse3}|{self.rse4}',
                             grouping='NONE', account=account, weight=None, lifetime=None, locked=False, subscription_id=None)
-        rule = get_rule(rule_ids[0]
+        rule = get_rule(rule_ids[0])
         assert (rule['locks_ok_cnt'] == 25), f"Expected 25 locks_ok, got {rule['locks_ok_cnt']}"
         assert (rule['locks_replicating_cnt'] == 0), f"Expected 0 locks_replicating, got {rule['locks_replicating_cnt']}"
         dsl1 = list(get_dataset_locks(scope, 'ds1'))