Skip to content

Fix native host test CI target mismatch - #38

Merged
XRDOGE-XRPL merged 1 commit into
mainfrom
copilot/fix-failing-github-actions-job-another-one
Sep 18, 2026
Merged

XRDOGE-XRPL merged 1 commit into
mainfrom
copilot/fix-failing-github-actions-job-another-one

Conversation

Copilot AI commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

The Android CI job was failing during native host validation because CTest registered two executables but the workflow only built one of them. That left client_state_stress_test missing and caused the job to abort before subsequent validation steps.

  • Root cause

    • app/src/main/cpp/CMakeLists.txt registers both client_state_test and client_state_stress_test.
    • The GitHub Actions workflow invoked ctest after building only client_state_test, which produced the missing-executable failure.
  • Changes

    • Updated the CI workflow to build both native host test targets before running CTest.
    • Updated the repo’s release validation script to use the same complete native test build command so local validation matches CI behavior.
cmake -S app/src/main/cpp -B build/native-tests -DANDROIDSA_ENABLE_NATIVE_TESTS=ON
cmake --build build/native-tests --target client_state_test client_state_stress_test
ctest --test-dir build/native-tests --output-on-failure

Co-authored-by: XRDOGE-XRPL <166251416+XRDOGE-XRPL@users.noreply.github.com>
@XRDOGE-XRPL
XRDOGE-XRPL marked this pull request as ready for review September 18, 2026 08:20
@XRDOGE-XRPL
XRDOGE-XRPL merged commit c1ad09c into main Sep 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants