Merge branch 'stable-3.0' into stable-3.1 * stable-3.0: Bump Bazel version to 3.7.0 Upgrade bazlets to latest stable-3.0 to build with 3.0.13 API Change-Id: Icb27a831a2d85b32210d3f8ce2409f4133e24183
diff --git a/WORKSPACE b/WORKSPACE index 41ddc5f..2218eff 100644 --- a/WORKSPACE +++ b/WORKSPACE
@@ -3,7 +3,7 @@ load("//:bazlets.bzl", "load_bazlets") load_bazlets( - commit = "a010a539824bd467059f414412a47b80101352d7", + commit = "3f9dadc615dc4053369a42d9ada37dafd8d4763c", #local_path = "/home/<user>/projects/bazlets", )
diff --git a/src/main/java/com/googlesource/gerrit/plugins/readonly/ReadOnlyEndpoint.java b/src/main/java/com/googlesource/gerrit/plugins/readonly/ReadOnlyEndpoint.java index 2482cab..6a4b916 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/readonly/ReadOnlyEndpoint.java +++ b/src/main/java/com/googlesource/gerrit/plugins/readonly/ReadOnlyEndpoint.java
@@ -40,8 +40,8 @@ } @Override - public String apply(ConfigResource resource) { - return state.isReadOnly() ? "on" : "off"; + public Response<String> apply(ConfigResource resource) { + return Response.ok(state.isReadOnly() ? "on" : "off"); } }
diff --git a/src/test/java/com/googlesource/gerrit/plugins/readonly/AbstractReadOnlyTest.java b/src/test/java/com/googlesource/gerrit/plugins/readonly/AbstractReadOnlyTest.java index 56be07a..b14bcce 100644 --- a/src/test/java/com/googlesource/gerrit/plugins/readonly/AbstractReadOnlyTest.java +++ b/src/test/java/com/googlesource/gerrit/plugins/readonly/AbstractReadOnlyTest.java
@@ -15,6 +15,7 @@ package com.googlesource.gerrit.plugins.readonly; import static com.google.common.truth.Truth.assertThat; +import static com.google.gerrit.testing.GerritJUnit.assertThrows; import static org.apache.http.HttpStatus.SC_SERVICE_UNAVAILABLE; import com.google.gerrit.acceptance.GitUtil; @@ -146,12 +147,9 @@ setReadOnly(true); // Push should fail - try { - pushTo("refs/for/master"); - fail("expected TransportException"); - } catch (TransportException e) { - assertThat(e).hasMessageThat().contains(expectedMessage); - } + TransportException thrown = + assertThrows(TransportException.class, () -> pushTo("refs/for/master")); + assertThat(thrown).hasMessageThat().contains(expectedMessage); // Disable read-only setReadOnly(false);