From d942a531e638310619dd378578b2666ef4932ac7 Mon Sep 17 00:00:00 2001 From: Andrew Mayorov Date: Thu, 23 Mar 2023 13:23:56 +0300 Subject: [PATCH] fix(test): avoid unnecessary cleanups This might make some noise in the logs, depending on the testsuite configuration, because some test setup might be missing required applications. --- apps/emqx_bridge/test/emqx_bridge_api_SUITE.erl | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/apps/emqx_bridge/test/emqx_bridge_api_SUITE.erl b/apps/emqx_bridge/test/emqx_bridge_api_SUITE.erl index fdbeb1be4..8899cd24a 100644 --- a/apps/emqx_bridge/test/emqx_bridge_api_SUITE.erl +++ b/apps/emqx_bridge/test/emqx_bridge_api_SUITE.erl @@ -174,7 +174,6 @@ end_per_group(cluster, Config) -> ); end_per_group(_, _Config) -> emqx_mgmt_api_test_util:end_suite(?SUITE_APPS), - mria:clear_table(emqx_authn_mnesia), ok. init_per_testcase(t_broken_bpapi_vsn, Config) -> @@ -1070,6 +1069,10 @@ t_bridges_probe(Config) -> Config ), + emqx_common_test_helpers:on_exit(fun() -> + delete_user_auth(Chain, AuthenticatorID, User, Config) + end), + ?assertMatch( {ok, 400, #{ <<"code">> := <<"TEST_FAILED">>, @@ -1290,6 +1293,10 @@ add_user_auth(Chain, AuthenticatorID, User, Config) -> Node = ?config(api_node, Config), erpc:call(Node, emqx_authentication, add_user, [Chain, AuthenticatorID, User]). +delete_user_auth(Chain, AuthenticatorID, User, Config) -> + Node = ?config(api_node, Config), + erpc:call(Node, emqx_authentication, delete_user, [Chain, AuthenticatorID, User]). + str(S) when is_list(S) -> S; str(S) when is_binary(S) -> binary_to_list(S).