Remove tests that used a NUL byte to assert invalid configs
These tests are not compatible with Jetty 9.4.43.v20210629.
Change-Id: I0d1c04db26571c6d5c2f02e0f47aff6279b424e5
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/AbstractGetCodeOwnersForPathRestIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/AbstractGetCodeOwnersForPathRestIT.java
index 7aa024d..9b675e8 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/AbstractGetCodeOwnersForPathRestIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/AbstractGetCodeOwnersForPathRestIT.java
@@ -65,13 +65,6 @@
protected abstract String getUrl(String path);
@Test
- public void getCodeOwnerConfigForInvalidPath() throws Exception {
- RestResponse r = adminRestSession.get(getUrl("\0"));
- r.assertBadRequest();
- assertThat(r.getEntityContent()).contains("Nul character not allowed");
- }
-
- @Test
public void getCodeOwnersWithAccountOptions() throws Exception {
codeOwnerConfigOperations
.newCodeOwnerConfig()
diff --git a/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnerConfigForPathInBranchRestIT.java b/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnerConfigForPathInBranchRestIT.java
index 8ae7b98..2df8529 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnerConfigForPathInBranchRestIT.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/acceptance/restapi/GetCodeOwnerConfigForPathInBranchRestIT.java
@@ -49,20 +49,6 @@
}
@Test
- @GerritConfig(name = "plugin.code-owners.enableExperimentalRestEndpoints", value = "true")
- public void getCodeOwnerConfigsForInvalidPath() throws Exception {
- RestResponse r =
- adminRestSession.get(
- String.format(
- "/projects/%s/branches/%s/code_owners.config/%s",
- IdString.fromDecoded(project.get()),
- IdString.fromDecoded("master"),
- IdString.fromDecoded("\0")));
- r.assertBadRequest();
- assertThat(r.getEntityContent()).contains("Nul character not allowed");
- }
-
- @Test
@GerritConfig(name = "plugin.code-owners.backend", value = "non-existing-backend")
@GerritConfig(name = "plugin.code-owners.enableExperimentalRestEndpoints", value = "true")
public void cannotGetCodeOwnerConfigIfPluginConfigurationIsInvalid() throws Exception {
diff --git a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerResolverTest.java b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerResolverTest.java
index 06e5ab1..dbf1012 100644
--- a/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerResolverTest.java
+++ b/javatests/com/google/gerrit/plugins/codeowners/backend/CodeOwnerResolverTest.java
@@ -456,8 +456,7 @@
}
@Test
- public void resolvePathCodeOwnersWithAnnotations_annotationOnAllUsersWildcard()
- throws Exception {
+ public void resolvePathCodeOwnersWithAnnotations_annotationOnAllUsersWildcard() throws Exception {
CodeOwnerConfig codeOwnerConfig =
CodeOwnerConfig.builder(CodeOwnerConfig.Key.create(project, "master", "/"), TEST_REVISION)
.addCodeOwnerSet(