Skip to content

Commit ef1af63

Browse files
authored
GH-47584: [C++][CI] Remove "large memory" mark from TestListArray::TestOverflowCheck (#47585)
### Rationale for this change As found out in #46600 (comment) : the one test that is marked "large memory" in array_list_test.cc actually shows a very small memory consumption. ### What changes are included in this PR? Make test unconditional. ### Are these changes tested? Yes, by construction. ### Are there any user-facing changes? No. * GitHub Issue: #47584 Authored-by: Antoine Pitrou <[email protected]> Signed-off-by: Antoine Pitrou <[email protected]>
1 parent e1c6fd5 commit ef1af63

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

cpp/src/arrow/array/array_list_test.cc

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -982,11 +982,7 @@ TYPED_TEST(TestListArray, ValidateDimensions) { this->TestValidateDimensions();
982982

983983
TYPED_TEST(TestListArray, CornerCases) { this->TestCornerCases(); }
984984

985-
#ifndef ARROW_LARGE_MEMORY_TESTS
986-
TYPED_TEST(TestListArray, DISABLED_TestOverflowCheck) { this->TestOverflowCheck(); }
987-
#else
988985
TYPED_TEST(TestListArray, TestOverflowCheck) { this->TestOverflowCheck(); }
989-
#endif
990986

991987
class TestListConversions : public ::testing::Test {
992988
private:

0 commit comments

Comments
 (0)