Fix coverage flags

emilydolson · Jul 9, 2024 · cccec70 · cccec70
1 parent df39578
commit cccec70
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/Makefile b/Makefile
@@ -18,7 +18,7 @@ debug:
 	$(CXX) $(FLAG) $(DEBUG_flags) $(INCLUDE) systematics_bindings.cpp -o phylotrackpy/systematics$(SUFFIX)
 
 coverage:
-	$(CXX) $(FLAG) $(INCLUDE) -DNDEBUG -fprofile-arcs -ftest-coverage --coverage systematics_bindings.cpp -o phylotrackpy/systematics$(SUFFIX)
+	$(CXX) $(FLAG) $(INCLUDE) -DNDEBUG -DEMP_OPTIONAL_THROW_ON -fprofile-arcs -ftest-coverage --coverage systematics_bindings.cpp -o phylotrackpy/systematics$(SUFFIX)
 
 clean:
 	rm -rf phylotrackpy/systematics$(SUFFIX)
diff --git a/systematics_bindings.cpp b/systematics_bindings.cpp
@@ -418,7 +418,7 @@ PYBIND11_MODULE(systematics, m) {
         .def("get_num_roots", static_cast<size_t (sys_t::*) () const>(&sys_t::GetNumRoots), R"mydelimiter(
             Returns the number of independent phylogenies currently being tracked.
         )mydelimiter")
-        .def("get_next_id", static_cast<size_t (sys_t::*) () const>(&sys_t::GetNextID), R"mydelimiter(
+        .def("get_next_id", static_cast<uint64_t (sys_t::*) () const>(&sys_t::GetNextID), R"mydelimiter(
             Returns the ID of the next taxon that will be created.
         )mydelimiter")
         .def("get_ave_depth", static_cast<double (sys_t::*) () const>(&sys_t::GetAveDepth), R"mydelimiter(

diff --git a/test/test_systematics.py b/test/test_systematics.py
@@ -291,7 +291,7 @@ def test_shared_ancestor():
 def test_bad_load():
     sys = systematics.Systematics(taxon_info_fun, True, True, False, False)
     with raises(RuntimeError):
-        sys.load_from_file(f"fake.file.path", "genome", True)
+        sys.load_from_file("fake.file.path", "genome", True)
 
 
 def test_load_data():