🎈tests

DESCRIPTION

@@ -1,6 +1,6 @@
 Package: dodgr
 Title: Distances on Directed Graphs
-Version: 0.2.0.008
+Version: 0.2.0.009
 Authors@R: c(
     person("Mark", "Padgham", email="[email protected]", role=c("aut", "cre")),
     person("Andreas", "Petutschnig", role="aut"),

src/fund-cycles.h

@@ -108,7 +108,7 @@ class AdjacencyMatrix
 inline void AdjacencyMatrix::connect (size_t i, size_t j)
 {
     if (AdjacencyMatrix::m_Adjacencies [AdjacencyMatrix::get_adj_index (i, j)])
-        return;
+        return; // nocov
 
     AdjacencyMatrix::m_Adjacencies [AdjacencyMatrix::get_adj_index (i, j)] = true;
     AdjacencyMatrix::m_nEdges++;
@@ -117,7 +117,7 @@ inline void AdjacencyMatrix::connect (size_t i, size_t j)
 inline void AdjacencyMatrix::disconnect (size_t i, size_t j)
 {
     if (!AdjacencyMatrix::m_Adjacencies [AdjacencyMatrix::get_adj_index (i, j)])
-        return;
+        return; // nocov
 
     AdjacencyMatrix::m_Adjacencies [AdjacencyMatrix::get_adj_index (i, j)] = false;
     AdjacencyMatrix::m_nEdges--;
@@ -189,7 +189,7 @@ template <class TObject>
 void Graph<TObject>::computeFundamentalCycles()
 {
     if (!Graph<TObject>::m_fundamentalCycles.empty())
-        return;
+        return; // nocov
 
     std::unique_ptr <TreeNode []>
         aTree (new TreeNode [Graph::m_nodeArray.size ()]);

tests/testthat/test-iso.R

@@ -26,6 +26,19 @@ test_that("isodists", {
               expect_true (length (unique (d$dlim)) == length (dlim))
 })
 
+test_that ("errors", {
+              expect_silent (hsc <- sf_to_sc (hampi))
+              expect_silent (net <- weight_streetnet (hsc,
+                                                      wt_profile = "bicycle"))
+              npts <- 100
+              from <- sample (net$.vx0, size = npts)
+              dlim <- c (1, 2, 5, 10, 20) * 100
+              expect_error (d <- dodgr_isodists (net, from = from),
+                            "dlim must be specified")
+              expect_error (d <- dodgr_isodists (net, from = from, dlim = "a"),
+                            "dlim must be numeric")
+             })
+
 test_that("isochrones", {
               expect_silent (hsc <- sf_to_sc (hampi))
               # This all exists just to test the next line: