From 5f23bc98ed172c3d5a1d9de4eb35b9f703c99c25 Mon Sep 17 00:00:00 2001 From: Nikita Putikhin Date: Fri, 5 Jul 2024 15:08:01 +0200 Subject: Switch rust test install base to /data/local/tmp for consistency with cc Previously this code was not doing anything because the template used for rust did not use the template string that gets replaced with the folder coming from this code. So all tests were getting deployed into /data/local/tmp regardless of being vendor tests. The other change in the chain adds the template string, so this code starts working and system and vendor tests start being deployed into different folders. We want to keep the system test folder as before to avoid potential problems with tests (see b/350479879#comment9). Bug: 347370677 Test: presubmit Change-Id: I93d5a8db3470d1603e26f9cf5d12249f29141a19 --- rust/test.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'rust/test.go') diff --git a/rust/test.go b/rust/test.go index 3087d8d94..b7ddd06a3 100644 --- a/rust/test.go +++ b/rust/test.go @@ -116,7 +116,8 @@ func (test *testDecorator) compilerProps() []interface{} { } func (test *testDecorator) install(ctx ModuleContext) { - testInstallBase := "/data/local/tests/unrestricted" + // TODO: (b/167308193) Switch to /data/local/tests/unrestricted as the default install base. + testInstallBase := "/data/local/tmp" if ctx.RustModule().InVendorOrProduct() { testInstallBase = "/data/local/tests/vendor" } -- cgit v1.2.3-59-g8ed1b