Skip to content

Commit

Permalink
update to more specific tests
Browse files Browse the repository at this point in the history
  • Loading branch information
js8544 committed Sep 11, 2023
1 parent 1a44fbe commit 6ec817c
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions cpp/src/arrow/compute/kernels/aggregate_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -671,8 +671,9 @@ TEST(TestNullSumKernel, Basics) {

TEST(TestSumKernel, Overflow) {
for (const auto& ty : {uint64(), int64()}) {
auto scalar = ScalarFromJSON(ty, "10000000000000000");
int64_t length = 100000;
auto scalar = ScalarFromJSON(
ty, ty->id() == Type::UINT64 ? "184467440737095517" : "92233720368547759");
int64_t length = 100;
auto array = MakeArrayFromScalar(*scalar, length).ValueOrDie();
ASSERT_RAISES(Invalid, SumChecked(array));
}
Expand Down

0 comments on commit 6ec817c

Please sign in to comment.