Skip to content

Commit de4efa0

Browse files
authored
[SYCL] Move XFAIL tracker checker test to compile-time (#15832)
We have all the info at compile time, no need to run this on a device every time. Note the directory is moved from `test-e2e` to `test`, it's hard to see in the github UI. We can drop the `--exclude` since the test isn't in the same folder anymore. Using `../` is a little gross, but it's worth it to save the resources of running E2E tests multiple times trying to fix failures in this test. Also manually tested that changing the XFAIL number causes the test to fail in `check-sycl`. Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>
1 parent 41164a7 commit de4efa0

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

sycl/test-e2e/no-xfail-without-tracker.cpp renamed to sycl/test/no-xfail-without-tracker.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@
2525
// verify that against the reference
2626
// - ...and check if the list of improperly XFAIL-ed tests needs to be updated.
2727
//
28-
// RUN: grep -rI "XFAIL:" %S -A 1 --include=*.c --include=*.cpp \
29-
// RUN: --exclude=no-xfail-without-tracker.cpp --no-group-separator | \
28+
// RUN: grep -rI "XFAIL:" %S/../test-e2e \
29+
// RUN: -A 1 --include=*.c --include=*.cpp --no-group-separator | \
3030
// RUN: grep -v "XFAIL:" | \
3131
// RUN: grep -Pv "XFAIL-TRACKER:\s+(?:https://github.com/[\w\d-]+/[\w\d-]+/issues/[\d]+)|(?:[\w]+-[\d]+)" > %t | \
3232
// RUN: cat %t | wc -l | FileCheck %s --check-prefix NUMBER-OF-XFAIL-WITHOUT-TRACKER

0 commit comments

Comments
 (0)