Merge pull request #3158 from IntersectMBO/better_xor

improve: logic for paired argument validation

Author: mkoura

cardano_node_tests/testnet_cleanup.py

@@ -53,7 +53,7 @@ def main() -> int:
     if not socket_env:
         LOGGER.error("The `CARDANO_NODE_SOCKET_PATH` environment variable is not set.")
         return 1
-    if bool(args.address) ^ bool(args.skey_file):
+    if bool(args.address) != bool(args.skey_file):
         LOGGER.error(
             "Both address and skey file must be provided, or neither of them should be provided."
         )

cardano_node_tests/tests/test_scripts.py

@@ -55,7 +55,7 @@ def multisig_tx(
     build_method: str = clusterlib_utils.BuildMethods.BUILD_RAW,
 ) -> clusterlib.TxRawOutput:
     """Build and submit multisig transaction."""
-    if bool(multisig_script) ^ bool(script_utxos):
+    if bool(multisig_script) != bool(script_utxos):
         err = "Both `multisig_script` and `script_utxos` must be provided"
         raise ValueError(err)