murong00 commented on a change in pull request #8103: URL: https://github.com/apache/pulsar/pull/8103#discussion_r524909204
########## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminTest.java ########## @@ -455,9 +455,11 @@ public void properties() throws Exception { assertEquals(e.getResponse().getStatus(), Status.NOT_FOUND.getStatusCode()); } + AsyncResponse response = mock(AsyncResponse.class); + // Check deleting non-existing property try { - properties.deleteTenant("non-existing"); + properties.deleteTenant(response, "non-existing", false); Review comment: This change adds a force option so the test need pass this option. I can add a test to cover the above situations later. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org