Skip to content

Conversation

@thunderbiscuit
Copy link
Member

This PR fixes the test-android workflow to use an emulator and run the connected tests for bdk-android. The workflow as it stands does not actually run any tests!

Fixes #243.

@notmandatory
Copy link
Member

Is this one still WIP for release 0.27 ? I think it is breaking because our Kotlin builds need to use 21.4.X not the default 25.x.

@notmandatory notmandatory removed this from the Release 0.27.0 milestone Jan 25, 2023
@thunderbiscuit thunderbiscuit force-pushed the connected-android-tests branch from ed7fc67 to 4670f28 Compare February 1, 2023 16:11
@thunderbiscuit thunderbiscuit force-pushed the connected-android-tests branch 3 times, most recently from 3243870 to 201f598 Compare March 21, 2023 19:14
@thunderbiscuit
Copy link
Member Author

thunderbiscuit commented Mar 22, 2023

Useful links:

  1. Error: failed to run custom build command for ring v0.16.20 bbqsrc/cargo-ndk#76
  2. `aarch64-linux-android-ar` not installed termux/termux-packages#8214

The problem might be coming from the plugin, where we define the following, which might throw off the macos image:

// if ANDROID_NDK_ROOT is not set then set it to github actions default
if (System.getenv("ANDROID_NDK_ROOT") == null) {
    environment(
        Pair("ANDROID_NDK_ROOT", "${System.getenv("ANDROID_SDK_ROOT")}/ndk-bundle")
    )
}

@thunderbiscuit thunderbiscuit force-pushed the connected-android-tests branch from 109d1af to 5cbbb8c Compare March 22, 2023 02:24
@thunderbiscuit
Copy link
Member Author

See #242 for related links.

@thunderbiscuit thunderbiscuit force-pushed the connected-android-tests branch from abd8918 to dc5e706 Compare November 15, 2023 15:53
@thunderbiscuit thunderbiscuit changed the base branch from master to test-ci-android November 15, 2023 15:56
@thunderbiscuit thunderbiscuit merged commit dc5e706 into bitcoindevkit:test-ci-android Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Fix Android instrumentation tests in CI

2 participants