From ac86d89d9f8344705e2fa5c4e04cbf5349eca850 Mon Sep 17 00:00:00 2001 From: Conor Hoekstra Date: Thu, 4 Feb 2021 02:26:50 -0500 Subject: [PATCH] Temporarily turn on test (shoud fail on 10.2) --- cpp/tests/groupby/group_count_test.cpp | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/cpp/tests/groupby/group_count_test.cpp b/cpp/tests/groupby/group_count_test.cpp index d23f26bc6e2..b2642758448 100644 --- a/cpp/tests/groupby/group_count_test.cpp +++ b/cpp/tests/groupby/group_count_test.cpp @@ -243,18 +243,17 @@ TYPED_TEST(FixedPointTestBothReps, GroupBySumProductMinMaxDecimalAsValue) // commented out until we drop support for CUDA 10.2 // group_by hash tests - // auto agg5 = cudf::make_sum_aggregation(); - // test_single_agg(keys, vals, expect_keys, expect_vals_sum, std::move(agg5)); + auto agg5 = cudf::make_sum_aggregation(); + test_single_agg(keys, vals, expect_keys, expect_vals_sum, std::move(agg5)); - // auto agg6 = cudf::make_min_aggregation(); - // test_single_agg(keys, vals, expect_keys, expect_vals_min, std::move(agg6)); + auto agg6 = cudf::make_min_aggregation(); + test_single_agg(keys, vals, expect_keys, expect_vals_min, std::move(agg6)); - // auto agg7 = cudf::make_max_aggregation(); - // test_single_agg(keys, vals, expect_keys, expect_vals_max, std::move(agg7)); + auto agg7 = cudf::make_max_aggregation(); + test_single_agg(keys, vals, expect_keys, expect_vals_max, std::move(agg7)); - // auto agg8 = cudf::make_product_aggregation(); - // EXPECT_THROW(test_single_agg(keys, vals, expect_keys, {}, std::move(agg8)), - // cudf::logic_error); + auto agg8 = cudf::make_product_aggregation(); + EXPECT_THROW(test_single_agg(keys, vals, expect_keys, {}, std::move(agg8)), cudf::logic_error); } }