diff options
author | Antonio Barone <syntonyze@gmail.com> | 2021-01-07 15:07:37 +0100 |
---|---|---|
committer | Antonio Barone <syntonyze@gmail.com> | 2021-01-07 15:08:11 +0100 |
commit | 011c577ef4ee2149d73db974e0ef2fcd3f66bfcf (patch) | |
tree | 215af8072274a7ed1b1bfafc4b86537b62e2e316 | |
parent | b197452eeaa6e8b1270465e6640331f2a9ae6b38 (diff) | |
parent | 99c822ebf3b72023f8b926444bbdf299fafb00df (diff) |
Merge branch 'stable-3.2' into stable-3.3
* stable-3.2:
Split integration tests to different targets
Don't check read permission when authgroup isn't set
Change-Id: I4a1e1be5c4323de1554091786c55ca9a84d391e5
-rw-r--r-- | BUILD | 10 | ||||
-rw-r--r-- | src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java | 3 |
2 files changed, 7 insertions, 6 deletions
@@ -30,18 +30,16 @@ junit_tests( ], ) -junit_tests( - name = "replication_it", - srcs = glob([ - "src/test/java/**/*IT.java", - ]), +[junit_tests( + name = f[:f.index(".")].replace("/", "_"), + srcs = [f], tags = ["replication"], visibility = ["//visibility:public"], deps = PLUGIN_TEST_DEPS + PLUGIN_DEPS + [ ":replication__plugin", ":replication_util", ], -) +) for f in glob(["src/test/java/**/*IT.java"])] java_library( name = "replication_util", diff --git a/src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java b/src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java index 8d7227f..bd34676 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java +++ b/src/main/java/com/googlesource/gerrit/plugins/replication/Destination.java @@ -335,6 +335,9 @@ public class Destination { if (PushOne.ALL_REFS.equals(ref)) { return true; } + if (userProvider.get().isInternalUser()) { + return true; + } try { permissionBackend .user(userProvider.get()) |