about summary refs log tree commit diff
diff options
context:
space:
mode:
authorflip1995 <hello@philkrones.com>2020-01-26 15:13:14 +0100
committerflip1995 <hello@philkrones.com>2020-02-12 09:34:27 +0100
commitea780a69f3d1d47eaacd2811ab7d742c1627b3d5 (patch)
tree2b544ebff6908da0584ea2438320790d5f26a546
parent935b94d80b4456c0c9e50526af2c9f441b846949 (diff)
downloadrust-ea780a69f3d1d47eaacd2811ab7d742c1627b3d5.tar.gz
rust-ea780a69f3d1d47eaacd2811ab7d742c1627b3d5.zip
Extract driver test
-rw-r--r--.github/driver.sh29
-rw-r--r--.github/workflows/clippy.yml28
-rw-r--r--.github/workflows/clippy_bors.yml32
3 files changed, 35 insertions, 54 deletions
diff --git a/.github/driver.sh b/.github/driver.sh
new file mode 100644
index 00000000000..eb81e45a64e
--- /dev/null
+++ b/.github/driver.sh
@@ -0,0 +1,29 @@
+#! /bin/bash
+
+set -ex
+
+# Check sysroot handling
+sysroot=$(./target/debug/clippy-driver --print sysroot)
+test "$sysroot" = "$(rustc --print sysroot)"
+
+if [[ ${OS} == "Windows" ]]; then
+  desired_sysroot=C:/tmp
+else
+  desired_sysroot=/tmp
+fi
+sysroot=$(./target/debug/clippy-driver --sysroot $desired_sysroot --print sysroot)
+test "$sysroot" = $desired_sysroot
+
+sysroot=$(SYSROOT=$desired_sysroot ./target/debug/clippy-driver --print sysroot)
+test "$sysroot" = $desired_sysroot
+
+# Make sure this isn't set - clippy-driver should cope without it
+unset CARGO_MANIFEST_DIR
+
+# Run a lint and make sure it produces the expected output. It's also expected to exit with code 1
+# FIXME: How to match the clippy invocation in compile-test.rs?
+./target/debug/clippy-driver -Dwarnings -Aunused -Zui-testing --emit metadata --crate-type bin tests/ui/cstring.rs 2> cstring.stderr && exit 1
+sed -e "s,tests/ui,\$DIR," -e "/= help/d" cstring.stderr > normalized.stderr
+diff normalized.stderr tests/ui/cstring.stderr
+
+# TODO: CLIPPY_CONF_DIR / CARGO_MANIFEST_DIR
diff --git a/.github/workflows/clippy.yml b/.github/workflows/clippy.yml
index 7997344aabd..9661f826abd 100644
--- a/.github/workflows/clippy.yml
+++ b/.github/workflows/clippy.yml
@@ -68,31 +68,9 @@ jobs:
       run: ../target/debug/cargo-clippy
       working-directory: clippy_workspace_tests
     - name: Test clippy-driver
-      run: |
-        (
-          set -ex
-          # Check sysroot handling
-          sysroot=$(./target/debug/clippy-driver --print sysroot)
-          test "$sysroot" = "$(rustc --print sysroot)"
-
-          desired_sysroot=/tmp
-          sysroot=$(./target/debug/clippy-driver --sysroot $desired_sysroot --print sysroot)
-          test "$sysroot" = $desired_sysroot
-
-          sysroot=$(SYSROOT=$desired_sysroot ./target/debug/clippy-driver --print sysroot)
-          test "$sysroot" = $desired_sysroot
-
-          # Make sure this isn't set - clippy-driver should cope without it
-          unset CARGO_MANIFEST_DIR
-
-          # Run a lint and make sure it produces the expected output. It's also expected to exit with code 1
-          # FIXME: How to match the clippy invocation in compile-test.rs?
-          ./target/debug/clippy-driver -Dwarnings -Aunused -Zui-testing --emit metadata --crate-type bin tests/ui/cstring.rs 2> cstring.stderr && exit 1
-          sed -e 's,tests/ui,$DIR,' -e '/= help/d' cstring.stderr > normalized.stderr
-          diff normalized.stderr tests/ui/cstring.stderr
-
-          # TODO: CLIPPY_CONF_DIR / CARGO_MANIFEST_DIR
-        )
+      run: bash .github/driver.sh
+      env:
+        OS: ${{ runner.os }}
 
     - name: Run cargo-cache --autoclean
       run: |
diff --git a/.github/workflows/clippy_bors.yml b/.github/workflows/clippy_bors.yml
index cb2041e4990..49e4251f9da 100644
--- a/.github/workflows/clippy_bors.yml
+++ b/.github/workflows/clippy_bors.yml
@@ -136,36 +136,10 @@ jobs:
       shell: bash
       working-directory: clippy_workspace_tests
     - name: Test clippy-driver
-      run: |
-        (
-          set -ex
-          # Check sysroot handling
-          sysroot=$(./target/debug/clippy-driver --print sysroot)
-          test "$sysroot" = "$(rustc --print sysroot)"
-
-          if [[ ${{ runner.os }} == "Windows" ]]; then
-            desired_sysroot=C:/tmp
-          else
-            desired_sysroot=/tmp
-          fi
-          sysroot=$(./target/debug/clippy-driver --sysroot $desired_sysroot --print sysroot)
-          test "$sysroot" = $desired_sysroot
-
-          sysroot=$(SYSROOT=$desired_sysroot ./target/debug/clippy-driver --print sysroot)
-          test "$sysroot" = $desired_sysroot
-
-          # Make sure this isn't set - clippy-driver should cope without it
-          unset CARGO_MANIFEST_DIR
-
-          # Run a lint and make sure it produces the expected output. It's also expected to exit with code 1
-          # FIXME: How to match the clippy invocation in compile-test.rs?
-          ./target/debug/clippy-driver -Dwarnings -Aunused -Zui-testing --emit metadata --crate-type bin tests/ui/cstring.rs 2> cstring.stderr && exit 1
-          sed -e 's,tests/ui,$DIR,' -e '/= help/d' cstring.stderr > normalized.stderr
-          diff normalized.stderr tests/ui/cstring.stderr
-
-          # TODO: CLIPPY_CONF_DIR / CARGO_MANIFEST_DIR
-        )
+      run: bash .github/driver.sh
       shell: bash
+      env:
+        OS: ${{ runner.os }}
 
     - name: Run cargo-cache --autoclean
       run: |