Skip to content

Commit 9d220d1

Browse files
authored
Merge pull request #1304 from Geode-solutions/fix/dhi-133836
fix(Test): all tests are essential by default
2 parents a117bc1 + 35d1bd0 commit 9d220d1

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

cmake/CppTargets.cmake

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -258,13 +258,7 @@ function(add_geode_test)
258258
TIMEOUT 300
259259
FAIL_REGULAR_EXPRESSION "Sanitizer"
260260
)
261-
if(GEODE_TEST_ESSENTIAL)
262-
add_dependencies(essential ${target_name})
263-
set_tests_properties(${target_name}
264-
PROPERTIES
265-
LABELS essential
266-
)
267-
elseif(GEODE_TEST_UNSTABLE)
261+
if(GEODE_TEST_UNSTABLE)
268262
set_tests_properties(${target_name}
269263
PROPERTIES
270264
LABELS unstable
@@ -275,6 +269,12 @@ function(add_geode_test)
275269
LABELS local
276270
TIMEOUT 0
277271
)
272+
else()
273+
add_dependencies(essential ${target_name})
274+
set_tests_properties(${target_name}
275+
PROPERTIES
276+
LABELS essential
277+
)
278278
endif()
279279
if(GEODE_TEST_FIXTURES_SETUP)
280280
set_tests_properties(${target_name}

0 commit comments

Comments
 (0)