Skip to content

Commit

Permalink
Merge pull request epiforecasts#445 from DavisVaughan/fix/reliance-on…
Browse files Browse the repository at this point in the history
…-complete-column-order

Don't rely on exact column ordering when grouped data is involved
  • Loading branch information
seabbs authored Feb 4, 2022
2 parents 35d6938 + fd1219f commit a13fd4e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/testthat/test-processing.R
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ test_that("complete_cumulative_columns works", {
input_data <- get_input_data_for_complete_cumulative_columns_test()
expected_data <- get_expected_data_for_complete_cumulative_columns_test()
actual_data <- complete_cumulative_columns(input_data)
expect_equal(colnames(actual_data), colnames(expected_data))
expect_equal(sort(colnames(actual_data)), sort(colnames(expected_data)))
expect_true(!any(is.na(actual_data$cases_total)))
})

Expand Down

0 comments on commit a13fd4e

Please sign in to comment.