From 5e2bc7853f3233fecc8bee955d00acf00336b97e Mon Sep 17 00:00:00 2001
From: zhouzb <zhouzb@emqx.io>
Date: Thu, 13 Jan 2022 16:53:47 +0800
Subject: [PATCH] test: fix test cases for pgsql authz

---
 apps/emqx_authz/test/emqx_authz_postgresql_SUITE.erl | 2 +-
 apps/emqx_authz/test/emqx_authz_test_lib.erl         | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/apps/emqx_authz/test/emqx_authz_postgresql_SUITE.erl b/apps/emqx_authz/test/emqx_authz_postgresql_SUITE.erl
index 6ececd3f8..794f5a8cf 100644
--- a/apps/emqx_authz/test/emqx_authz_postgresql_SUITE.erl
+++ b/apps/emqx_authz/test/emqx_authz_postgresql_SUITE.erl
@@ -167,7 +167,7 @@ t_pgsql_error(_Config) ->
                    listener => {tcp, default}
                   },
 
-    ok = setup_config(
+    {error, _} = setup_config(
       #{<<"query">> => <<"SOME INVALID STATEMENT">>}),
 
     ok = emqx_authz_test_lib:test_samples(
diff --git a/apps/emqx_authz/test/emqx_authz_test_lib.erl b/apps/emqx_authz/test/emqx_authz_test_lib.erl
index bacbbe10a..b7de5c3da 100644
--- a/apps/emqx_authz/test/emqx_authz_test_lib.erl
+++ b/apps/emqx_authz/test/emqx_authz_test_lib.erl
@@ -40,8 +40,10 @@ reset_authorizers(Nomatch, ChacheEnabled) ->
 
 setup_config(BaseConfig, SpecialParams) ->
     Config = maps:merge(BaseConfig, SpecialParams),
-    {ok, _} = emqx_authz:update(?CMD_REPLACE, [Config]),
-    ok.
+    case emqx_authz:update(?CMD_REPLACE, [Config]) of
+      {ok, _} -> ok;
+      {error, Reason} -> {error, Reason}
+    end.
 
 is_tcp_server_available(Host, Port) ->
     case gen_tcp:connect(Host, Port, [], ?DEFAULT_CHECK_AVAIL_TIMEOUT) of