Skip to content

Commit 536011b

Browse files
committed
test(ci): Add intentional test failure to validate CI failure messaging
Temporary commit — adds ASSERT_EQ(1, 0) to ValuesTest.empty to trigger the CI failure reporting flow. Will be reverted after validation.
1 parent 1c133af commit 536011b

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

velox/exec/tests/CMakeLists.txt

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -451,10 +451,4 @@ add_test(
451451

452452
target_link_libraries(velox_driver_test velox_exec velox_exec_test_lib GTest::gtest)
453453

454-
# TODO: Remove after validating CI failure messaging (PR #17015)
455-
add_test(
456-
NAME dummy_fail_test
457-
COMMAND bash -c "echo 'Dummy test to validate CI failure reporting' && exit 1"
458-
)
459-
460454
velox_add_library(velox_aggregate_registry_util INTERFACE HEADERS AggregateRegistryTestUtil.h)

velox/exec/tests/ValuesTest.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,9 @@ class ValuesTest : public OperatorTestBase {
4646
};
4747

4848
TEST_F(ValuesTest, empty) {
49+
// TODO: Remove after validating CI failure reporting (PR #17015)
50+
ASSERT_EQ(1, 0) << "Intentional failure to validate CI failure messaging";
51+
4952
// Base case: no vectors.
5053
AssertQueryBuilder(
5154
PlanBuilder().values(std::vector<RowVectorPtr>{}).planNode())

0 commit comments

Comments
 (0)