refactor fix to use fewer statements

Author: bohendo

crytic_compile/utils/naming.py

@@ -106,23 +106,24 @@ def convert_filename(
             filename = filename.relative_to(Path(crytic_compile.package_name))
         except ValueError:
             pass
+
     if not filename.exists():
+        if cwd.joinpath(Path("contracts"), filename).exists():
+            filename = cwd.joinpath("contracts", filename)
+        elif working_dir.joinpath(filename).exists():
+            filename = working_dir.joinpath(filename)
         # how node.js loads dependencies from node_modules:
         # https://nodejs.org/api/modules.html#loading-from-node_modules-folders
-        if cwd.joinpath(Path("node_modules"), filename).exists():
+        elif cwd.joinpath(Path("node_modules"), filename).exists():
             filename = cwd.joinpath("node_modules", filename)
         else:
-            for folder in cwd.parents:
-                if folder.joinpath(Path("node_modules"), filename).exists():
-                    filename = folder.joinpath(Path("node_modules"), filename)
+            for parent in cwd.parents:
+                if parent.joinpath(Path("node_modules"), filename).exists():
+                    filename = parent.joinpath(Path("node_modules"), filename)
                     break
+
     if not filename.exists():
-        if cwd.joinpath(Path("contracts"), filename).exists():
-            filename = cwd.joinpath("contracts", filename)
-        elif working_dir.joinpath(filename).exists():
-            filename = working_dir.joinpath(filename)
-        else:
-            raise InvalidCompilation(f"Unknown file: {filename}")
+        raise InvalidCompilation(f"Unknown file: {filename}")
     elif not filename.is_absolute():
         filename = cwd.joinpath(filename)