Set 'check_continuity' default to True for nojump transformation

p-j-smith · Sep 17, 2023 · 847ac79 · 847ac79
1 parent 6549191
commit 847ac79
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/mda_tui/widgets/transformations.py b/src/mda_tui/widgets/transformations.py
@@ -366,7 +366,7 @@ class NoJump(Vertical):
     def compose(self) -> ComposeResult:
         """Create layout of parameter widgets"""
 
-        check_continuity = Switch()
+        check_continuity = Switch(value=True)
 
         # Define tooltips
         check_continuity_tooltip = "check the trajectory step size is not more than 1"

diff --git a/tests/test_transformations.py b/tests/test_transformations.py
@@ -8,6 +8,7 @@
 from textual.widgets import (
     Input,
     Select,
+    Switch,
 )
 
 from mda_tui.widgets import (
@@ -153,6 +154,7 @@ async def test_unwrap(app, universe_filenames: tuple[pathlib.Path, pathlib.Path]
 @pytest.mark.xfail(
     mda.__version__ not in mda_min_version,
     reason="bug in NoJump transformation is fixed in MDAnalysis >2.6.1",
+    strict=True,
 )
 @pytest.mark.asyncio()
 async def test_nojump(app, universe_filenames: tuple[pathlib.Path, pathlib.Path]):
@@ -168,6 +170,7 @@ async def test_nojump(app, universe_filenames: tuple[pathlib.Path, pathlib.Path]
             Select,
         )
         nojump_transformation = pilot.app.query_one(transformations.NoJump)
+        nojump_transformation.query_one(Switch).value = True
 
         topology_reader_input.value = pdb.as_posix()
         trajectory_reader_input.value = xtc.as_posix()