diff options
author | Håkon Hallingstad <hakon@verizonmedia.com> | 2020-02-05 17:27:47 +0100 |
---|---|---|
committer | Håkon Hallingstad <hakon@verizonmedia.com> | 2020-02-05 17:27:47 +0100 |
commit | 1193b4a06618a0b5f092aedfeb65af37b11fa0e3 (patch) | |
tree | 5d44af9fc0798ae65c0107f4aeee2596943ad5a1 /vespaclient-container-plugin | |
parent | d05911599c0b9706d54ff7819349cbcadd30ed0a (diff) |
Reduce access logging
Avoids writing access logs in various tests.
1. Disables by-default access logging with Application, since it is used in
unit tests. 2. However many tests create additional DeployState which renders
this ineffective, and so this PR also explicitly disables access logging in
services.xml of some tests.
(1) might be unnecessary if we anyway have to do (2) everywhere, but this is
not clear to me.
Diffstat (limited to 'vespaclient-container-plugin')
-rw-r--r-- | vespaclient-container-plugin/src/test/java/com/yahoo/document/restapi/resource/RestApiTest.java | 2 | ||||
-rw-r--r-- | vespaclient-container-plugin/src/test/rest-api-application/services.xml | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/vespaclient-container-plugin/src/test/java/com/yahoo/document/restapi/resource/RestApiTest.java b/vespaclient-container-plugin/src/test/java/com/yahoo/document/restapi/resource/RestApiTest.java index aeddc762586..0661363477f 100644 --- a/vespaclient-container-plugin/src/test/java/com/yahoo/document/restapi/resource/RestApiTest.java +++ b/vespaclient-container-plugin/src/test/java/com/yahoo/document/restapi/resource/RestApiTest.java @@ -202,7 +202,7 @@ public class RestApiTest { // Get logs through some hackish fetch method. Logs is something the mocked backend write. String getLog() throws IOException { - // The mocked backend will throw a runtime exception wtih a log if delete is called three times.. + // The mocked backend will throw a runtime exception with a log if delete is called three times.. Request request = new Request("http://localhost:" + getFirstListenPort() + remove_test_uri); HttpDelete delete = new HttpDelete(request.getUri()); doRest(delete); diff --git a/vespaclient-container-plugin/src/test/rest-api-application/services.xml b/vespaclient-container-plugin/src/test/rest-api-application/services.xml index 346740bc815..ae1b87635a9 100644 --- a/vespaclient-container-plugin/src/test/rest-api-application/services.xml +++ b/vespaclient-container-plugin/src/test/rest-api-application/services.xml @@ -2,6 +2,8 @@ <!-- Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. --> <container version="1.0" jetty="true"> + <accesslog type="disabled"/> + <handler id="com.yahoo.document.restapi.resource.RestApiWithTestDocumentHandler" bundle="integration-test"> <binding>http://*/document/v1/*</binding> </handler> |