Remove buildDir from cc package

Bug: 182885307
Test: m nothing
Change-Id: I964af3cb9a3e7a1dfbd9d5176a702591f6b19f16
This commit is contained in:
Paul Duffin 2021-03-22 23:21:32 +00:00
parent 2e6f90e0d0
commit c3e6ce04d9
4 changed files with 35 additions and 57 deletions

View File

@ -16,7 +16,6 @@ package cc
import ( import (
"fmt" "fmt"
"io/ioutil"
"os" "os"
"path/filepath" "path/filepath"
"reflect" "reflect"
@ -27,33 +26,12 @@ import (
"android/soong/android" "android/soong/android"
) )
var buildDir string
func setUp() {
var err error
buildDir, err = ioutil.TempDir("", "soong_cc_test")
if err != nil {
panic(err)
}
}
func tearDown() {
os.RemoveAll(buildDir)
}
func TestMain(m *testing.M) { func TestMain(m *testing.M) {
run := func() int { os.Exit(m.Run())
setUp()
defer tearDown()
return m.Run()
}
os.Exit(run())
} }
var ccFixtureFactory = android.NewFixtureFactory( var ccFixtureFactory = android.NewFixtureFactory(
&buildDir, nil,
prepareForCcTest, prepareForCcTest,
) )
@ -101,7 +79,7 @@ func testCc(t *testing.T, bp string) *android.TestContext {
// deprecated // deprecated
func testCcNoVndk(t *testing.T, bp string) *android.TestContext { func testCcNoVndk(t *testing.T, bp string) *android.TestContext {
t.Helper() t.Helper()
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
return testCcWithConfig(t, config) return testCcWithConfig(t, config)
@ -114,7 +92,7 @@ func testCcNoVndk(t *testing.T, bp string) *android.TestContext {
// deprecated // deprecated
func testCcNoProductVndk(t *testing.T, bp string) *android.TestContext { func testCcNoProductVndk(t *testing.T, bp string) *android.TestContext {
t.Helper() t.Helper()
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
@ -141,7 +119,7 @@ func testCcErrorWithConfig(t *testing.T, pattern string, config android.Config)
// deprecated // deprecated
func testCcError(t *testing.T, pattern string, bp string) { func testCcError(t *testing.T, pattern string, bp string) {
t.Helper() t.Helper()
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
testCcErrorWithConfig(t, pattern, config) testCcErrorWithConfig(t, pattern, config)
@ -155,7 +133,7 @@ func testCcError(t *testing.T, pattern string, bp string) {
// deprecated // deprecated
func testCcErrorProductVndk(t *testing.T, pattern string, bp string) { func testCcErrorProductVndk(t *testing.T, pattern string, bp string) {
t.Helper() t.Helper()
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.ProductVndkVersion = StringPtr("current") config.TestProductVariables.ProductVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
@ -474,7 +452,7 @@ func TestVndk(t *testing.T) {
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.ProductVndkVersion = StringPtr("current") config.TestProductVariables.ProductVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
@ -495,7 +473,7 @@ func TestVndk(t *testing.T) {
// Check VNDK snapshot output. // Check VNDK snapshot output.
snapshotDir := "vndk-snapshot" snapshotDir := "vndk-snapshot"
snapshotVariantPath := filepath.Join(buildDir, snapshotDir, "arm64") snapshotVariantPath := filepath.Join("out/soong", snapshotDir, "arm64")
vndkLibPath := filepath.Join(snapshotVariantPath, fmt.Sprintf("arch-%s-%s", vndkLibPath := filepath.Join(snapshotVariantPath, fmt.Sprintf("arch-%s-%s",
"arm64", "armv8-a")) "arm64", "armv8-a"))
@ -596,7 +574,7 @@ func TestVndkLibrariesTxtAndroidMk(t *testing.T) {
name: "llndk.libraries.txt", name: "llndk.libraries.txt",
insert_vndk_version: true, insert_vndk_version: true,
}` }`
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
ctx := testCcWithConfig(t, config) ctx := testCcWithConfig(t, config)
@ -646,7 +624,7 @@ func TestVndkUsingCoreVariant(t *testing.T) {
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
config.TestProductVariables.VndkUseCoreVariant = BoolPtr(true) config.TestProductVariables.VndkUseCoreVariant = BoolPtr(true)
@ -673,7 +651,7 @@ func TestDataLibs(t *testing.T) {
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
config.TestProductVariables.VndkUseCoreVariant = BoolPtr(true) config.TestProductVariables.VndkUseCoreVariant = BoolPtr(true)
@ -724,7 +702,7 @@ func TestDataLibsRelativeInstallPath(t *testing.T) {
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
config.TestProductVariables.VndkUseCoreVariant = BoolPtr(true) config.TestProductVariables.VndkUseCoreVariant = BoolPtr(true)
@ -1349,7 +1327,7 @@ func TestVndkExt(t *testing.T) {
nocrt: true, nocrt: true,
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.ProductVndkVersion = StringPtr("current") config.TestProductVariables.ProductVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
@ -1794,7 +1772,7 @@ func TestProductVndkExtDependency(t *testing.T) {
nocrt: true, nocrt: true,
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.ProductVndkVersion = StringPtr("current") config.TestProductVariables.ProductVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
@ -2349,7 +2327,7 @@ func TestMakeLinkType(t *testing.T) {
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
// native:vndk // native:vndk
@ -3141,7 +3119,7 @@ func TestDataLibsPrebuiltSharedTestLibrary(t *testing.T) {
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
config.TestProductVariables.VndkUseCoreVariant = BoolPtr(true) config.TestProductVariables.VndkUseCoreVariant = BoolPtr(true)
@ -3536,7 +3514,7 @@ func TestInstallSharedLibs(t *testing.T) {
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
ctx := testCcWithConfig(t, config) ctx := testCcWithConfig(t, config)
hostBin := ctx.ModuleForTests("bin", config.BuildOSTarget.String()).Description("install") hostBin := ctx.ModuleForTests("bin", config.BuildOSTarget.String()).Description("install")

View File

@ -52,7 +52,7 @@ func TestArchGenruleCmd(t *testing.T) {
}, },
} }
` `
config := android.TestArchConfig(buildDir, nil, bp, fs) config := android.TestArchConfig(t.TempDir(), nil, bp, fs)
ctx := testGenruleContext(config) ctx := testGenruleContext(config)

View File

@ -199,7 +199,7 @@ func TestStubsVersions(t *testing.T) {
}, },
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.Platform_version_active_codenames = []string{"R"} config.TestProductVariables.Platform_version_active_codenames = []string{"R"}
ctx := testCcWithConfig(t, config) ctx := testCcWithConfig(t, config)
@ -222,7 +222,7 @@ func TestStubsVersions_NotSorted(t *testing.T) {
}, },
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.Platform_version_active_codenames = []string{"R"} config.TestProductVariables.Platform_version_active_codenames = []string{"R"}
testCcErrorWithConfig(t, `"libfoo" .*: versions: not sorted`, config) testCcErrorWithConfig(t, `"libfoo" .*: versions: not sorted`, config)
} }

View File

@ -86,7 +86,7 @@ func TestVendorSnapshotCapture(t *testing.T) {
symbol_file: "", symbol_file: "",
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
ctx := testCcWithConfig(t, config) ctx := testCcWithConfig(t, config)
@ -94,7 +94,7 @@ func TestVendorSnapshotCapture(t *testing.T) {
// Check Vendor snapshot output. // Check Vendor snapshot output.
snapshotDir := "vendor-snapshot" snapshotDir := "vendor-snapshot"
snapshotVariantPath := filepath.Join(buildDir, snapshotDir, "arm64") snapshotVariantPath := filepath.Join("out/soong", snapshotDir, "arm64")
snapshotSingleton := ctx.SingletonForTests("vendor-snapshot") snapshotSingleton := ctx.SingletonForTests("vendor-snapshot")
var jsonFiles []string var jsonFiles []string
@ -212,7 +212,7 @@ func TestVendorSnapshotDirected(t *testing.T) {
nocrt: true, nocrt: true,
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
config.TestProductVariables.DirectedVendorSnapshot = true config.TestProductVariables.DirectedVendorSnapshot = true
@ -224,7 +224,7 @@ func TestVendorSnapshotDirected(t *testing.T) {
// Check Vendor snapshot output. // Check Vendor snapshot output.
snapshotDir := "vendor-snapshot" snapshotDir := "vendor-snapshot"
snapshotVariantPath := filepath.Join(buildDir, snapshotDir, "arm64") snapshotVariantPath := filepath.Join("out/soong", snapshotDir, "arm64")
snapshotSingleton := ctx.SingletonForTests("vendor-snapshot") snapshotSingleton := ctx.SingletonForTests("vendor-snapshot")
var includeJsonFiles []string var includeJsonFiles []string
@ -516,7 +516,7 @@ func TestVendorSnapshotUse(t *testing.T) {
"vndk/libvndk.so": nil, "vndk/libvndk.so": nil,
} }
config := TestConfig(buildDir, android.Android, nil, "", mockFS) config := TestConfig(t.TempDir(), android.Android, nil, "", mockFS)
config.TestProductVariables.DeviceVndkVersion = StringPtr("BOARD") config.TestProductVariables.DeviceVndkVersion = StringPtr("BOARD")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
ctx := CreateTestContext(config) ctx := CreateTestContext(config)
@ -628,7 +628,7 @@ func TestVendorSnapshotSanitizer(t *testing.T) {
}, },
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("BOARD") config.TestProductVariables.DeviceVndkVersion = StringPtr("BOARD")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
ctx := testCcWithConfig(t, config) ctx := testCcWithConfig(t, config)
@ -707,7 +707,7 @@ func TestVendorSnapshotExclude(t *testing.T) {
"device/vendor.cpp": nil, "device/vendor.cpp": nil,
} }
config := TestConfig(buildDir, android.Android, nil, "", mockFS) config := TestConfig(t.TempDir(), android.Android, nil, "", mockFS)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
ctx := CreateTestContext(config) ctx := CreateTestContext(config)
@ -730,7 +730,7 @@ func TestVendorSnapshotExclude(t *testing.T) {
// Verify the content of the vendor snapshot. // Verify the content of the vendor snapshot.
snapshotDir := "vendor-snapshot" snapshotDir := "vendor-snapshot"
snapshotVariantPath := filepath.Join(buildDir, snapshotDir, "arm64") snapshotVariantPath := filepath.Join("out/soong", snapshotDir, "arm64")
snapshotSingleton := ctx.SingletonForTests("vendor-snapshot") snapshotSingleton := ctx.SingletonForTests("vendor-snapshot")
var includeJsonFiles []string var includeJsonFiles []string
@ -799,7 +799,7 @@ func TestVendorSnapshotExcludeInVendorProprietaryPathErrors(t *testing.T) {
"device/vendor.cpp": nil, "device/vendor.cpp": nil,
} }
config := TestConfig(buildDir, android.Android, nil, "", mockFS) config := TestConfig(t.TempDir(), android.Android, nil, "", mockFS)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
ctx := CreateTestContext(config) ctx := CreateTestContext(config)
@ -873,7 +873,7 @@ func TestRecoverySnapshotCapture(t *testing.T) {
recovery_available: true, recovery_available: true,
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.RecoverySnapshotVersion = StringPtr("current") config.TestProductVariables.RecoverySnapshotVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
ctx := testCcWithConfig(t, config) ctx := testCcWithConfig(t, config)
@ -881,7 +881,7 @@ func TestRecoverySnapshotCapture(t *testing.T) {
// Check Recovery snapshot output. // Check Recovery snapshot output.
snapshotDir := "recovery-snapshot" snapshotDir := "recovery-snapshot"
snapshotVariantPath := filepath.Join(buildDir, snapshotDir, "arm64") snapshotVariantPath := filepath.Join("out/soong", snapshotDir, "arm64")
snapshotSingleton := ctx.SingletonForTests("recovery-snapshot") snapshotSingleton := ctx.SingletonForTests("recovery-snapshot")
var jsonFiles []string var jsonFiles []string
@ -991,7 +991,7 @@ func TestRecoverySnapshotExclude(t *testing.T) {
"device/recovery.cpp": nil, "device/recovery.cpp": nil,
} }
config := TestConfig(buildDir, android.Android, nil, "", mockFS) config := TestConfig(t.TempDir(), android.Android, nil, "", mockFS)
config.TestProductVariables.RecoverySnapshotVersion = StringPtr("current") config.TestProductVariables.RecoverySnapshotVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
ctx := CreateTestContext(config) ctx := CreateTestContext(config)
@ -1014,7 +1014,7 @@ func TestRecoverySnapshotExclude(t *testing.T) {
// Verify the content of the recovery snapshot. // Verify the content of the recovery snapshot.
snapshotDir := "recovery-snapshot" snapshotDir := "recovery-snapshot"
snapshotVariantPath := filepath.Join(buildDir, snapshotDir, "arm64") snapshotVariantPath := filepath.Join("out/soong", snapshotDir, "arm64")
snapshotSingleton := ctx.SingletonForTests("recovery-snapshot") snapshotSingleton := ctx.SingletonForTests("recovery-snapshot")
var includeJsonFiles []string var includeJsonFiles []string
@ -1091,7 +1091,7 @@ func TestRecoverySnapshotDirected(t *testing.T) {
nocrt: true, nocrt: true,
} }
` `
config := TestConfig(buildDir, android.Android, nil, bp, nil) config := TestConfig(t.TempDir(), android.Android, nil, bp, nil)
config.TestProductVariables.DeviceVndkVersion = StringPtr("current") config.TestProductVariables.DeviceVndkVersion = StringPtr("current")
config.TestProductVariables.RecoverySnapshotVersion = StringPtr("current") config.TestProductVariables.RecoverySnapshotVersion = StringPtr("current")
config.TestProductVariables.Platform_vndk_version = StringPtr("VER") config.TestProductVariables.Platform_vndk_version = StringPtr("VER")
@ -1104,7 +1104,7 @@ func TestRecoverySnapshotDirected(t *testing.T) {
// Check recovery snapshot output. // Check recovery snapshot output.
snapshotDir := "recovery-snapshot" snapshotDir := "recovery-snapshot"
snapshotVariantPath := filepath.Join(buildDir, snapshotDir, "arm64") snapshotVariantPath := filepath.Join("out/soong", snapshotDir, "arm64")
snapshotSingleton := ctx.SingletonForTests("recovery-snapshot") snapshotSingleton := ctx.SingletonForTests("recovery-snapshot")
var includeJsonFiles []string var includeJsonFiles []string