Fix run_experiment not handling param_files of None

Fixes #119

curifactory/experiment.py

@@ -22,7 +22,7 @@
 
 def run_experiment(  # noqa: C901 -- TODO: this does need to be broken up at some point
     experiment_name,
-    param_files,
+    param_files=None,
     overwrite_override=None,
     cache_dir_override=None,
     mngr: ArtifactManager = None,
@@ -170,8 +170,9 @@ def run_experiment(  # noqa: C901 -- TODO: this does need to be broken up at som
 
     if run_string is None:
         run_string = f"experiment {experiment_name}"
-        for param_file in param_files:
-            run_string += f" -p {param_file}"
+        if param_files is not None:
+            for param_file in param_files:
+                run_string += f" -p {param_file}"
         if param_set_names is not None:
             for param_set_name in param_set_names:
                 run_string += f" --names {param_set_name}"

test/test_experiment.py

@@ -428,6 +428,12 @@ def test_empty_parameters_errors(clear_filesystem):
     )
 
 
+def test_experiment_none_parameters_runs(clear_filesystem):
+    """Not passing in any value to run_experiment for parameters should be equivalent to passing [] or implicitly passing the experiment's name."""
+    results, manager = run_experiment("simple_cache")
+    assert len(manager.records) == 4
+
+
 def test_invalid_args_names_errors(clear_filesystem):
     """Using a --names flag but with a non-existant parameterset name should error."""
     with pytest.raises(RuntimeError) as exc_info: