aboutsummaryrefslogtreecommitdiff
path: root/java/java_test.go
Commit message (Collapse)AuthorAgeFilesLines
* Propagate the dex jar path as an OptionalPath which is either valid orMartin Stjernholm2021-09-231-3/+3
| | | | | | | | | | | | | | | | | | | | | | invalid with a message. This will allow propagating any error from the deapexer module for prebuilt APEXes to the location where the dex jars get used. It's only at those points that we can raise errors about not being able to extract files from the deapexer modules if they are invalid, and this way we avoid encoding knowledge there about why they may be invalid. To keep the refactoring limited it intentionally does not change any of the existing logic for when dexJarFiles are set or not (non-nil vs nil prior to this change), although there may be opportunity to use this for more conditions when dex jars aren't available. The refactoring is also not extended to dexpreopt.ClassLoaderContextMap. Test: m nothing Bug: 192006406 Change-Id: I68986dccd9a9b3fee4d24caa1947ea17a36caedc
* Rename BuildDir and NinjaBuildDir.Lukacs T. Berki2021-08-271-1/+1
| | | | | | | | | | These are just out/ and out/soong/ and the old names were quite confusing. Test: Presubmits. Merged-In: I999ca62162062f27e449f83ffb00fa580d4065b8 Merged-In: I2fcd80a9e376a6842a06aa104a9f6f5c5e16f8c7 Change-Id: Ib481d2aac40df8da1b74174dc8ec576c1cb48be0
* Replace android.BuildOs with Config.BuildOSColin Cross2021-07-201-5/+5
| | | | | | | | | Replace the android.BuildOs constant with Config.BuildOS so that it can vary based on the product config. Bug: 190084016 Test: all Soong tests Change-Id: Ia67f872d8b2ab788747a22e3a9659dc21c9775cd
* Allow disabling errorprone even when RUN_ERROR_PRONE is trueCole Faust2021-06-181-1/+65
| | | | | | | | | | | | Some modules use -XepDisableAllChecks to disable errorprone. However, this still causes them to be compiled twice when RUN_ERROR_PRONE is true. Allow the new enabled property to be set to false to disable errorprone entirely, so that those modules are only compiled once. Bug: 190944875 Test: New unit tests Change-Id: Ie68695db762fffcaf11bbbcb0509c4fcab73f5c5
* Add a build flag to always enable errorprone per-targetCole Faust2021-06-141-0/+28
| | | | | | | | | | | | | | | Currently, errorprone is only run if the RUN_ERROR_PRONE enviornment variable is true. Add a flag that individual modules can use to always enable errorprone. In a followup cl, I plan to add another flag that will force all errorprone checks to be errors, so that modules can be confident that they're not ignoring any errorprone checks. Bug: 190944875 Test: New unit test and manually Change-Id: Iab0c81642ed22a736add054147829e91a891d179
* Move java_sdk_library tests to sdk_library_test.goColin Cross2021-06-011-816/+0
| | | | | | | | Seperate the java_sdk_library tests into their own file. Bug: 186723288 Test: go test ./java/... Change-Id: I899c2946cb2234dc595a4281e64bbb239b89bda8
* Merge changes Ia74a2b83,I30a46c8f,Iac7c0149Jaewoong Jung2021-04-221-101/+0
|\ | | | | | | | | | | | | * changes: Add lint.strict_updatability_linting Move Java lint tests to lint_test.go Forbid bypassing updatability lint checks.
| * Move Java lint tests to lint_test.goJaewoong Jung2021-04-211-101/+0
| | | | | | | | | | | | Test: lint_test.go Bug: 182349282 Change-Id: I30a46c8f704e66cd04541c78d3f22a140d3284ef
* | Add new stub_only_static_libs attr for sdk_libraryAnton Hansson2021-04-221-20/+40
|/ | | | | | | | | | | | | | Allow java_sdk_libraries to include libraries statically into their stubs. The immediate use-case of this is to embed libcore notice files into their stubs. Also extend the java_sdk_library tests for impl/stub-only-libs, plus some not assert utils. Bug: 173186484 Bug: 184839599 Test: soong tests Change-Id: I1ebf2f35c048eab5cec5125482a0304fe660f188
* Reland: Add jni_libs property to java testsColin Cross2021-04-191-0/+33
| | | | | | | | | | | | Add jni_libs property to java tests and treat it as test data that should be copied to the lib or lib64 directory in the test directory. This relands I3a118b933ab30dcd731c6dc2708da9bc63ab5520 with fixes for the test on mac. Fixes: 176593487 Test: java_test.go Change-Id: I2f8c46643cff7a7ae6dc4d4dbad58f0396e45f09
* Merge "Revert "Add jni_libs property to java tests""Colin Cross2021-04-171-26/+0
|\
| * Revert "Add jni_libs property to java tests"Colin Cross2021-04-171-26/+0
| | | | | | | | | | | | | | | | | | This reverts commit 246164a055a529e081a7690cc9043b496757a30c. Reason for revert: broke the mac build Bug: 176593487 Change-Id: Id0242e37aa40d84d291bf7b8ee46dfe98c4d5905
* | Merge changes from topic "tvts-test-host-jni"Treehugger Robot2021-04-171-0/+26
|\| | | | | | | | | | | * changes: Add jni_libs property to java tests Add Target to cc.SharedLibraryInfo
| * Add jni_libs property to java testsColin Cross2021-04-161-0/+26
| | | | | | | | | | | | | | | | | | Add jni_libs property to java tests and treat it as test data that should be copied to the lib or lib64 directory in the test directory. Fixes: 176593487 Test: java_test.go Change-Id: I3a118b933ab30dcd731c6dc2708da9bc63ab5520
* | Introduce NewApi lint checksPedro Loureiro2021-04-131-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are enabling NewApi lint check where the min sdk != compile sdk. At the same time, we are introducing baseline files for existing projects that fail this check in order to keep the build running. At the very least we stop introducing new problems and teams might realise of risks in their projects they were not aware of. Bug: 150847901 Test: m lint-check Change-Id: Icfa5eb98cc6b6708149f0c52fac8fc1440d9c3b0 Merged-In: Icfa5eb98cc6b6708149f0c52fac8fc1440d9c3b0
* | Merge "Remove unused java testing methods"Paul Duffin2021-03-311-17/+0
|\ \
| * | Remove unused java testing methodsPaul Duffin2021-03-311-17/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | Also, stops exporting methods that are no longer used outside the java package. Bug: 181070625 Test: m nothing Change-Id: I23d35bbc21f82f2dae802aa53badda4c58b41024
* | | Remove varargs from RunTest(t *testing.T)Paul Duffin2021-03-311-1/+6
|/ / | | | | | | | | | | | | | | Use GroupFixturePreparers instead. Bug: 182885307 Test: m nothing Change-Id: Iaedb0ddc9d6a704f4d41363e705f3025a1291dc8
* | Remove extraneous calls to TestingBuildParams.RelativeToTop()Paul Duffin2021-03-301-8/+8
| | | | | | | | | | | | | | | | Deprecated the method to try and prevent any other uses being added. Bug: 183650682 Test: m nothing Change-Id: Ia6f43851e5a00c9d96af780e3bd21e03175e1a2f
* | Automatically call TestingBuildParams.RelativeToTop()Paul Duffin2021-03-301-2/+2
| | | | | | | | | | | | | | | | | | Fixes the few tests that break due to this and which cannot easily be separated into their own changes. Bug: 183650682 Test: m nothing Change-Id: Ia2f31213a1f114a78e66a81d89279ecde9f4c465
* | Split droidstubs out of droiddoc.goColin Cross2021-03-251-240/+0
| | | | | | | | | | | | | | | | Split part of droiddoc.go into droidstubs.go. Also split droiddoc_test.go and droidstubs_test.go out of java_test.go. Test: go test ./java Change-Id: Iea742e75b6925b135016f7bbf3a168c696a6c433
* | Support multiple rsp files in RuleBuilderColin Cross2021-03-251-1/+1
| | | | | | | | | | | | | | | | | | | | The lint rule is manually creating a second rsp file because Ninja only supports on per rule. Move the support into RuleBuilder so that it can apply the same rewrites that it does to the primary one. Test: TestRuleBuilder_Build Change-Id: Iec250a2d60e74ccf1b4ad085a960fec6867ea559
* | Fix TestDroiddoc's reliance on absolute pathsPaul Duffin2021-03-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | TestDroiddoc compares the value returned by OutputFiles(""), which will usually be absolute paths including the temporary buildDir, against paths returned from TestingBuildParam.RelativeToTop(), which does not currently change the Path contents and so will include absolute temporary paths. However, a follow up change to this will make the TestingBuildParam.RelativeToTop() also change the Path contents at which point this test would be comparing relative to absolute paths. So, this change makes sure that they are all converted to relative to top paths before comparison. Bug: 183650682 Test: m droid Change-Id: Ia4478f527af27a920945f5849525e5031cc5b8b6
* | Convert test that disallows non existent paths to use fixturesPaul Duffin2021-03-241-27/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change needed to add some additional files to the registered files for PrepareForTestWithJavaDefaultModules because otherwise they would fail when "TestAllowNonExistentPaths = false". Those files were being added by the TestJavaLintRequiresCustomLintFileToExist (albeit in some cases in different locations to that required by the default modules but as the files are needed by the modules defined in PrepareForTestWithJavaDefaultModules they should be defined in it. A couple of other places also provided some files so moving them into PrepareForTestWithJavaDefaultModules caused some conflicts which needed to be resolved. Bug: 183184375 Test: m nothing Change-Id: I76ce9f1673c1c1c4000635b76b8377d582224bf1
* | Fix "filename too long" issue on Luci caused by test namePaul Duffin2021-03-241-1/+1
| | | | | | | | | | | | | | | | | | | | Stop including fields in the test name for TestJavaSdkLibraryEnforce to reduce its length to avoid the filename too long limit. Test: m nothing Check the lengths before (~240) and after (~74) to make sure there was a sizeable reduction. Change-Id: I275a1110e5102b8ea8376759f28c7c6333a5efee
* | Group all the preparations needed for testing dexpreoptPaul Duffin2021-03-241-2/+2
| | | | | | | | | | | | | | | | | | Make it easier to test dexpreopt functionality by grouping all the fixture preparations together. Bug: 177892522 Test: m nothing Change-Id: I94f66e3ec82efc4fd791f4fdab678d298565e452
* | Remove javaFixtureFactoryPaul Duffin2021-03-241-44/+49
| | | | | | | | | | | | | | | | | | Replaces uses of javaFixtureFactory with prepareForJavaTest and removes the unused javaFixtureFactory. Bug: 182885307 Test: m nothing Change-Id: I809772d14af2af211b9e15ad676fbdc06b07cd46
* | Remove unused test methods and buildDirPaul Duffin2021-03-241-123/+2
| | | | | | | | | | | | Bug: 182885307 Test: m nothing Change-Id: I251145a82479428a82f19cbde6b415d149acd619
* | Remove uses of buildDir from java/java_test.go and java/sdk_test.goPaul Duffin2021-03-241-51/+52
| | | | | | | | | | | | | | | | | | | | Remove any uses, either direct (or indirect via testJavaConfig or similar methods), of the package level buildDir variable from this file. Bug: 182885307 Test: m nothing Change-Id: I4d9dc39d3b1e5c37ba1f9e72da94048949c0fd17
* | Register java_plugin in PrepareForTestWithJavaBuildComponentsPaul Duffin2021-03-241-2/+0
| | | | | | | | | | | | Bug: 182885307 Test: m nothing Change-Id: I550d39ba46c548b6b099d8dc6a9c458ca931b2fa
* | Register the propagateRROEnforcementMutatorPaul Duffin2021-03-221-4/+0
| | | | | | | | | | | | | | | | | | This adds the registration of the mutator to the existing register... method which will include it in PrepareForTestWithJavaBuildComponents. Bug: 182885307 Test: m nothing Change-Id: I6cea716a3ff4d8abdb80543b7e0ddf22246ffa30
* | Add preparer for overlay pre-singleton registrationPaul Duffin2021-03-221-1/+1
| | | | | | | | | | | | | | | | | | | | It appears as though this is the first pre-singleton type to actually be registered with the InitRegistrationContext as it failed due to an uninitialized map, so this change also fixes that. Bug: 182885307 Test: m nothing Change-Id: Ibbf6d0db5f3c2fcc89291a16aa5f16b8b5009bd3
* | Remove emptyFixtureFactory from apex and javaPaul Duffin2021-03-221-3/+17
| | | | | | | | | | | | Bug: 183235980 Test: m nothing Change-Id: I350b45e2f57430fb158f4141a566e75de17208cd
* | Merge changes Iab4e09d9,Icf2f24dd,I15be5ef1,Ic0db9619Colin Cross2021-03-191-7/+7
|\ \ | | | | | | | | | | | | | | | | | | | | | * changes: Run lint actions in sbox Support sbox-in-RBE Move android package on top of remotexec Support sandboxing inputs in RuleBuilder
| * | Run lint actions in sboxColin Cross2021-03-181-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Run lint actions in sbox with RuleBuilder.SandboxInputs. This copies all input files into the sandbox, which prevents the lint tool from finding nearby source files that were not presented to it. Using SandboxInputs requires use of PathForInput or PathForOutput anywhere a path is used outside of the RuleBuilderCommand methods that take paths so that they can be translated to the paths that will be used in the sandbox. Bug: 181681346 Test: lint_test.go Test: m lint-check dist Test: m USE_RBE=true RBE_LINT=true lint-check dist Test: m USE_RBE=true RBE_LINT=true RBE_LINT_EXEC_STRATEGY=remote lint-check dist Change-Id: Iab4e09d961891ef182643583d4d456e413bc5e39
* | | Merge "Remove javaMockFS()"Paul Duffin2021-03-191-2/+0
|\ \ \
| * | | Remove javaMockFS()Paul Duffin2021-03-181-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, removes the workaround that allowed preparers to override files added in javaMockFS(). The performance of the java tests was determined by running: m nothing rm -fr out/soong/.bootstrap/soong-java/test/ time m nothing Prior to starting this work that returned: real 1m30.060s user 33m3.028s sys 13m25.413s Immediately prior to this change that returned: real 1m2.777s user 14m14.352s sys 9m40.745s After applying this change that returned: real 0m20.282s user 3m7.429s sys 0m43.601s So, this reduced the real time cost of running the java tests from 90 seconds to 20 seconds. Bug: 182638834 Test: m nothing Change-Id: I30490fd40ddb0684a5a4975ce1811bab5dc79e4e
* | | | Merge "Fix TestTurbine to work without javaMockFS()"Paul Duffin2021-03-191-22/+14
|\| | |
| * | | Fix TestTurbine to work without javaMockFS()Paul Duffin2021-03-181-22/+14
| | | | | | | | | | | | | | | | | | | | | | | | Bug: 182638834 Test: m nothing Change-Id: Idf400b5f0d171a504acf006425cbfc514b391e74
* | | | Merge "Convert TestClasspath to test fixtures"Paul Duffin2021-03-191-1/+8
|\| | | | |/ / |/| |
| * | Convert TestClasspath to test fixturesPaul Duffin2021-03-181-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The main complication with this change was that many of the tests check paths that are provided by the default java modules. The location of them is different in the test fixtures that it is when using testConfig() and so the test needed to be changed. Bug: 182638834 Test: m nothing Change-Id: I6d325dbd3ba39e5de7e53c576d7cfe07bd95a965
* | | Merge "Clean up java package use of python build components"Paul Duffin2021-03-181-4/+1
|\ \ \
| * | | Clean up java package use of python build componentsPaul Duffin2021-03-171-4/+1
| |/ / | | | | | | | | | | | | | | | Bug: 182885307 Test: m nothing Change-Id: I206ebc075c568a09008ec8b63122922dd87ba318
* / / Add emptyFixtureFactory in java packagePaul Duffin2021-03-171-2/+3
|/ / | | | | | | | | | | Bug: 182402754 Test: m nothing Change-Id: I6a01b846a8aee5a0ace9e6ac056ee439bf6f12f7
* | Merge "Export a few utility methods"Yuexi Ma2021-03-171-0/+13
|\ \
| * | Export a few utility methodsYuexi Ma2021-03-161-0/+13
| | | | | | | | | | | | | | | | | | | | | Add a few utility methods to support generating TradeFed test plan in C-Suite Test: go test . Change-Id: I86a505068b5d5aa0f5a37b3e3a2465839a7480ef
* | | Fix TestJavaSdkLibrary... tests to work without javaMockFS()Paul Duffin2021-03-161-12/+58
| | | | | | | | | | | | | | | | | | Bug: 182638834 Test: m nothing Change-Id: I140b88b3fdf2c532d266a196fee84675c518d859
* | | Convert JavaSdkLibrary... tests in java package to test fixturesPaul Duffin2021-03-161-136/+122
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These tests rely on files provided by javaMockFS() so need to are being converted to test fixtures to allow them to remove that dependency which will allow javaMockFS() to be removed. Bug: 182638834 Test: m nothing Change-Id: If736b95d134fe73c0a0ccd438ec31e93568fb929
* | | Convert TestJavaSdkLibraryEnforce to test fixturesPaul Duffin2021-03-151-16/+16
| | | | | | | | | | | | | | | | | | Bug: 182638834 Test: m nothing Change-Id: I99efe76008f9ca0996177f47923274ca594a6858
* | | Create sub tests for each test in TestJavaSdkLibraryEnforcePaul Duffin2021-03-151-18/+28
| | | | | | | | | | | | | | | | | | Bug: 182638834 Test: m nothing Change-Id: I509b071ee4e470da45159b2174aae7acf1b0eb1a
generated by cgit v1.2.3 (git 2.25.1) at 2025-12-24 01:37:06 +0000