Added regression tests for noclippy tag.

.bazelci/presubmit.yml

@@ -17,13 +17,15 @@ tasks:
       - "//test/versioned_dylib:versioned_dylib_test"
     build_flags:
       - "--config=rustfmt"
+      - "--config=clippy"
   ubuntu2004:
     name: "Minimum Supported Version"
     bazel: "3.5.0"
     build_targets: *default_linux_targets
     test_targets: *default_linux_targets
     build_flags:
       - "--config=rustfmt"
+      - "--config=clippy"
   macos:
     osx_targets: &osx_targets
       - "--" # Allows negative patterns; hack for https://github.com/bazelbuild/continuous-integration/pull/245
@@ -35,6 +37,7 @@ tasks:
     test_targets: *osx_targets
     build_flags:
       - "--config=rustfmt"
+      - "--config=clippy"
   rbe_ubuntu1604:
     test_targets:
       - "--" # Allows negative patterns; hack for https://github.com/bazelbuild/continuous-integration/pull/245
@@ -51,10 +54,12 @@ tasks:
       - "-@examples//ffi/rust_calling_c:matrix_dylib_test"
     build_flags:
       - "--config=rustfmt"
+      - "--config=clippy"
   windows:
     build_flags:
       - "--enable_runfiles" # this is not enabled by default on windows and is necessary for the cargo build scripts
       - "--config=rustfmt"
+      - "--config=clippy"
     windows_targets: &windows_targets
       - "--" # Allows negative patterns; hack for https://github.com/bazelbuild/continuous-integration/pull/245
       - "..."
@@ -81,6 +86,7 @@ tasks:
       - //...
     build_flags:
       - "--config=rustfmt"
+      - "--config=clippy"
   docs_linux:
     name: Docs
     platform: ubuntu1804
@@ -89,20 +95,11 @@ tasks:
       - //...
     run_targets:
       - "//:test_docs"
-  clippy_examples:
-    name: Clippy on Examples
-    platform: ubuntu1804
-    working_directory: examples
-    build_flags:
-      - "--aspects=@rules_rust//rust:defs.bzl%rust_clippy_aspect"
-      - "--output_groups=clippy_checks"
-    build_targets:
-      - //...
   clippy_failure:
     name: Negative Clippy Tests
     platform: ubuntu1804
-    shell_commands:
-      - ./test/clippy/clippy_failure_test.sh
+    run_targets:
+      - "///test/clippy:clippy_failure_test"
   rustfmt_failure:
     name: Negative Rustfmt Tests
     platform: ubuntu2004
@@ -112,6 +109,8 @@ tasks:
     name: Ubuntu 20.04 with Clang
     platform: ubuntu2004
     build_flags:
+      - "--config=rustfmt"
+      - "--config=clippy"
       - "--repo_env=CC=clang"
       # TODO(hlopko): Make this work (some tests were failing)
       # - "--linkopt=-fuse-ld=lld"
@@ -127,6 +126,9 @@ tasks:
       - "//..."
     test_targets:
       - "//..."
+    build_flags:
+      - "--config=rustfmt"
+      - "--config=clippy"
   crate_universe_rbe_ubuntu1604:
     name: Crate Universe Examples
     platform: rbe_ubuntu1604
@@ -139,6 +141,7 @@ tasks:
       - "//..."
     build_flags:
       - "--config=rustfmt"
+      - "--config=clippy"
   crate_universe_examples_macos:
     name: Crate Universe Examples
     platform: macos
@@ -151,6 +154,7 @@ tasks:
       - "//..."
     build_flags:
       - "--config=rustfmt"
+      - "--config=clippy"
   crate_universe_examples_windows:
     name: Crate Universe Examples
     platform: windows
@@ -160,6 +164,7 @@ tasks:
     build_flags:
       - "--enable_runfiles" # this is not enabled by default on windows and is necessary for the cargo build scripts
       - "--config=rustfmt"
+      - "--config=clippy"
     crate_universe_windows_targets: &crate_universe_windows_targets
       - "//..."
       # TODO: There are windows specific build issues in the generated

.bazelrc

