Capture/Replay: Add suffix to label for test file search

When tests are run with a wildcard filter on the backend, e.g. "TEST/ES2_Vulkan*" the same tests from different backends may end up in the same trace file directory, e.g. TEST_ES2_Vulkan and TEST_ES2_Vulkan_SwiftShader. So far, when searching for the ES2_Vulkan trace files the check for trace files would them also pick up all the _SwiftShader files, and the skript would assume that the file count is wrong and skip the replay. By adding "_capture" the start of the commonly used suffix, this is avoided and replays are no longer skipped because mis-counting capture files. Bug: angleproject:5972 Change-Id: I7af54aaf51ee7070333ffe81974d866bd89dbd6f Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2897236 Commit-Queue: Gert Wollny <gert.wollny@collabora.com> Reviewed-by: Jamie Madill <jmadill@chromium.org> Reviewed-by: Cody Northrop <cnorthrop@google.com>

Capture/Replay: Add suffix to label for test file search
1af7caa4 · Gert Wollny · Commit Bot · 022401f9 · 1af7caa4
Commit 1af7caa4 authored May 14, 2021 by Gert Wollny Committed by Commit Bot May 17, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

capture_replay_tests.py src/tests/capture_replay_tests.py +1 -1

No files found.
--- a/src/tests/capture_replay_tests.py
+++ b/src/tests/capture_replay_tests.py
@@ -457,7 +457,7 @@ class Test():

    def CanRunReplay(self, trace_folder_path):
        test_files = []
-        label = self.GetLabel()
+        label = self.GetLabel() + "_capture"
        assert (self.context_id == 0)
        for f in os.listdir(trace_folder_path):
            if os.path.isfile(os.path.join(trace_folder_path, f)) and f.startswith(label):