ci: separate ui & normal tests

Signed-off-by: Benno Lossin <lossin@kernel.org>

Author: BennoLossin

.github/workflows/ci.yml

@@ -89,7 +89,24 @@ jobs:
           git rebase --exec "bash exec.sh" --root
         env:
           RUSTFLAGS: "-Dwarnings"
-  test:
+  ui-tests:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: ${{github.event.pull_request.commits}}
+          ref: ${{github.event.pull_request.head.sha}}
+      - uses: dtolnay/rust-toolchain@nightly
+        with:
+          components: rust-src
+      - run: cargo install cargo-expand
+      - run: git config user.name "github-runner" && git config user.email "<>"
+      - run: git rebase --exec 'cargo test ui_ --locked --all-targets' --root
+        env:
+          RUSTFLAGS: >
+            -Dwarnings
+            --cfg UI_TESTS
+  tests:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
@@ -170,7 +187,6 @@ jobs:
         env:
           ASAN_OPTIONS: "detect_odr_violation=0:detect_leaks=0"
           RUSTFLAGS: >
-            --cfg NO_UI_TESTS
             --cfg NO_ALLOC_FAIL_TESTS
             -Z sanitizer=address
             -Dwarnings
@@ -187,7 +203,6 @@ jobs:
         env:
           ASAN_OPTIONS: "detect_odr_violation=0:detect_leaks=0"
           RUSTFLAGS: >
-            --cfg NO_UI_TESTS
             --cfg NO_ALLOC_FAIL_TESTS
             -Z sanitizer=address
             -Dwarnings
@@ -197,7 +212,6 @@ jobs:
             --root
         env:
           RUSTFLAGS: >
-            --cfg NO_UI_TESTS
             --cfg NO_ALLOC_FAIL_TESTS
             -Z sanitizer=leak
             -Dwarnings
@@ -208,7 +222,6 @@ jobs:
             --root
         env:
           RUSTFLAGS: >
-            --cfg NO_UI_TESTS
             --cfg NO_ALLOC_FAIL_TESTS
             -Z sanitizer=leak
             -Dwarnings

Cargo.toml

@@ -36,7 +36,7 @@ prettyplease = { version = "0.2", features = ["verbatim"] }
 [lints.rust]
 non_ascii_idents = "deny"
 unexpected_cfgs = { level = "warn", check-cfg = [
-    'cfg(NO_UI_TESTS)',
+    'cfg(UI_TESTS)',
     'cfg(NO_ALLOC_FAIL_TESTS)',
     'cfg(kernel)',
 ] }

tests/ui.rs

@@ -1,8 +1,8 @@
 #![cfg_attr(not(RUSTC_LINT_REASONS_IS_STABLE), feature(lint_reasons))]
 
 #[test]
-#[cfg_attr(any(miri, NO_UI_TESTS), ignore)]
-fn compile_fail() {
+#[cfg_attr(not(UI_TESTS), ignore)]
+fn ui_compile_fail() {
     let test_cases = trybuild::TestCases::new();
     test_cases.compile_fail("tests/ui/compile-fail/pinned_drop/*.rs");
     test_cases.compile_fail("tests/ui/compile-fail/pin_data/*.rs");
@@ -11,7 +11,7 @@ fn compile_fail() {
 }
 
 #[test]
-#[cfg_attr(any(miri, NO_UI_TESTS), ignore)]
-fn expand() {
+#[cfg_attr(not(UI_TESTS), ignore)]
+fn ui_expand() {
     macrotest::expand("tests/ui/expand/*.rs");
 }