@@ -4,3 +4,7 @@
 # Enable rustfmt for all targets in the workspace
 build:rustfmt --aspects=//rust:defs.bzl%rustfmt_aspect
 build:rustfmt --output_groups=+rustfmt_checks
+
+# Enable clippy for all targets in the workspace
+build:clippy --aspects=//rust:defs.bzl%rust_clippy_aspect
+build:clippy --output_groups=+clippy_checks

examples/.bazelrc

@@ -4,3 +4,7 @@
 # Enable rustfmt for all targets in the workspace
 build:rustfmt --aspects=@rules_rust//rust:defs.bzl%rustfmt_aspect
 build:rustfmt --output_groups=+rustfmt_checks
+
+# Enable clippy for all targets in the workspace
+build:clippy --aspects=@rules_rust//rust:defs.bzl%rust_clippy_aspect
+build:clippy --output_groups=+clippy_checks

examples/crate_universe/.bazelrc

@@ -4,3 +4,7 @@
 # Enable rustfmt for all targets in the workspace
 build:rustfmt --aspects=@rules_rust//rust:defs.bzl%rustfmt_aspect
 build:rustfmt --output_groups=+rustfmt_checks
+
+# Enable clippy for all targets in the workspace
+build:clippy --aspects=@rules_rust//rust:defs.bzl%rust_clippy_aspect
+build:clippy --output_groups=+clippy_checks

test/clippy/BUILD.bazel

@@ -50,18 +50,21 @@ rust_binary(
     name = "bad_binary",
     srcs = ["bad_src/main.rs"],
     edition = "2018",
+    tags = ["noclippy"],
 )
 
 rust_library(
     name = "bad_library",
     srcs = ["bad_src/lib.rs"],
     edition = "2018",
+    tags = ["noclippy"],
 )
 
 rust_test(
     name = "bad_test",
     srcs = ["bad_src/lib.rs"],
     edition = "2018",
+    tags = ["noclippy"],
 )
 
 # Clippy analysis of failing targets.
@@ -84,3 +87,8 @@ rust_clippy(
     tags = ["manual"],
     deps = [":bad_test"],
 )
+
+sh_binary(
+    name = "clippy_failure_test",
+    srcs = ["clippy_failure_test.sh"],
+)

test/clippy/clippy_failure_test.sh

@@ -7,6 +7,13 @@
 
 set -euo pipefail
 
+if [[ -z "${BUILD_WORKSPACE_DIRECTORY:-}" ]]; then
+  echo "This script should be run under Bazel"
+  exit 1
+fi
+
+cd "${BUILD_WORKSPACE_DIRECTORY}"
+
 # Executes a bazel build command and handles the return value, exiting
 # upon seeing an error.
 #
@@ -31,6 +38,31 @@ function test_all() {
   local -r BUILD_FAILED=1
   local -r TEST_FAIL=3
 
+  temp_dir="$(mktemp -d -t ci-XXXXXXXXXX)"
+  new_workspace="${temp_dir}/rules_rust_test_clippy"
+
+  mkdir -p "${new_workspace}/test/clippy" && \
+  cp -r test/clippy/* "${new_workspace}/test/clippy/" && \
+  cat << EOF > "${new_workspace}/WORKSPACE.bazel"
+workspace(name = "rules_rust_test_clippy")
+local_repository(
+    name = "rules_rust",
+    path = "${BUILD_WORKSPACE_DIRECTORY}",
+)
+load("@rules_rust//rust:repositories.bzl", "rust_repositories")
+rust_repositories()
+EOF
+
+  # Drop the 'noclippy' tags
+  if [ "$(uname)" == "Darwin" ]; then
+    SEDOPTS=(-i '' -e)
+  else
+    SEDOPTS=(-i)
+  fi
+  sed ${SEDOPTS[@]} 's/"noclippy"//' "${new_workspace}/test/clippy/BUILD.bazel"
+
+  pushd "${new_workspace}"
+
   check_build_result $BUILD_OK ok_binary_clippy
   check_build_result $BUILD_OK ok_library_clippy
   check_build_result $BUILD_OK ok_test_clippy