@@ -128,11 +128,13 @@ test_that("Metadata and grouping are dropped by `as_tibble`", {
128128test_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