| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
[test] Infer status-file variables from build flags This auto-generates all build-time flags for status-file configuration from the flags specified in V8's BUILD.gn file. This makes makes adding such a flag a one-line change in the future, instead of >10 lines in >10 files as in https://crrev.com/c/4521710. This also raises an improved exception when a build flag is missing either in production or in tests, showing a hint with the location where to add it. This also removes redundant test data and instead bootstraps V8's build configuration for testing from a single blueprint. Cq-Include-Trybots: luci.v8.try:v8_numfuzz_dbg Cq-Include-Trybots: luci.v8.try:v8_numfuzz_rel Cq-Include-Trybots: luci.v8.try:v8_numfuzz_tsan_rel Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel Cq-Include-Trybots: luci.v8.try:v8_android_arm64_n5x_rel Cq-Include-Trybots: luci.v8.try:v8_linux64_msan_rel Cq-Include-Trybots: luci.v8.try:v8_linux64_ubsan_rel Bug: chromium:1132088 Change-Id: I53c002857fcacf7368c23c249209170e7af6291b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4528239 Reviewed-by: Liviu Rau <liviurau@google.com> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/main@{#87674} | 3 年前 | |
[test] Refactor testrunner (4) - Removed duplication and unnecessary indirection from all suites testcfgs. - Introduced a more comprehensive context to cover both command context and other platform specific concerns. - Propagated above context to TestLoader to allow for test counting command execution on all platforms. - Wrapped original pool with another class to give it a new interface and allow injecting different implementations in the future. - Consolidated progress indicators under a single processor in the pipeline. - Consolidated result retention requirements calculation outside of pipeline chain. - Refactored LoaderProc and got it under tests. - Added some more tests for the standard runner. - Extracted BuildConfig class. Bug: v8:12785 Change-Id: I87be040e91f792a983662bb5a10d55b36a14ea7f Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3701595 Reviewed-by: Michael Achenbach <machenbach@chromium.org> Commit-Queue: Liviu Rau <liviurau@chromium.org> Cr-Commit-Position: refs/heads/main@{#81770} | 3 年前 |