DEV: remove pytest skips on windows

Author: rmcar17

tests/test_app/test_app.py

@@ -111,6 +111,7 @@ def test_mfinder_result_roundtrip(five_otu: Alignment) -> None:
     assert isinstance(inflated, ModelFinderResult)
     assert str(got.best_aicc) == str(inflated.best_aicc)
 
+
 @pytest.mark.skipif(cogent3_vers < "2025.3.1", reason="requires cogent3 >= 2025.3.1")
 @pytest.mark.parametrize("use_hook", [None, "piqtree"])
 def test_quick_tree_hook(four_otu: Alignment, use_hook: str | None) -> None:

tests/test_iqtree/test_build_tree.py

@@ -1,4 +1,3 @@
-import platform
 import re
 
 import pytest
@@ -88,10 +87,6 @@ def test_rate_model_build_tree(
     )
 
 
-@pytest.mark.skipif(
-    platform.system() == "Windows",
-    reason="IQ-TREE errors can't be caught yet on Windows",
-)
 def test_build_tree_inadequate_bootstrapping(four_otu: ArrayAlignment) -> None:
     with pytest.raises(IqTreeError, match=re.escape("#replicates must be >= 1000")):
         piqtree.build_tree(four_otu, Model(DnaModel.GTR), bootstrap_replicates=10)

tests/test_iqtree/test_random_trees.py

@@ -1,5 +1,3 @@
-import platform
-
 import pytest
 
 import piqtree
@@ -47,10 +45,6 @@ def test_random_trees_no_seed(
 
 @pytest.mark.parametrize("num_taxa", [-1, 0, 1, 2])
 @pytest.mark.parametrize("tree_mode", list(piqtree.TreeGenMode))
-@pytest.mark.skipif(
-    platform.system() == "Windows",
-    reason="IQ-TREE errors can't be caught yet on Windows",
-)
 def test_invalid_taxa(
     num_taxa: int,
     tree_mode: piqtree.TreeGenMode,

tests/test_iqtree/test_segmentation_fault.py

@@ -1,7 +1,5 @@
 """Test combinations of calls which under previous versions resulted in a segmentation fault."""
 
-import platform
-
 import pytest
 from cogent3 import make_aligned_seqs, make_tree
 
@@ -10,10 +8,6 @@
 from piqtree.model import DiscreteGammaModel, DnaModel, FreeRateModel, Model
 
 
-@pytest.mark.skipif(
-    platform.system() == "Windows",
-    reason="IQ-TREE errors can't be caught yet on Windows",
-)
 def test_two_build_random_trees() -> None:
     """
     Calling build tree twice followed by random trees with a bad input
@@ -28,10 +22,6 @@ def test_two_build_random_trees() -> None:
         random_trees(3, 2, TreeGenMode.BALANCED, 1)
 
 
-@pytest.mark.skipif(
-    platform.system() == "Windows",
-    reason="IQ-TREE errors can't be caught yet on Windows",
-)
 def test_two_fit_random_trees() -> None:
     """
     Calling fit tree twice followed by random trees with a bad input
@@ -49,10 +39,6 @@ def test_two_fit_random_trees() -> None:
 
 @pytest.mark.parametrize("rate_model_class", [DiscreteGammaModel, FreeRateModel])
 @pytest.mark.parametrize("categories", [0, -4])
-@pytest.mark.skipif(
-    platform.system() == "Windows",
-    reason="IQ-TREE errors can't be caught yet on Windows",
-)
 def test_two_invalid_models(
     rate_model_class: type[DiscreteGammaModel] | type[FreeRateModel],
     categories: int,