Switch to positive values and update tests

Author: waltsims

kwave/options/simulation_execution_options.py

@@ -209,7 +209,7 @@ def checkpoint_interval(self) -> Optional[int]:
     @checkpoint_interval.setter
     def checkpoint_interval(self, value: Optional[int]):
         if value is not None:
-            if not isinstance(value, int) or value < 0:
+            if not isinstance(value, int) or value <= 0:
                 raise ValueError("Checkpoint interval must be a positive integer in seconds")
         self._checkpoint_interval = value
 

tests/test_simulation_execution_options.py

@@ -301,12 +301,12 @@ def test_checkpoint_interval_validation(self):
         options = self.default_options
 
         # Test valid values
-        options.checkpoint_interval = 0
-        self.assertEqual(options.checkpoint_interval, 0)
         options.checkpoint_interval = 100
         self.assertEqual(options.checkpoint_interval, 100)
 
         # Test invalid values
+        with self.assertRaises(ValueError):
+            options.checkpoint_interval = 0
         with self.assertRaises(ValueError):
             options.checkpoint_interval = -1
         with self.assertRaises(ValueError):