[java] Run tests on the JUnit Platform (JUnit 5) (#10752)

* [java] Use java_test_suite and from contrib_rule_jvm

First step in the migration of the test codebase to JUnit 5

issue #10196

* [java] Manage JUnit 5 dependencies, using junit-vintage for existing tests

Second step in the migration of the test codebase to JUnit 5

issue #10196

* [java] Move interface HasMagicNumbers to a separate class

issue #10196

* [java] Include missing dependencies

issue #10196

* [java] Include additional JUnit 5 dependencies

issue #10196q

* [java] Use JUnit 4 runner by default

Since the existing tests are based on JUnit 4

issue #10196

* [java] Remove lint tests for contrib_rules_jvm macros

Pull java_test and java_library from contrib_rules_jvm
and not from the default packages

issue #10196

* [java] Bump to contrib_rules_jvm version 0.3.0

* [java] Use spotbugs as linter from contrib_rules_jvm

Co-authored-by: Diego Molina <diemol@users.noreply.github.com>

Author: bonigarcia

WORKSPACE

@@ -130,6 +130,21 @@ load("@rules_jvm_external//:setup.bzl", "rules_jvm_external_setup")
 
 rules_jvm_external_setup()
 
+http_archive(
+    name = "contrib_rules_jvm",
+    sha256 = "79773625951a7a6dc0fc1636ef2388253b69a1e8109b34198c191953280648ab",
+    strip_prefix = "rules_jvm-0.3.0",
+    url = "https://github.com/bazel-contrib/rules_jvm/archive/v0.3.0.tar.gz",
+)
+
+load("@contrib_rules_jvm//:repositories.bzl", "contrib_rules_jvm_deps")
+
+contrib_rules_jvm_deps()
+
+load("@contrib_rules_jvm//:setup.bzl", "contrib_rules_jvm_setup")
+
+contrib_rules_jvm_setup()
+
 load("//java:maven_deps.bzl", "selenium_java_deps")
 
 selenium_java_deps()

java/BUILD.bazel

@@ -1,5 +1,5 @@
 load("@rules_jvm_external//:defs.bzl", "artifact")
-load("//java:defs.bzl", "spotbugs_config")
+load("@contrib_rules_jvm//java:defs.bzl", "spotbugs_config")
 
 exports_files(
     srcs = [
@@ -41,7 +41,7 @@ spotbugs_config(
     name = "spotbugs-config",
     effort = "default",
     exclude_filter = "spotbugs-excludes.xml",
-    fail_on_warning = True,
+    fail_on_warning = False,
     visibility = [
         "//visibility:public",
     ],

java/defs.bzl

@@ -9,8 +9,16 @@ load("//java/private:library.bzl", _java_export = "java_export", _java_library =
 load("//java/private:merge_jars.bzl", _merge_jars = "merge_jars")
 load("//java/private:module.bzl", _java_module = "java_module")
 load("//java/private:selenium_test.bzl", _selenium_test = "selenium_test")
-load("//java/private:spotbugs_config.bzl", _spotbugs_config = "spotbugs_config")
-load("//java/private:suite.bzl", _java_selenium_test_suite = "java_selenium_test_suite", _java_test_suite = "java_test_suite")
+load("//java/private:suite.bzl", _java_selenium_test_suite = "java_selenium_test_suite")
+load(
+    "@contrib_rules_jvm//java:defs.bzl",
+    _JUNIT5_DEPS = "JUNIT5_DEPS",
+    _JUNIT5_VINTAGE_DEPS = "JUNIT5_VINTAGE_DEPS",
+    _java_test_suite = "java_test_suite",
+)
+
+def java_test_suite(name, runner = "junit4", **kwargs):
+    _java_test_suite(name = name, runner = runner, **kwargs)
 
 artifact = _artifact
 java_binary = _java_binary
@@ -21,8 +29,8 @@ java_library = _java_library
 java_module = _java_module
 java_selenium_test_suite = _java_selenium_test_suite
 java_test = _java_test
-java_test_suite = _java_test_suite
 javadoc = _javadoc
 merge_jars = _merge_jars
 selenium_test = _selenium_test
-spotbugs_config = _spotbugs_config
+JUNIT5_DEPS = _JUNIT5_DEPS
+JUNIT5_VINTAGE_DEPS = _JUNIT5_VINTAGE_DEPS

java/maven_deps.bzl

@@ -4,6 +4,8 @@ load("@rules_jvm_external//:specs.bzl", "maven")
 def selenium_java_deps():
     netty_version = "4.1.77.Final"
     opentelemetry_version = "1.14.0"
+    junit_jupiter_version = "5.8.2"
+    junit_platform_version = "1.8.2"
 
     maven_install(
         artifacts = [
@@ -66,6 +68,14 @@ def selenium_java_deps():
                     "org.hamcrest:hamcrest-library",
                 ],
             ),
+            "org.junit.jupiter:junit-jupiter-api:%s" % junit_jupiter_version,
+            "org.junit.jupiter:junit-jupiter-engine:%s" % junit_jupiter_version,
+            "org.junit.jupiter:junit-jupiter-params:%s" % junit_jupiter_version,
+            "org.junit.vintage:junit-vintage-engine:%s" % junit_jupiter_version,
+            "org.junit.platform:junit-platform-launcher:%s" % junit_platform_version,
+            "org.junit.platform:junit-platform-reporting:%s" % junit_platform_version,
+            "org.junit.platform:junit-platform-commons:%s" % junit_platform_version,
+            "org.junit.platform:junit-platform-engine:%s" % junit_platform_version,
             "net.bytebuddy:byte-buddy:1.12.10",
             "dev.failsafe:failsafe:3.2.4",
             "net.sourceforge.htmlunit:htmlunit-core-js:2.61.1",

java/maven_install.json

@@ -1,11 +1,11 @@
 load("@apple_rules_lint//lint:defs.bzl", "get_lint_config")
 load(
-    "@rules_java//java:defs.bzl",
+    "@contrib_rules_jvm//java:defs.bzl",
+    "spotbugs_test",
     _java_library = "java_library",
     _java_test = "java_test",
 )
 load(":export.bzl", _java_export = "java_export")
-load(":spotbugs.bzl", "spotbugs_test")
 
 def add_lint_tests(name, **kwargs):
     srcs = kwargs.get("srcs", [])
@@ -29,14 +29,6 @@ def add_lint_tests(name, **kwargs):
             timeout = "moderate",
         )
 
-def java_export(name, **kwargs):
-    add_lint_tests(name, **kwargs)
-    _java_export(name = name, **kwargs)
-
-def java_library(name, **kwargs):
-    add_lint_tests(name, **kwargs)
-    _java_library(name = name, **kwargs)
-
-def java_test(name, **kwargs):
-    add_lint_tests(name, **kwargs)
-    _java_test(name = name, **kwargs)
+java_export = _java_export
+java_library = _java_library
+java_test = _java_test

java/private/library.bzl

@@ -1,5 +1,5 @@
 load("@rules_jvm_external//:defs.bzl", "artifact")
-load("//java:defs.bzl", "java_library")
+load("//java:defs.bzl", "JUNIT5_VINTAGE_DEPS", "java_library")
 
 java_library(
     name = "api",
@@ -40,7 +40,7 @@ java_library(
         "//java/src/org/openqa/selenium:core",
         artifact("junit:junit"),
         artifact("org.testng:testng"),
-    ],
+    ] + JUNIT5_VINTAGE_DEPS,
 )
 
 java_library(

java/private/spotbugs.bzl

@@ -1,5 +1,5 @@
 load("@rules_jvm_external//:defs.bzl", "artifact")
-load("//java:defs.bzl", "java_library")
+load("//java:defs.bzl", "JUNIT5_VINTAGE_DEPS", "java_library")
 
 java_library(
     name = "condition",
@@ -10,5 +10,5 @@ java_library(
     deps = [
         "//java/src/com/thoughtworks/selenium:api",
         artifact("junit:junit"),
-    ],
+    ] + JUNIT5_VINTAGE_DEPS,
 )