test: fix flaky influxdb test
This commit is contained in:
parent
ad28d8418d
commit
81a104690d
|
@ -990,9 +990,7 @@ t_write_failure(Config) ->
|
||||||
?assertMatch([_ | _], Trace),
|
?assertMatch([_ | _], Trace),
|
||||||
[#{result := Result} | _] = Trace,
|
[#{result := Result} | _] = Trace,
|
||||||
?assert(
|
?assert(
|
||||||
{error, {error, {closed, "The connection was lost."}}} =:= Result orelse
|
not emqx_ee_connector_influxdb:is_unrecoverable_error(Result),
|
||||||
{error, {error, closed}} =:= Result orelse
|
|
||||||
{error, {recoverable_error, econnrefused}} =:= Result,
|
|
||||||
#{got => Result}
|
#{got => Result}
|
||||||
);
|
);
|
||||||
async ->
|
async ->
|
||||||
|
@ -1000,11 +998,7 @@ t_write_failure(Config) ->
|
||||||
?assertMatch([#{action := nack} | _], Trace),
|
?assertMatch([#{action := nack} | _], Trace),
|
||||||
[#{result := Result} | _] = Trace,
|
[#{result := Result} | _] = Trace,
|
||||||
?assert(
|
?assert(
|
||||||
{error, {recoverable_error, {closed, "The connection was lost."}}} =:=
|
not emqx_ee_connector_influxdb:is_unrecoverable_error(Result),
|
||||||
Result orelse
|
|
||||||
{error, {error, closed}} =:= Result orelse
|
|
||||||
{error, {recoverable_error, econnrefused}} =:= Result orelse
|
|
||||||
{error, {recoverable_error, noproc}} =:= Result,
|
|
||||||
#{got => Result}
|
#{got => Result}
|
||||||
)
|
)
|
||||||
end,
|
end,
|
||||||
|
|
|
@ -35,6 +35,9 @@
|
||||||
desc/1
|
desc/1
|
||||||
]).
|
]).
|
||||||
|
|
||||||
|
%% only for test
|
||||||
|
-export([is_unrecoverable_error/1]).
|
||||||
|
|
||||||
-type ts_precision() :: ns | us | ms | s.
|
-type ts_precision() :: ns | us | ms | s.
|
||||||
|
|
||||||
%% influxdb servers don't need parse
|
%% influxdb servers don't need parse
|
||||||
|
@ -655,12 +658,6 @@ str(S) when is_list(S) ->
|
||||||
|
|
||||||
is_unrecoverable_error({error, {unrecoverable_error, _}}) ->
|
is_unrecoverable_error({error, {unrecoverable_error, _}}) ->
|
||||||
true;
|
true;
|
||||||
is_unrecoverable_error({error, {recoverable_error, _}}) ->
|
|
||||||
false;
|
|
||||||
is_unrecoverable_error({error, {error, econnrefused}}) ->
|
|
||||||
false;
|
|
||||||
is_unrecoverable_error({error, econnrefused}) ->
|
|
||||||
false;
|
|
||||||
is_unrecoverable_error(_) ->
|
is_unrecoverable_error(_) ->
|
||||||
false.
|
false.
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue