outline tests we need to do

dwhswenson · May 15, 2021 · May 17, 2021 · Jun 9, 2021 · Jun 17, 2021 · Jul 5, 2021
commit 31df2f275291ac4813c203d02dd6b6936e0b8616
diff --git a/openpathsampling/analysis/filters.py b/openpathsampling/analysis/filters.py
@@ -410,7 +410,7 @@ def _get_modified_shooting_point(step):
         return NOT_EXTRACTED
     details = step.change.canonical.details
     try:
-        shooting_pt = ...
+        shooting_pt = details.modified_shooting_snapshot
     except AttributeError:
         # return the original shooting point for one-way
         shooting_pt = details.shooting_snapshot

diff --git a/openpathsampling/tests/analysis/test_filters.py b/openpathsampling/tests/analysis/test_filters.py
@@ -1,3 +1,75 @@
 from openpathsampling.analysis.filters import *
 
-# need to build a pretty complex test suite here
+import pytest
+
+@pytest.fixture
+def mstis_network():
+    pass
+
+def make_shooting_move(scheme, accepted=None, ensemble=None):
+    pass
+
+def make_repex_move(scheme, accepted=None, ensembles=None):
+    pass
+
+def make_msouter_move(scheme, accepted=None):
+    pass
+
+def make_minus_move(scheme, accepted=None):
+    pass
+
+def test_canonical_mover_step_filter():
+    pass
+
+def test_trial_replica_step_filter():
+    pass
+
+def test_trial_ensemble_step_filter():
+    pass
+
+def test_rejected_steps_filter():
+    pass
+
+def test_accepted_steps_filter():
+    pass
+
+def test_all_steps_filter():
+    pass
+
+def test_all_samples_filter():
+    pass
+
+def test_ensemble_sample_filter():
+    pass
+
+def test_replica_sample_filter():
+    pass
+
+def test_minus_ensemble_sample_filter():
+    pass
+
+def test_ms_outer_ensemble_sample_filter():
+    pass
+
+def test_sampling_ensemble_sample_filter():
+    pass
+
+def test_active_samples_extractor():
+    pass
+
+def test_trial_samples_extractor():
+    pass
+
+def test_shooting_steps_filter():
+    pass
+
+def test_shooting_points_extractor():
+    pass
+
+def test_modified_shooting_points_extractor():
+    pass
+
+def test_canonical_movers_extractor():
+    pass
+
+