yuanbiao
/
emqx
1
0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #12337 from SergeTupchiy/refactor-emqx-cm-test-flush 

test(emqx_cm_SUITE): use one helper function `emqx_pool:flush_async_tasks/1`
This commit is contained in:
SergeTupchiy 2024-01-16 20:15:16 +02:00 committed by GitHub
parent ed78f2488b 80e82db282
commit 23c0dc58fb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
apps/emqx/test/emqx_cm_SUITE.erl
View File

@ -343,31 +343,9 @@ test_stepdown_session(Action, Reason) ->
end,
% sync
ignored = gen_server:call(?CM, ignore, infinity),
ok = flush_emqx_cm_pool(),
ok = emqx_pool:flush_async_tasks(?CM_POOL),
?assertEqual([], emqx_cm:lookup_channels(ClientId)).
%% Channel deregistration is delegated to emqx_pool as a sync tasks.
%% The emqx_pool is pool of workers, and there is no way to know
%% which worker was picked for the last deregistration task.
%% This help function creates a large enough number of async tasks
%% to sync with the pool workers.
%% The number of tasks should be large enough to ensure all workers have
%% the chance to work on at least one of the tasks.
flush_emqx_cm_pool() ->
Self = self(),
L = lists:seq(1, 1000),
lists:foreach(
fun(I) -> emqx_pool:async_submit_to_pool(?CM_POOL, fun() -> Self ! {done, I} end, []) end, L
),
lists:foreach(
fun(I) ->
receive
{done, I} -> ok
end
end,
L
).
t_discard_session_race(_) ->
ClientId = rand_client_id(),
?check_trace(