[rb] Make //rb:lint a test target that auto-corrects files too

Author: p0deje

.github/workflows/ci-ruby.yml

@@ -29,7 +29,7 @@ jobs:
     with:
       name: Lint
       cache-key: rb-lint
-      run: bazel run //rb:lint
+      run: bazel test //rb:lint
 
   unit-tests:
     name: Unit Tests

README.md

@@ -209,25 +209,28 @@ Build targets:
 | `bazel run //rb:selenium-webdriver-release` | Build and push selenium-webdriver gem to RubyGems |
 | `bazel run //rb:console`                    | Start REPL with all gems loaded                   |
 | `bazel run //rb:docs`                       | Generate YARD docs                                |
-| `bazel run //rb:lint`                       | Run RuboCop linter                                |
 
 Test targets:
 
-| Command                                                                              | Description                                             |
-|--------------------------------------------------------------------------------------|---------------------------------------------------------|
-| `bazel test //rb/spec/...`                                                           | Run both unit and integration tests using Chrome        |
-| `bazel test //rb/spec/integration/...`                                               | Run integration tests using Chrome                      |
-| `bazel test //rb/spec/integration/... --define browser=firefox`                      | Run integration tests using Firefox                     |
-| `bazel test //rb/spec/integration/... --define remote=true`                          | Run integration tests using Chrome and Selenium Server  |
-| `bazel test //rb/spec/integration/... --define browser=firefox --define remote=true` | Run integration tests using Firefox and Selenium Server |
-| `bazel test //rb/spec/unit/...`                                                      | Run unit tests                                          |
+| Command                                                                              | Description                                    |
+|--------------------------------------------------------------------------------------|------------------------------------------------|
+| `bazel test //rb/...`                                                                | Run unit, integration tests (Chrome) and lint  |
+| `bazel test //rb:lint`                                                               | Run RuboCop linter                             |
+| `bazel test //rb/spec/...`                                                           | Run unit and integration tests (Chrome)        |
+| `bazel test --test_size_filters large //rb/...`                                      | Run integration tests using (Chrome)           |
+| `bazel test //rb/spec/integration/...`                                               | Run integration tests using (Chrome)           |
+| `bazel test //rb/spec/integration/... --define browser=firefox`                      | Run integration tests using (Firefox)          |
+| `bazel test //rb/spec/integration/... --define remote=true`                          | Run integration tests using (Chrome and Grid)  |
+| `bazel test //rb/spec/integration/... --define browser=firefox --define remote=true` | Run integration tests using (Firefox and Grid) |
+| `bazel test --test_size_filters small //rb/...`                                      | Run unit tests                                 |
+| `bazel test //rb/spec/unit/...`                                                      | Run unit tests                                 |
 
 Suffix `...` tells Bazel to run all the test targets. They are conveniently named by test file name with `_spec.rb` removed so you can run them individually:
 
 | Test file                                                      | Test target                                              |
 |----------------------------------------------------------------|----------------------------------------------------------|
 | `rb/spec/integration/selenium/webdriver/chrome/driver_spec.rb` | `//rb/spec/integration/selenium/webdriver/chrome:driver` |
-| `rb/spec/integration/selenium/webdriver/chrome/driver_spec.rb` | `//rb/spec/integration/selenium/webdriver/chrome:driver` |
+| `rb/spec/unit/selenium/webdriver/proxy_spec.rb`                | `//rb/spec/unit/selenium/webdriver:proxy`                |
 
 Supported browsers:
 

rb/BUILD.bazel

@@ -5,6 +5,7 @@ load(
     "rb_binary",
     "rb_gem_build",
     "rb_gem_push",
+    "rb_test",
 )
 
 package(default_visibility = ["//:__subpackages__"])
@@ -172,17 +173,17 @@ rb_binary(
     ],
 )
 
-rb_binary(
+rb_test(
     name = "lint",
-    testonly = True,
     srcs = glob(["**/*.rb"]),
     args = [
+        "--autocorrect",
         "--config=rb/.rubocop.yml",
         "rb/",
     ],
     data = [".rubocop.yml"],
     main = "@bundle//:bin/rubocop",
-    tags = ["local"],
+    tags = ["no-sandbox"],
     deps = [
         ":selenium-devtools",
         ":selenium-webdriver",