Skip to content

Commit ff3dfae

Browse files
committed
doc: minor annotations
1 parent 0bf702f commit ff3dfae

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

tests/testthat/test-methods-epi_df.R

+6
Original file line numberDiff line numberDiff line change
@@ -128,11 +128,13 @@ test_that("Metadata and grouping are dropped by `as_tibble`", {
128128
test_that("Renaming columns gives appropriate colnames and metadata", {
129129
edf <- tibble::tibble(geo_value = 1, time_value = 1, age = 1, value = 1) %>%
130130
as_epi_df(additional_metadata = list(other_keys = "age"))
131+
# renaming using base R
131132
renamed_edf1 <- edf %>%
132133
`[`(c("geo_value", "time_value", "age", "value")) %>%
133134
`names<-`(c("geo_value", "time_value", "age_group", "value"))
134135
expect_identical(names(renamed_edf1), c("geo_value", "time_value", "age_group", "value"))
135136
expect_identical(attr(renamed_edf1, "metadata")$other_keys, c("age_group"))
137+
# renaming using select
136138
renamed_edf2 <- edf %>%
137139
as_epi_df(additional_metadata = list(other_keys = "age")) %>%
138140
select(geo_value, time_value, age_group = age, value)
@@ -143,13 +145,17 @@ test_that("Renaming columns while grouped gives appropriate colnames and metadat
143145
gedf <- tibble::tibble(geo_value = 1, time_value = 1, age = 1, value = 1) %>%
144146
as_epi_df(additional_metadata = list(other_keys = "age")) %>%
145147
group_by(geo_value)
148+
# renaming using base R
146149
renamed_gedf1 <- gedf %>%
147150
`[`(c("geo_value", "time_value", "age", "value")) %>%
148151
`names<-`(c("geo_value", "time_value", "age_group", "value"))
152+
# tets type preservation
149153
expect_true(inherits(renamed_gedf1, "epi_df"))
150154
expect_true(inherits(renamed_gedf1, "grouped_df"))
155+
# the names are right
151156
expect_identical(names(renamed_gedf1), c("geo_value", "time_value", "age_group", "value"))
152157
expect_identical(attr(renamed_gedf1, "metadata")$other_keys, c("age_group"))
158+
# renaming using select
153159
renamed_gedf2 <- gedf %>%
154160
as_epi_df(additional_metadata = list(other_keys = "age")) %>%
155161
select(geo_value, time_value, age_group = age, value)

0 commit comments

Comments
 (0)