Apply suggestions from code review

Co-authored-by: Kabilar Gunalan <[email protected]>
kabilar · Mar 2, 2022 · 3f53fed · 3f53fed
1 parent ba8d3e1
commit 3f53fed
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/tests/__init__.py b/tests/__init__.py
@@ -116,7 +116,7 @@ def pipeline():
            'get_ephys_root_data_dir': pipeline.get_ephys_root_data_dir}
     if verbose and _tear_down:
         pipeline.subject.Subject.delete()
-    if _tear_down:
+    elif not verbose and _tear_down:
         with QuietStdOut():
             pipeline.subject.Subject.delete()
 

diff --git a/tests/test_ingest.py b/tests/test_ingest.py
@@ -64,8 +64,7 @@ def test_find_valid_full_path(pipeline, sessions_csv):
 
 def test_find_root_directory(pipeline, sessions_csv):
     """
-    Test that ephys_root_data_dir loaded as docker directory
-        /main/test_data/workflow_ephys_data1/
+    Test that `find_root_directory` works correctly.
     """
     from element_interface.utils import find_root_directory
 
@@ -88,7 +87,7 @@ def test_find_root_directory(pipeline, sessions_csv):
     root_dir = find_root_directory(ephys_root_data_dir, session_full_path)
 
     assert root_dir.as_posix() == '/main/test_data/workflow_ephys_data1',\
-        'Root path does not match docker: /main/test_data/workflow_ephys_data1'
+        'Root path does not match: /main/test_data/workflow_ephys_data1'
 
 
 def test_paramset_insert(kilosort_paramset, pipeline):