5
73%
master: 73%

Ran 10 Aug 2020 07:47AM UTC

Files 226

Run time 1min

Badge

Embed ▾

Committed 10 Aug 2020 07:42AM UTC coverage: 73.369% (+0.7%) from 72.664%

Job # RUN_TEST=coverage COMPOSER_FLAGS="--prefer-stable --prefer-lowest" ES_VERSION=73

Build Type

push

travis-ci

Committed by

web-flow

Commit Message

Allow string argument in AbstractUpdateAction::setRefresh (#1791)

Ref #1475 

Currently, `AbstractUpdateAction::setRefresh` only accepts boolean values: 
- `true` - to perform an immediate refresh
- `false` - to not refresh at all

The [documentation](https://www.elastic.co/guide/en/elasticsearch/reference/current/indices-refresh.html), however, strongly suggests the usage of `wait_for` option to ensure a good cluster performance while doing the refresh operation. But the problem is that `wait_for` is of type `string`; thus, we cannot pass it as an argument to the method.

This modifies `AbstractUpdateAction::setRefresh` so that arguments of type `string` such as `wait_for` can also be passed to the method. Additionally, the return type of its getter counterpart, `AbstractUpdateAction::getRefresh`, has also been modified to mirror the accepted types of the setter.

Run Details

3116 of 4247 relevant lines covered (73.37%)

53.82 hits per line

ruflin / Elastica / 3960 / 5
73%
master: 73%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3960.5 (RUN_TEST=coverage COMPOSER_FLAGS="--prefer-stable --prefer-lowest" ES_VERSION=73)

ruflin / Elastica / 3960 / 5 73% master: 73%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3960.5 (RUN_TEST=coverage COMPOSER_FLAGS="--prefer-stable --prefer-lowest" ES_VERSION=73)

ruflin / Elastica / 3960 / 5
73%
master: 73%

README BADGES
x