diff options
author | Colin Cross <ccross@android.com> | 2019-01-09 22:17:55 -0800 |
---|---|---|
committer | Colin Cross <ccross@android.com> | 2019-01-10 07:09:29 +0000 |
commit | fb6d78120233e5aff492fa827c27ba87354fc9e8 (patch) | |
tree | c6164fe2c9a88320aee84a78a9967ba4d0a99f0e /java/java_test.go | |
parent | c33514e3b68bf36ddca57624edbaed8005314c0c (diff) |
Split out sdk.go from java.go
Split out SDK handling functions from java.go to sdk.go and tests
from java_test.go to sdk.go.
Test: sdk_test.go
Change-Id: I83ef48cbe5230572c1d4ecc0e89021d2f7c71b76
Diffstat (limited to 'java/java_test.go')
-rw-r--r-- | java/java_test.go | 238 |
1 files changed, 0 insertions, 238 deletions
diff --git a/java/java_test.go b/java/java_test.go index c4cf9059d..889cae762 100644 --- a/java/java_test.go +++ b/java/java_test.go @@ -19,13 +19,10 @@ import ( "io/ioutil" "os" "path/filepath" - "reflect" "strconv" "strings" "testing" - "github.com/google/blueprint/proptools" - "android/soong/android" "android/soong/cc" "android/soong/genrule" @@ -354,241 +351,6 @@ func TestBinary(t *testing.T) { } -var classpathTestcases = []struct { - name string - unbundled bool - moduleType string - host android.OsClass - properties string - bootclasspath []string - system string - classpath []string -}{ - { - name: "default", - bootclasspath: []string{"core.platform.api.stubs", "core-lambda-stubs"}, - system: "core-platform-api-stubs-system-modules", - classpath: []string{"ext", "framework"}, - }, - { - name: "blank sdk version", - properties: `sdk_version: "",`, - bootclasspath: []string{"core.platform.api.stubs", "core-lambda-stubs"}, - system: "core-platform-api-stubs-system-modules", - classpath: []string{"ext", "framework"}, - }, - { - - name: "sdk v14", - properties: `sdk_version: "14",`, - bootclasspath: []string{`""`}, - system: "bootclasspath", // special value to tell 1.9 test to expect bootclasspath - classpath: []string{"prebuilts/sdk/14/public/android.jar", "prebuilts/sdk/tools/core-lambda-stubs.jar"}, - }, - { - - name: "current", - properties: `sdk_version: "current",`, - bootclasspath: []string{"android_stubs_current", "core-lambda-stubs"}, - system: "bootclasspath", // special value to tell 1.9 test to expect bootclasspath - }, - { - - name: "system_current", - properties: `sdk_version: "system_current",`, - bootclasspath: []string{"android_system_stubs_current", "core-lambda-stubs"}, - system: "bootclasspath", // special value to tell 1.9 test to expect bootclasspath - }, - { - - name: "system_14", - properties: `sdk_version: "system_14",`, - bootclasspath: []string{`""`}, - system: "bootclasspath", // special value to tell 1.9 test to expect bootclasspath - classpath: []string{"prebuilts/sdk/14/system/android.jar", "prebuilts/sdk/tools/core-lambda-stubs.jar"}, - }, - { - - name: "test_current", - properties: `sdk_version: "test_current",`, - bootclasspath: []string{"android_test_stubs_current", "core-lambda-stubs"}, - system: "bootclasspath", // special value to tell 1.9 test to expect bootclasspath - }, - { - - name: "core_current", - properties: `sdk_version: "core_current",`, - bootclasspath: []string{"core.current.stubs", "core-lambda-stubs"}, - system: "bootclasspath", // special value to tell 1.9 test to expect bootclasspath - }, - { - - name: "nostdlib", - properties: `no_standard_libs: true, system_modules: "none"`, - system: "none", - bootclasspath: []string{`""`}, - classpath: []string{}, - }, - { - - name: "nostdlib system_modules", - properties: `no_standard_libs: true, system_modules: "core-platform-api-stubs-system-modules"`, - system: "core-platform-api-stubs-system-modules", - bootclasspath: []string{`""`}, - classpath: []string{}, - }, - { - - name: "host default", - moduleType: "java_library_host", - properties: ``, - host: android.Host, - bootclasspath: []string{"jdk8/jre/lib/jce.jar", "jdk8/jre/lib/rt.jar"}, - classpath: []string{}, - }, - { - name: "host nostdlib", - moduleType: "java_library_host", - host: android.Host, - properties: `no_standard_libs: true`, - classpath: []string{}, - }, - { - - name: "host supported default", - host: android.Host, - properties: `host_supported: true,`, - classpath: []string{}, - bootclasspath: []string{"jdk8/jre/lib/jce.jar", "jdk8/jre/lib/rt.jar"}, - }, - { - name: "host supported nostdlib", - host: android.Host, - properties: `host_supported: true, no_standard_libs: true, system_modules: "none"`, - classpath: []string{}, - }, - { - - name: "unbundled sdk v14", - unbundled: true, - properties: `sdk_version: "14",`, - bootclasspath: []string{`""`}, - system: "bootclasspath", // special value to tell 1.9 test to expect bootclasspath - classpath: []string{"prebuilts/sdk/14/public/android.jar", "prebuilts/sdk/tools/core-lambda-stubs.jar"}, - }, - { - - name: "unbundled current", - unbundled: true, - properties: `sdk_version: "current",`, - bootclasspath: []string{`""`}, - system: "bootclasspath", // special value to tell 1.9 test to expect bootclasspath - classpath: []string{"prebuilts/sdk/current/public/android.jar", "prebuilts/sdk/tools/core-lambda-stubs.jar"}, - }, -} - -func TestClasspath(t *testing.T) { - for _, testcase := range classpathTestcases { - t.Run(testcase.name, func(t *testing.T) { - moduleType := "java_library" - if testcase.moduleType != "" { - moduleType = testcase.moduleType - } - - bp := moduleType + ` { - name: "foo", - srcs: ["a.java"], - ` + testcase.properties + ` - }` - - variant := "android_common" - if testcase.host == android.Host { - variant = android.BuildOs.String() + "_common" - } - - convertModulesToPaths := func(cp []string) []string { - ret := make([]string, len(cp)) - for i, e := range cp { - ret[i] = moduleToPath(e) - } - return ret - } - - bootclasspath := convertModulesToPaths(testcase.bootclasspath) - classpath := convertModulesToPaths(testcase.classpath) - - bc := strings.Join(bootclasspath, ":") - if bc != "" { - bc = "-bootclasspath " + bc - } - - c := strings.Join(classpath, ":") - if c != "" { - c = "-classpath " + c - } - system := "" - if testcase.system == "none" { - system = "--system=none" - } else if testcase.system != "" { - system = "--system=" + filepath.Join(buildDir, ".intermediates", testcase.system, "android_common", "system") + "/" - } - - t.Run("1.8", func(t *testing.T) { - // Test default javac 1.8 - config := testConfig(nil) - if testcase.unbundled { - config.TestProductVariables.Unbundled_build = proptools.BoolPtr(true) - } - ctx := testContext(config, bp, nil) - run(t, ctx, config) - - javac := ctx.ModuleForTests("foo", variant).Rule("javac") - - got := javac.Args["bootClasspath"] - if got != bc { - t.Errorf("bootclasspath expected %q != got %q", bc, got) - } - - got = javac.Args["classpath"] - if got != c { - t.Errorf("classpath expected %q != got %q", c, got) - } - - var deps []string - if len(bootclasspath) > 0 && bootclasspath[0] != `""` { - deps = append(deps, bootclasspath...) - } - deps = append(deps, classpath...) - - if !reflect.DeepEqual(javac.Implicits.Strings(), deps) { - t.Errorf("implicits expected %q != got %q", deps, javac.Implicits.Strings()) - } - }) - - // Test again with javac 1.9 - t.Run("1.9", func(t *testing.T) { - config := testConfig(map[string]string{"EXPERIMENTAL_USE_OPENJDK9": "true"}) - if testcase.unbundled { - config.TestProductVariables.Unbundled_build = proptools.BoolPtr(true) - } - ctx := testContext(config, bp, nil) - run(t, ctx, config) - - javac := ctx.ModuleForTests("foo", variant).Rule("javac") - got := javac.Args["bootClasspath"] - expected := system - if testcase.system == "bootclasspath" { - expected = bc - } - if got != expected { - t.Errorf("bootclasspath expected %q != got %q", expected, got) - } - }) - }) - } - -} - func TestPrebuilts(t *testing.T) { ctx := testJava(t, ` java_library { |