| Age | Commit message (Collapse) | Author |
|---|
|
This is a follow-up to
https://android-review.git.corp.google.com/c/platform/art/+/3080063.
Test: atest --host libnativebridge-tests
Test: atest libnativebridge-tests
Bug: 189484095
Flag: TEST_ONLY
Change-Id: I8d30747d8713e9c1d29e08758df7c7d9d68e2846
|
|
libnativebridge_tests uses test_per_src: true to avoid running
multiple tests in the same process. gtest_isolated uses separate
processes for each test, but without the various incompatibilities
caused by test_per_src. Switch to isolated: true instead.
Create a temporary directory in the constructor of the shared
NativeBridgeTest base class instead and put the code_cache directory
inside it to keep the tests from trying to modify the current
directory.
Also add the tests to TEST_MAPPING and remove the preupload check.
Bug: 189484095
Test: atest --host libnativebridge_tests
Test: atest libnativebridge_tests
Flag: TEST_ONLY
Change-Id: Iea522c1895f1f5996b10e545c44d6b00bda1ee3d
|
|
- Bitrot 1: NativeBridge2Signal_test stopped working when version 3 was
added in https://r.android.com/234059.
- Bitrot 2: The code path that PreInitializeNativeBridgeFail1_test
tested was removed completely when missing app data directory was
allowed in https://r.android.com/1208629, so delete the test.
- Use cc_test_library for the dynamic test libs so they are installed in
the test directory rather than in system/lib(64).
- Use test_per_src since native_bridge.cc doesn't support reloading the
native bridge implementation lib after unloading.
- Add a custom script to run the tests and a preupload check that the
script is run. (Configuring atest for these tests still TBD in
b/189484095 - test_per_src is a complication.)
Test: art/libnativebridge/tests/runtests.sh
Bug: 122710865
Bug: 189484095
Change-Id: Ib2b387e1d858127ca4bb44c548a5105ea8b838d8
|
|
Based on:
https://source.android.com/setup/contribute/respectful-code
Bug: 161896447
Bug: 161850439
Bug: 161336379
Test: treehugger
Change-Id: I11c2bc377bccb84c5ad587350c76972614262c23
|
|
This change moves system/core/libnative{bridge,loader} under art/.
Bug: 137364733
Test: m
Change-Id: I9be7333d00fcd3f36cd80520e50a30ea840187ad
|