diff options
author | David Ostrovsky <david@ostrovsky.org> | 2020-04-18 00:00:32 +0200 |
---|---|---|
committer | Matthias Sohn <matthias.sohn@sap.com> | 2020-04-18 18:51:36 +0200 |
commit | 9b2f8ced9018dad519fefdc19d3940104b95434a (patch) | |
tree | 3ff07393cafe587518705ee7a9d1a4c8b05fb237 /org.eclipse.jgit.test | |
parent | a0aebbd52899c21eedc9b021f595e45da07820d2 (diff) | |
download | jgit-9b2f8ced9018dad519fefdc19d3940104b95434a.tar.gz jgit-9b2f8ced9018dad519fefdc19d3940104b95434a.zip |
Bazel: Disable SecurityManagerMissingPermissionsTest test
In Id5376f09f0d a test with dependency on log4j library was added, but
the library was missed to be added to the Bazel build tool chain.
Given that Bazel test runner doesn't suport custom security manager the
test wouldn't pass even if the missing dependency would be added. The
only solution we have for now is to exclude that test from Bazel tool
chain.
Filed a feature request for bazel to support such tests at
https://github.com/bazelbuild/bazel/issues/11146
Bug: 562274
Change-Id: I873a0e09addc583455b68122f66cd3952e485f0e
Signed-off-by: David Ostrovsky <david@ostrovsky.org>
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
Diffstat (limited to 'org.eclipse.jgit.test')
-rw-r--r-- | org.eclipse.jgit.test/BUILD | 1 | ||||
-rw-r--r-- | org.eclipse.jgit.test/tests.bzl | 4 |
2 files changed, 5 insertions, 0 deletions
diff --git a/org.eclipse.jgit.test/BUILD b/org.eclipse.jgit.test/BUILD index b34ef2a796..3c4c61bfbb 100644 --- a/org.eclipse.jgit.test/BUILD +++ b/org.eclipse.jgit.test/BUILD @@ -40,6 +40,7 @@ DATA = [ EXCLUDED = [ PKG + "api/SecurityManagerTest.java", + PKG + "api/SecurityManagerMissingPermissionsTest.java", ] RESOURCES = glob(["resources/**"]) diff --git a/org.eclipse.jgit.test/tests.bzl b/org.eclipse.jgit.test/tests.bzl index f27efccc23..d030316452 100644 --- a/org.eclipse.jgit.test/tests.bzl +++ b/org.eclipse.jgit.test/tests.bzl @@ -42,6 +42,10 @@ def tests(tests): additional_deps = [ "//lib:jsch", ] + if src.endswith("SecurityManagerMissingPermissionsTest.java"): + additional_deps = [ + "//lib:log4j", + ] if src.endswith("JSchSshTest.java"): additional_deps = [ "//lib:jsch", |