From f5e9d3eab282b98b5605fd295267d87c6ea0add5 Mon Sep 17 00:00:00 2001
From: "Zaiming (Stone) Shi" <zmstone@gmail.com>
Date: Tue, 28 Mar 2023 14:28:49 +0200
Subject: [PATCH] test: use valid config for jq function timeout

---
 .../test/emqx_rule_funcs_SUITE.erl            | 20 +++++--------------
 1 file changed, 5 insertions(+), 15 deletions(-)

diff --git a/apps/emqx_rule_engine/test/emqx_rule_funcs_SUITE.erl b/apps/emqx_rule_engine/test/emqx_rule_funcs_SUITE.erl
index 5d78f5e4a..209332fe7 100644
--- a/apps/emqx_rule_engine/test/emqx_rule_funcs_SUITE.erl
+++ b/apps/emqx_rule_engine/test/emqx_rule_funcs_SUITE.erl
@@ -28,7 +28,7 @@
 
 init_per_suite(Config) ->
     application:load(emqx_conf),
-    ConfigConf = <<"rule_engine {jq_function_default_timeout {}}">>,
+    ConfigConf = <<"rule_engine {jq_function_default_timeout=10s}">>,
     ok = emqx_common_test_helpers:load_config(emqx_rule_engine_schema, ConfigConf),
     Config.
 
@@ -691,20 +691,10 @@ t_jq(_) ->
         ConfigRootKey,
         jq_function_default_timeout
     ]),
-    case DefaultTimeOut =< 15000 of
-        true ->
-            got_timeout =
-                try
-                    apply_func(jq, [TOProgram, <<"-2">>])
-                catch
-                    throw:{jq_exception, {timeout, _}} ->
-                        %% Got timeout as expected
-                        got_timeout
-                end;
-        false ->
-            %% Skip test as we don't want it to take to long time to run
-            ok
-    end.
+    ?assertThrow(
+        {jq_exception, {timeout, _}},
+        apply_func(jq, [TOProgram, <<"-2">>])
+    ).
 
 ascii_string() -> list(range(0, 127)).