Skip to content

Commit 977bcf5

Browse files
committed
test(utils): add vector, NA, and integer tests for ud_convert
Addresses reviewer feedback: existing tests only covered scalar inputs. Adds tests for numeric vectors (small and length-1000), NA passthrough, and integer input to improve coverage of real PEcAn usage patterns.
1 parent 4e2a1a6 commit 977bcf5

1 file changed

Lines changed: 17 additions & 0 deletions

File tree

base/utils/tests/testthat/test-ud_convert.R

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,4 +53,21 @@ test_that("model-specific flux conversions", {
5353
test_that("photosynthesis parameters", {
5454
# Photosynthesis energy parameters
5555
expect_equal(ud_convert(1000, "J/mol", "kJ/mol"), 1)
56+
})
57+
58+
test_that("ud_convert handles vector inputs", {
59+
expect_equal(ud_convert(c(0, 10, 100), "degC", "K"), c(273.15, 283.15, 373.15))
60+
x <- seq(0, 100, length.out = 1000)
61+
result <- ud_convert(x, "cm", "m")
62+
expect_equal(result, x / 100)
63+
expect_length(result, 1000)
64+
})
65+
66+
test_that("ud_convert handles NA in vectors", {
67+
result <- ud_convert(c(1, NA, 3), "m", "cm")
68+
expect_equal(result, c(100, NA, 300))
69+
})
70+
71+
test_that("ud_convert handles integer input", {
72+
expect_equal(ud_convert(1L, "m", "cm"), 100)
5673
})

0 commit comments

Comments
 (0)