Skip to content

Commit 1660084

Browse files
rainmanjamclaude
andcommitted
fix: use ENABLE_FRONTEND_API/ENABLE_QT to gate e2e tests
Changed e2e test build condition from OBS::libobs to checking ENABLE_FRONTEND_API OR ENABLE_QT. CI builds with both OFF don't have full OBS headers installed. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 2559571 commit 1660084

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

tests/CMakeLists.txt

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -622,10 +622,10 @@ set_tests_properties(
622622
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
623623
)
624624

625-
# E2E Streaming Workflow Tests (requires live Restreamer server AND OBS)
625+
# E2E Streaming Workflow Tests (requires live Restreamer server AND full OBS headers)
626626
# These tests compile restreamer-api.c which needs OBS headers
627-
# Only build when OBS is available (not in CI without OBS)
628-
if(TARGET OBS::libobs)
627+
# Only build when frontend/Qt is enabled (meaning full OBS SDK is available)
628+
if(ENABLE_FRONTEND_API OR ENABLE_QT)
629629
add_executable(test_e2e_streaming_workflow e2e/test_streaming_workflow.c obs_stubs.c)
630630
target_sources(
631631
test_e2e_streaming_workflow
@@ -655,12 +655,12 @@ set_tests_properties(
655655
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
656656
TIMEOUT 300
657657
)
658-
endif() # End if(TARGET OBS::libobs) for test_e2e_streaming_workflow
658+
endif() # End if(ENABLE_FRONTEND_API OR ENABLE_QT) for test_e2e_streaming_workflow
659659

660-
# Comprehensive E2E Streaming Tests (requires live Restreamer server AND OBS)
660+
# Comprehensive E2E Streaming Tests (requires live Restreamer server AND full OBS headers)
661661
# These tests compile restreamer-api.c which needs OBS headers
662-
# Only build when OBS is available (not in CI without OBS)
663-
if(TARGET OBS::libobs)
662+
# Only build when frontend/Qt is enabled (meaning full OBS SDK is available)
663+
if(ENABLE_FRONTEND_API OR ENABLE_QT)
664664
add_executable(test_streaming_e2e e2e/test_streaming_e2e.c obs_stubs.c)
665665
target_sources(
666666
test_streaming_e2e
@@ -714,7 +714,7 @@ set_tests_properties(
714714
WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
715715
TIMEOUT 600
716716
)
717-
endif() # End if(TARGET OBS::libobs) for test_streaming_e2e
717+
endif() # End if(ENABLE_FRONTEND_API OR ENABLE_QT) for test_streaming_e2e
718718

719719
if(FALSE) # Temporarily disabled
720720
# End-to-End workflow tests

0 commit comments

Comments
 (0)