33313df0a2
Add ThreadLeakLingering option to Rest client tests
...
Some Rest tests use the Sun HTTP server which has lingering threads after shutdown. Similar to ESTestCase, this adds an
option to wait 5 seconds for these threads to terminate.
2016-06-23 10:21:37 +02:00
0bf447c697
Group client projects under :client
...
:client ---------> :client:rest
:client-sniffer -> :client:sniffer
:client-test ----> :client:test
This lines the client up with how we do things like modules and
plugins.
2016-06-22 14:26:41 -04:00
c2839c1577
[TEST] Add client-test module and make client tests use randomized runner directly
...
The lucene-test dependency caused issues with IDEs as they would always load the lucene 5 jar although they shouldn't have, which caused jarhell in es core tests.
If we depend directly on randomized runner we don't have this problem. It is luckily still compatible with java 1.7. This requires though adding a thin module that includes the base test class which can be shared between client and client-sniffer.
2016-06-22 19:37:08 +02:00
f0b6abe439
rename onSuccess to onResponse
...
That makes it a bit clearer that it's about the response and whether we decide if it was a good one or a failure (based on status code)
2016-06-21 17:11:09 +02:00
cb4bfcb864
Take SniffOnFailureListener out of Sniffer and make FailureListener final on RestClient
2016-06-21 15:29:59 +02:00
8c60374284
Build: do not load integ test class if --skip-integ-tests-in-disguise is specified in NamingConventionsCheck
...
Projects that don't depend on elasticsearch-test fail otherwise because org.elasticsearch.test.EsIntegTestCase (default integ test class) is not in the classpath. They should provide their onw integ test base class, but having integration tests should not be mandatory. One can simply set skipIntegTestsInDisguise to true to prevent loading of integ test class.
2016-06-17 13:46:27 +02:00
7c8013f9fd
Build: remove explicit targetCompatibility from forbiddenapis config
...
targetCompatibility set to the project is enough.
2016-06-15 11:43:48 +02:00
70fb67cc7b
Build: targetCompatibility and sourceCompatibility are enough to make sure we compile client and client-sniffer with target and source 1.7
2016-06-14 23:16:38 +02:00
cf93e904e3
remove message parameter from RequestLogger methods
...
This prevents useless string allocation.
2016-06-14 14:14:03 +02:00
1932f6bc7c
Rename RequestLogger#log methods to distinguish between the two
...
One method is to log a request that yielded a response, the other one for a failed request
2016-06-14 13:45:38 +02:00
caa6c96259
Build: make client and client-sniffer depend on lucene-test version 5, last 1.7 compatible version
2016-06-14 09:43:31 +02:00
116805b28b
remove TODO around copying hosts when rotating the collection, it's not a problem for now
2016-06-13 14:02:01 +02:00
8f7b7fb813
added comments to clarify RequestLogger and DeadHostState
2016-06-13 12:59:30 +02:00
3cd201e67e
[TEST] add comment on using animal-sniffer suppress annotation
2016-06-13 12:02:09 +02:00
50b6f4c02f
Build: changed forbidden-apis targetCompatibility to 1.7 for client and client-sniffer
2016-06-13 09:20:39 +02:00
777d438f48
[TEST] use jdk-internal bundled signature (rather than the previously removed jdk-non-portable)
2016-06-10 12:46:33 +02:00
3d7186c81f
make DeadHostState final
2016-06-10 11:40:35 +02:00
6db90da25e
[TEST] Remove usage of @SuppressForbidden due to sun HttpServer usage, resolve some violations that were hidden by it
2016-06-10 11:40:13 +02:00
a5e329e563
[TEST] use animalsniffer annotation @IgnoreJRERequirement (similar to @SuppressForbidden) rather than exclusion pattern
2016-06-10 10:11:21 +02:00
0af9d2c767
Build: add animalsniffer to detect usage of java8 apis in client and client-sniffer
2016-06-10 09:51:47 +02:00
f38ce72004
make forbiddenApisTest work for client and client-sniffer
2016-06-09 23:44:34 +02:00
bd773359d5
[TEST] add SuppresForbidden annotation for client project
2016-06-09 17:39:52 +02:00
d8c0fad08f
fix failing tests
2016-06-09 17:06:58 +02:00
9a4971d3fd
fix line length
2016-06-09 16:57:57 +02:00
de8b9fd579
use switch for status codes and add comments
2016-06-09 16:51:08 +02:00
cbdffc7965
s/elasticsearchResponse/response
2016-06-09 16:39:59 +02:00
853ea9385b
add comments on retries
2016-06-09 16:38:43 +02:00
742f9c6eaa
nextHost to return Iterable<HttpHost> rather than Iterator
2016-06-09 16:34:01 +02:00
c028bf9250
rename deadUntil to deadUntilNanos
2016-06-09 16:31:09 +02:00
85606e8a4b
renamed all time intervals arguments and members to include the time unit in their name
2016-06-09 16:27:52 +02:00
437c4f210b
rename ElasticsearchResponse to Response and ElasticsearchResponseException to ResponseException
2016-06-09 14:38:32 +02:00
04d620da74
require hosts when creating RestClient.Builder
...
Also fix order of arguments when using assertEquals
2016-06-08 12:37:50 +02:00
2cf04c0877
wrap entity only when not repeatable and improve RequestLoggerTests
2016-06-06 15:41:11 +02:00
1f7f6e2709
wrap log statement with logger.isDebugEnabled
2016-06-06 15:13:42 +02:00
a461dd84d2
Build: add hamcrest and securemock to version.properties
2016-06-06 15:02:52 +02:00
05e26a46d7
raise default socket timeout to 10s and default connect timeout to 1s
2016-06-04 01:06:18 +02:00
f2318ed5ae
Build: add missing licenses, SHAs and enable dependency licenses check
2016-06-04 00:56:42 +02:00
b15279b5ef
Allow to pass socket facttry registry to createDefaultHttpClient method
2016-06-03 23:59:26 +02:00
f17f0f9247
rename ElasticsearchResponse#getFirstHeader to getHeader
2016-06-03 18:28:31 +02:00
29eee328fe
[TEST] expand RequestLoggerTests to all the supported http methods
2016-06-03 18:23:52 +02:00
13a27a34f8
[TEST] add RestClient integration test
...
Relies on real HttpServer to test the actual interaction between RestClient and HttpClient, and how requests get sent in real life.
2016-06-03 16:20:12 +02:00
4572b69011
[TEST] add RestClient unit tests
...
Unit tests rely on mockito to mock the internal HttpClient instance. No http request is performed, we only simulate interaction between RestClient and its internal HttpClient.
2016-06-03 16:20:12 +02:00
9ed2d610ec
[TEST] rename restClientTests back to RestClientBuilderTests
2016-06-03 16:20:12 +02:00
24ea585c9e
don't use setDefaultHeaders from HttpClient
...
Store default headers ourselves instead, otherwise default ones cannot be replaced. Don't allow for multiple headers with same key, last one wins and replaces previous ones with same key.
Also fail with null params or headers.
2016-06-03 16:20:11 +02:00
47e52044e4
[TEST] add setHosts test and rename RestClientBuilderTests to RestClientTests
2016-06-03 16:01:07 +02:00
35dbdeeae5
check hosts is not null nor empty earlier, remove check from nextHost
...
if we check at set time, we don't need to check each single time in nextHost
2016-06-03 16:01:07 +02:00
6d66fbd9c1
add toString to DeadHostState class
2016-06-03 16:01:07 +02:00
c9db111387
add javadocs on closing responses
2016-06-03 16:01:07 +02:00
83c6e736de
add support for PATCH and TRACE methods
...
Although elasticsearch doesn't support these methods (RestController doesn't even allow to register handler for them), the RestClient should allow to send requests using them.
2016-06-03 16:01:07 +02:00
51e487fa55
[TEST] remove okhttp test dependency
...
Use sun HttpServer instead and disable forbidden-apis for test classes. It turns out to be more flexible than okhttp as it allows get & delete with body.
2016-06-03 16:01:07 +02:00
Yannick Welsch
Nik Everett
javanna