Skip to content

Commit 055ed1d

Browse files
committed
Made updates for epi_archive testing.
1 parent d1a902a commit 055ed1d

File tree

7 files changed

+102
-51
lines changed

7 files changed

+102
-51
lines changed

man/as_epi_archive.Rd

Lines changed: 8 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

man/epi_archive.Rd

Lines changed: 18 additions & 18 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

man/epi_slide.Rd

Lines changed: 8 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

man/epix_as_of.Rd

Lines changed: 8 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

man/epix_merge.Rd

Lines changed: 8 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

man/epix_slide.Rd

Lines changed: 8 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/testthat/test-methods-epi_archive.R

Lines changed: 44 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -23,18 +23,49 @@ test_that("Warning against max_version being same as edf's max version",{
2323
})
2424

2525
test_that("as_of properly grabs the data",{
26-
df_as_of <- ea %>%
27-
epix_as_of(max_version = as.Date("2020-07-01")) %>%
28-
na.omit() %>%
29-
as.data.frame()
30-
31-
df_filter <- ea$DT %>%
32-
filter(version == as.Date("2020-07-01")) %>%
33-
na.omit() %>%
34-
select(-version) %>%
35-
as.data.frame()
26+
d <- as.Date("2020-06-01")
3627

37-
expect_equal(df_as_of[1:4],df_filter)
28+
generate_time_values <- function() {
29+
x <- c()
30+
for (i in 0:3) {
31+
x <- c(rep(d+i,4),x)
32+
}
33+
return(rev(x))
34+
}
35+
36+
generate_versions <- function() {
37+
x <- c()
38+
for (i in 0:3) {
39+
x <- c(d+i,x)
40+
}
41+
x <- rep(x,4)
42+
return(rev(x))
43+
}
44+
45+
ea2 <- tibble(
46+
geo_value = "ca",
47+
time_value = generate_time_values(),
48+
version = generate_versions()
49+
) %>%
50+
filter(version >= time_value)
51+
52+
ea2 <- ea2[-c(9,4,3),]
53+
ea2 <- bind_cols(ea2,cases=c(1,2,0,1,2,1,4))
54+
ea2 <- as_epi_archive(ea2)
55+
56+
df_as_of <- ea2 %>%
57+
epix_as_of(max_version = as.Date("2020-06-03")) %>%
58+
as_tibble()
59+
60+
df_expected <- tibble(
61+
geo_value = "ca",
62+
time_value = d + 0:2,
63+
cases = c(2,1,1)
64+
)
65+
66+
expect_identical(df_as_of[[1]],df_expected[[1]])
67+
expect_identical(df_as_of[[2]],df_expected[[2]])
68+
expect_identical(df_as_of[[3]],df_expected[[3]])
3869
})
3970

4071
# epix_merge tests
@@ -50,7 +81,7 @@ test_that("data.table merging is utilized if second argument is a data.table",{
5081

5182
expect_identical(
5283
epix_merge(ea1,dt2),
53-
merge(dt1,dt2)
84+
merge(dt1,dt2,all=TRUE)
5485
)
5586
})
5687

@@ -62,6 +93,6 @@ test_that("data.table merging works as intended",{
6293

6394
expect_identical(
6495
as_epi_archive(ea$DT),
65-
as_epi_archive(merge(dt1,dt2))
96+
as_epi_archive(merge(dt1,dt2,all=TRUE))
6697
)
6798
})

0 commit comments

Comments
 (0)