From 0d2b29d3e6bd57b909809e81bba9e7719742eb54 Mon Sep 17 00:00:00 2001 From: David Wendt <45795991+davidwendt@users.noreply.github.com> Date: Thu, 23 Jan 2025 19:02:41 -0500 Subject: [PATCH] Disable intended disabled ORC tests (#17790) The following ORC_TEST tests were intended to be disabed: ``` OrcWriterTest.DISABLE_Over65kColumns OrcReaderTest.DISABLE_Over65kStripes OrcReaderTest.DISABLE_Over65kRowGroups ``` Unfortunately they are missing the end `D` and so were not actually disabled. This caused the memcheck nightly build to fail. Changing `DISABLE_` to `DISABLED_` properly disables the tests. Authors: - David Wendt (https://github.com/davidwendt) Approvers: - Bradley Dice (https://github.com/bdice) - Nghia Truong (https://github.com/ttnghia) - Shruti Shivakumar (https://github.com/shrshi) URL: https://github.com/rapidsai/cudf/pull/17790 --- cpp/tests/io/orc_test.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/cpp/tests/io/orc_test.cpp b/cpp/tests/io/orc_test.cpp index 1ade2143494..362a6df913d 100644 --- a/cpp/tests/io/orc_test.cpp +++ b/cpp/tests/io/orc_test.cpp @@ -2198,7 +2198,7 @@ TEST_F(OrcChunkedWriterTest, NoDataInSinkWhenNoWrite) // Tests whether Y dimension of grid sizes depends on the number of row groups // Disabled because of the high execution time (especially compared to the likelihood of regression) -TEST_F(OrcReaderTest, DISABLE_Over65kRowGroups) +TEST_F(OrcReaderTest, DISABLED_Over65kRowGroups) { auto constexpr row_group_size = 512; constexpr auto num_rows = (1 << 16) * row_group_size + 1; @@ -2222,7 +2222,7 @@ TEST_F(OrcReaderTest, DISABLE_Over65kRowGroups) // Tests whether Y dimension of grid sizes depends on the number of stripes // Disabled because of the high execution time (especially compared to the likelihood of regression) -TEST_F(OrcReaderTest, DISABLE_Over65kStripes) +TEST_F(OrcReaderTest, DISABLED_Over65kStripes) { auto constexpr stripe_size = 512; constexpr auto num_rows = (1 << 16) * stripe_size + 1; @@ -2247,7 +2247,7 @@ TEST_F(OrcReaderTest, DISABLE_Over65kStripes) // Tests whether Y dimension of grid sizes depends on the number of columns // Disabled because of the high execution time (especially compared to the likelihood of regression) -TEST_F(OrcWriterTest, DISABLE_Over65kColumns) +TEST_F(OrcWriterTest, DISABLED_Over65kColumns) { auto vals_col = random_values(8); dec64_col col{vals_col.begin(), vals_col.end(), numeric::scale_type{2}};