Skip to content

A
angle
Commit 62d4b9be by Jamie Madill Committed by Commit Bot
Options

Capture Tests: Add to CQ when script is changed.

This ensures we will force the CQ to verify the trace tests before changes to the capture test script itself, or the test expectations. Bug: angleproject:5530 Change-Id: I7d9d04a14ae0525a6ff59681216abbdb85b72c78 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2689365Reviewed-by: 's avatarCody Northrop <cnorthrop@google.com> Reviewed-by: 's avatarYuly Novikov <ynovikov@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org>
parent 537cd0d5
Showing with 7 additions and 2 deletions
infra/config/generated/commit-queue.cfg
...@@ -35,6 +35,7 @@ config_groups { ...@@ -35,6 +35,7 @@ config_groups {
builders { builders {
name: "angle/try/linux-trace-rel" name: "angle/try/linux-trace-rel"
location_regexp: ".+/[+]/src/libANGLE/capture/.+" location_regexp: ".+/[+]/src/libANGLE/capture/.+"
location_regexp: ".+/[+]/src/tests/capture.+"
} }
builders { builders {
name: "angle/try/mac-dbg" name: "angle/try/mac-dbg"
...@@ -73,6 +74,7 @@ config_groups { ...@@ -73,6 +74,7 @@ config_groups {
builders { builders {
name: "angle/try/win-trace-rel" name: "angle/try/win-trace-rel"
location_regexp: ".+/[+]/src/libANGLE/capture/.+" location_regexp: ".+/[+]/src/libANGLE/capture/.+"
location_regexp: ".+/[+]/src/tests/capture.+"
} }
builders { builders {
name: "angle/try/winuwp-x64-dbg" name: "angle/try/winuwp-x64-dbg"
......
infra/config/main.star
...@@ -180,7 +180,10 @@ def angle_standalone_builder(name, clang, debug, cpu, uwp = False, trace_tests = ...@@ -180,7 +180,10 @@ def angle_standalone_builder(name, clang, debug, cpu, uwp = False, trace_tests =
# Trace tests are only included automatically if files in the capture folder change. # Trace tests are only included automatically if files in the capture folder change.
if trace_tests: if trace_tests:
config = "trace" config = "trace"
location_regexp = ".+/[+]/src/libANGLE/capture/.+" location_regexp = [
".+/[+]/src/libANGLE/capture/.+",
".+/[+]/src/tests/capture.+",
]
else: else:
config = "angle" config = "angle"
location_regexp = None location_regexp = None
...@@ -216,7 +219,7 @@ def angle_standalone_builder(name, clang, debug, cpu, uwp = False, trace_tests = ...@@ -216,7 +219,7 @@ def angle_standalone_builder(name, clang, debug, cpu, uwp = False, trace_tests =
luci.cq_tryjob_verifier( luci.cq_tryjob_verifier(
cq_group = 'master', cq_group = 'master',
builder = "angle:try/" + name, builder = "angle:try/" + name,
location_regexp = [location_regexp] if location_regexp else None, location_regexp = location_regexp,
) )
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment