Skip to content
Snippets Groups Projects
Commit cd9e526b authored by Chris Lamb's avatar Chris Lamb 👀
Browse files

In the RData comparator, always explicitly return a None value in the failure...

In the RData comparator, always explicitly return a None value in the failure cases as we return a non-None value in the "success" one.
parent 5f41afe4
No related branches found
No related tags found
No related merge requests found
Pipeline #156061 passed with stage
in 12 minutes and 29 seconds
...@@ -90,7 +90,7 @@ def get_module_path_for_rdb(rdb): ...@@ -90,7 +90,7 @@ def get_module_path_for_rdb(rdb):
# If we are not in a container, we will never be able to locate the # If we are not in a container, we will never be able to locate the
# corresponding .rdx # corresponding .rdx
if rdb.container is None: if rdb.container is None:
return return None
# Calculate location of parallel .rdx file # Calculate location of parallel .rdx file
rdx_name = "{}.rdx".format(os.path.splitext(rdb.name)[0]) rdx_name = "{}.rdx".format(os.path.splitext(rdb.name)[0])
...@@ -105,7 +105,7 @@ def get_module_path_for_rdb(rdb): ...@@ -105,7 +105,7 @@ def get_module_path_for_rdb(rdb):
if not os.path.exists(rdx.path): if not os.path.exists(rdx.path):
# Corresponding .rdx does not exist # Corresponding .rdx does not exist
return return None
temp_dir = get_temporary_directory().name temp_dir = get_temporary_directory().name
prefix = os.path.join(temp_dir, "temp") prefix = os.path.join(temp_dir, "temp")
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment