Skip to content

Commit

Permalink
Merge pull request #3 from mattdowle/master
Browse files Browse the repository at this point in the history
tolerance: expect_identical => expect_equal
  • Loading branch information
gdemin authored Jan 2, 2019
2 parents 67e16d9 + 3fa9a82 commit bb1e232
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions tests/testthat/test_verbs.R
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,8 @@ mt_dt = as.data.table(mtcars)
res = dt_summarize(mtcars, (new_var1) := eval(expr1), (new_var2) := eval(expr2), by = am)
res2 = dt_summarize(mt_dt, (new_var1) := eval(expr1), (new_var2) := eval(expr2), by = am)
res3 = mt_dt[, list(agg = mean(mpg), agg2 = mean(hp)), by = am]
expect_identical(res3, res)
expect_identical(res3, res2)
expect_equal(res3, res)
expect_equal(res3, res2)



Expand Down
4 changes: 2 additions & 2 deletions tests/testthat/tests.R
Original file line number Diff line number Diff line change
Expand Up @@ -166,5 +166,5 @@ new_var2 = "agg2"
res = take(mtcars, (new_var) := eval(expr1), (new_var2) := eval(expr2), by = am)
res2 = take(mt_dt, (new_var) := eval(expr1), (new_var2) := eval(expr2), by = am)
res3 = mt_dt[, list(agg = mean(mpg), agg2 = mean(hp)), by = am]
expect_identical(res3, res)
expect_identical(res3, res2)
expect_equal(res3, res)
expect_equal(res3, res2)

0 comments on commit bb1e232

Please sign in to comment.