Browse Source

Rename remaining groups to pg.

Roberto Ostinelli 3 years ago
parent
commit
ea5a2ff33d
3 changed files with 79 additions and 79 deletions
  1. 35 35
      src/syn_pg.erl
  2. 42 42
      test/syn_pg_SUITE.erl
  3. 2 2
      test/syn_test_suite_helper.erl

+ 35 - 35
src/syn_pg.erl

@@ -80,7 +80,7 @@ is_member(Scope, GroupName, Pid) ->
             error({invalid_scope, Scope});
 
         TableByName ->
-            case find_groups_entry_by_name_and_pid(GroupName, Pid, TableByName) of
+            case find_pg_entry_by_name_and_pid(GroupName, Pid, TableByName) of
                 undefined -> false;
                 _ -> true
             end
@@ -111,7 +111,7 @@ is_local_member(Scope, GroupName, Pid) ->
             error({invalid_scope, Scope});
 
         TableByName ->
-            case find_groups_entry_by_name_and_pid(GroupName, Pid, TableByName) of
+            case find_pg_entry_by_name_and_pid(GroupName, Pid, TableByName) of
                 {{_, _}, _, _, _, Node} when Node =:= node() -> true;
                 _ -> false
             end
@@ -182,12 +182,12 @@ group_names_ordset(Scope, NodeParam) ->
             error({invalid_scope, Scope});
 
         TableByName ->
-            Groups = ets:select(TableByName, [{
+            DuplicatedGroups = ets:select(TableByName, [{
                 {{'$1', '_'}, '_', '_', '_', NodeParam},
                 [],
                 ['$1']
             }]),
-            ordsets:from_list(Groups)
+            ordsets:from_list(DuplicatedGroups)
     end.
 
 -spec publish(Scope :: atom(), GroupName :: term(), Message :: term()) -> {ok, RecipientCount :: non_neg_integer()}.
@@ -240,7 +240,7 @@ init(#state{
     table_by_pid = TableByPid
 }) ->
     %% purge remote & rebuild
-    purge_groups_for_remote_nodes(Scope, TableByName, TableByPid),
+    purge_pg_for_remote_nodes(Scope, TableByName, TableByPid),
     rebuild_monitors(Scope, TableByName, TableByPid),
     %% init
     HandlerState = #{},
@@ -262,7 +262,7 @@ handle_call({'3.0', join_on_node, RequesterNode, GroupName, Pid, Meta}, _From, #
 } = State) ->
     case is_process_alive(Pid) of
         true ->
-            case find_groups_entry_by_name_and_pid(GroupName, Pid, TableByName) of
+            case find_pg_entry_by_name_and_pid(GroupName, Pid, TableByName) of
                 undefined ->
                     %% add
                     MRef = case find_monitor_for_pid(Pid, TableByPid) of
@@ -288,7 +288,7 @@ handle_call({'3.0', leave_on_node, RequesterNode, GroupName, Pid}, _From, #state
     table_by_name = TableByName,
     table_by_pid = TableByPid
 } = State) ->
-    case find_groups_entry_by_name_and_pid(GroupName, Pid, TableByName) of
+    case find_pg_entry_by_name_and_pid(GroupName, Pid, TableByName) of
         undefined ->
             {reply, {{error, not_in_group}, undefined}, State};
 
@@ -319,7 +319,7 @@ handle_call(Request, From, #state{scope = Scope} = State) ->
 handle_info({'3.0', sync_join, GroupName, Pid, Meta, Time, Reason}, #state{nodes_map = NodesMap} = State) ->
     case maps:is_key(node(Pid), NodesMap) of
         true ->
-            handle_groups_sync(GroupName, Pid, Meta, Time, Reason, State);
+            handle_pg_sync(GroupName, Pid, Meta, Time, Reason, State);
 
         false ->
             %% ignore, race condition
@@ -332,7 +332,7 @@ handle_info({'3.0', sync_leave, GroupName, Pid, Meta, Reason}, #state{
     table_by_name = TableByName,
     table_by_pid = TableByPid
 } = State) ->
-    case find_groups_entry_by_name_and_pid(GroupName, Pid, TableByName) of
+    case find_pg_entry_by_name_and_pid(GroupName, Pid, TableByName) of
         undefined ->
             %% not in table, nothing to do
             ok;
@@ -351,7 +351,7 @@ handle_info({'DOWN', _MRef, process, Pid, Reason}, #state{
     table_by_name = TableByName,
     table_by_pid = TableByPid
 } = State) ->
-    case find_groups_entries_by_pid(Pid, TableByPid) of
+    case find_pg_entries_by_pid(Pid, TableByPid) of
         [] ->
             error_logger:warning_msg(
                 "SYN[~s<~s>] Received a DOWN message from an unknown process ~p with reason: ~p",
@@ -380,14 +380,14 @@ handle_info(Info, #state{scope = Scope} = State) ->
 %% ----------------------------------------------------------------------------------------------------------
 -spec get_local_data(State :: term()) -> {ok, Data :: term()} | undefined.
 get_local_data(#state{table_by_name = TableByName}) ->
-    {ok, get_groups_tuples_for_node(node(), TableByName)}.
+    {ok, get_pg_tuples_for_node(node(), TableByName)}.
 
 -spec save_remote_data(RemoteData :: term(), State :: term()) -> any().
-save_remote_data(GroupsTuplesOfRemoteNode, #state{scope = Scope} = State) ->
+save_remote_data(PgTuplesOfRemoteNode, #state{scope = Scope} = State) ->
     %% insert tuples
     lists:foreach(fun({GroupName, Pid, Meta, Time}) ->
-        handle_groups_sync(GroupName, Pid, Meta, Time, {syn_remote_scope_node_up, Scope, node(Pid)}, State)
-    end, GroupsTuplesOfRemoteNode).
+        handle_pg_sync(GroupName, Pid, Meta, Time, {syn_remote_scope_node_up, Scope, node(Pid)}, State)
+    end, PgTuplesOfRemoteNode).
 
 -spec purge_local_data_for_node(Node :: node(), State :: term()) -> any().
 purge_local_data_for_node(Node, #state{
@@ -395,15 +395,15 @@ purge_local_data_for_node(Node, #state{
     table_by_name = TableByName,
     table_by_pid = TableByPid
 }) ->
-    purge_groups_for_remote_node(Scope, Node, TableByName, TableByPid).
+    purge_pg_for_remote_node(Scope, Node, TableByName, TableByPid).
 
 %% ===================================================================
 %% Internal
 %% ===================================================================
 -spec rebuild_monitors(Scope :: atom(), TableByName :: atom(), TableByPid :: atom()) -> ok.
 rebuild_monitors(Scope, TableByName, TableByPid) ->
-    GroupsTuples = get_groups_tuples_for_node(node(), TableByName),
-    do_rebuild_monitors(GroupsTuples, #{}, Scope, TableByName, TableByPid).
+    PgTuples = get_pg_tuples_for_node(node(), TableByName),
+    do_rebuild_monitors(PgTuples, #{}, Scope, TableByName, TableByPid).
 
 -spec do_rebuild_monitors(
     [syn_pg_tuple()],
@@ -471,8 +471,8 @@ do_join_on_node(GroupName, Pid, Meta, MRef, Reason, RequesterNode, CallbackMetho
     %% return
     {reply, {ok, {CallbackMethod, Time, TableByName, TableByPid}}, State}.
 
--spec get_groups_tuples_for_node(Node :: node(), TableByName :: atom()) -> [syn_pg_tuple()].
-get_groups_tuples_for_node(Node, TableByName) ->
+-spec get_pg_tuples_for_node(Node :: node(), TableByName :: atom()) -> [syn_pg_tuple()].
+get_pg_tuples_for_node(Node, TableByName) ->
     ets:select(TableByName, [{
         {{'$1', '$2'}, '$3', '$4', '_', Node},
         [],
@@ -492,16 +492,16 @@ find_monitor_for_pid(Pid, TableByPid) when is_pid(Pid) ->
         '$end_of_table' -> undefined
     end.
 
--spec find_groups_entry_by_name_and_pid(GroupName :: term(), Pid :: pid(), TableByName :: atom()) ->
+-spec find_pg_entry_by_name_and_pid(GroupName :: term(), Pid :: pid(), TableByName :: atom()) ->
     Entry :: syn_pg_entry() | undefined.
-find_groups_entry_by_name_and_pid(GroupName, Pid, TableByName) ->
+find_pg_entry_by_name_and_pid(GroupName, Pid, TableByName) ->
     case ets:lookup(TableByName, {GroupName, Pid}) of
         [] -> undefined;
         [Entry] -> Entry
     end.
 
--spec find_groups_entries_by_pid(Pid :: pid(), TableByPid :: atom()) -> GroupEntries :: [syn_pg_entry()].
-find_groups_entries_by_pid(Pid, TableByPid) when is_pid(Pid) ->
+-spec find_pg_entries_by_pid(Pid :: pid(), TableByPid :: atom()) -> GroupEntries :: [syn_pg_entry()].
+find_pg_entries_by_pid(Pid, TableByPid) when is_pid(Pid) ->
     ets:select(TableByPid, [{
         {{Pid, '_'}, '_', '_', '_', '_'},
         [],
@@ -551,34 +551,34 @@ remove_from_local_table(GroupName, Pid, TableByName, TableByPid) ->
     true = ets:delete(TableByName, {GroupName, Pid}),
     true = ets:delete(TableByPid, {Pid, GroupName}).
 
--spec purge_groups_for_remote_nodes(Scope :: atom(), TableByName :: atom(), TableByPid :: atom()) -> any().
-purge_groups_for_remote_nodes(Scope, TableByName, TableByPid) ->
+-spec purge_pg_for_remote_nodes(Scope :: atom(), TableByName :: atom(), TableByPid :: atom()) -> any().
+purge_pg_for_remote_nodes(Scope, TableByName, TableByPid) ->
     LocalNode = node(),
-    RemoteNodesWithDoubles = ets:select(TableByName, [{
+    DuplicatedRemoteNodes = ets:select(TableByName, [{
         {{'_', '_'}, '_', '_', '_', '$6'},
         [{'=/=', '$6', LocalNode}],
         ['$6']
     }]),
-    RemoteNodes = ordsets:from_list(RemoteNodesWithDoubles),
+    RemoteNodes = ordsets:from_list(DuplicatedRemoteNodes),
     ordsets:fold(fun(RemoteNode, _) ->
-        purge_groups_for_remote_node(Scope, RemoteNode, TableByName, TableByPid)
+        purge_pg_for_remote_node(Scope, RemoteNode, TableByName, TableByPid)
     end, undefined, RemoteNodes).
 
--spec purge_groups_for_remote_node(Scope :: atom(), Node :: atom(), TableByName :: atom(), TableByPid :: atom()) -> true.
-purge_groups_for_remote_node(Scope, Node, TableByName, TableByPid) when Node =/= node() ->
+-spec purge_pg_for_remote_node(Scope :: atom(), Node :: atom(), TableByName :: atom(), TableByPid :: atom()) -> true.
+purge_pg_for_remote_node(Scope, Node, TableByName, TableByPid) when Node =/= node() ->
     %% loop elements for callback in a separate process to free scope process
-    GroupsTuples = get_groups_tuples_for_node(Node, TableByName),
+    PgTuples = get_pg_tuples_for_node(Node, TableByName),
     spawn(fun() ->
         lists:foreach(fun({GroupName, Pid, Meta, _Time}) ->
             syn_event_handler:call_event_handler(on_process_left,
                 [Scope, GroupName, Pid, Meta, {syn_remote_scope_node_down, Scope, Node}]
             )
-        end, GroupsTuples)
+        end, PgTuples)
     end),
     ets:match_delete(TableByName, {{'_', '_'}, '_', '_', '_', Node}),
     ets:match_delete(TableByPid, {{'_', '_'}, '_', '_', '_', Node}).
 
--spec handle_groups_sync(
+-spec handle_pg_sync(
     GroupName :: term(),
     Pid :: pid(),
     Meta :: term(),
@@ -586,12 +586,12 @@ purge_groups_for_remote_node(Scope, Node, TableByName, TableByPid) when Node =/=
     Reason :: term(),
     #state{}
 ) -> any().
-handle_groups_sync(GroupName, Pid, Meta, Time, Reason, #state{
+handle_pg_sync(GroupName, Pid, Meta, Time, Reason, #state{
     scope = Scope,
     table_by_name = TableByName,
     table_by_pid = TableByPid
 }) ->
-    case find_groups_entry_by_name_and_pid(GroupName, Pid, TableByName) of
+    case find_pg_entry_by_name_and_pid(GroupName, Pid, TableByName) of
         undefined ->
             %% new
             add_to_local_table(GroupName, Pid, Meta, Time, undefined, TableByName, TableByPid),

+ 42 - 42
test/syn_pg_SUITE.erl

@@ -66,7 +66,7 @@
 %% -------------------------------------------------------------------
 all() ->
     [
-        {group, three_nodes_groups}
+        {group, three_nodes_pg}
     ].
 
 %% -------------------------------------------------------------------
@@ -83,7 +83,7 @@ all() ->
 %% -------------------------------------------------------------------
 groups() ->
     [
-        {three_nodes_groups, [shuffle], [
+        {three_nodes_pg, [shuffle], [
             three_nodes_discover,
             three_nodes_join_leave_and_monitor,
             three_nodes_join_filter_unknown_node,
@@ -119,7 +119,7 @@ end_per_suite(_Config) ->
 %% Config0 = Config1 = [tuple()]
 %% Reason = any()
 %% -------------------------------------------------------------------
-init_per_group(three_nodes_groups, Config) ->
+init_per_group(three_nodes_pg, Config) ->
     %% start slave
     {ok, SlaveNode1} = syn_test_suite_helper:start_slave(syn_slave_1),
     {ok, SlaveNode2} = syn_test_suite_helper:start_slave(syn_slave_2),
@@ -134,7 +134,7 @@ init_per_group(three_nodes_groups, Config) ->
 
         Other ->
             ct:pal("*********** Could not get full cluster, skipping"),
-            end_per_group(three_nodes_groups, Config),
+            end_per_group(three_nodes_pg, Config),
             {skip, Other}
     end;
 
@@ -147,7 +147,7 @@ init_per_group(_GroupName, Config) ->
 %% GroupName = atom()
 %% Config0 = Config1 = [tuple()]
 %% -------------------------------------------------------------------
-end_per_group(three_nodes_groups, Config) ->
+end_per_group(three_nodes_pg, Config) ->
     SlaveNode1 = proplists:get_value(slave_node_1, Config),
     syn_test_suite_helper:connect_node(SlaveNode1),
     SlaveNode2 = proplists:get_value(slave_node_2, Config),
@@ -203,14 +203,14 @@ three_nodes_discover(Config) ->
     {'EXIT', {{invalid_scope, custom_abcdef}, _}} = catch syn_registry:subcluster_nodes(custom_abcdef),
 
     %% check
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_ab, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_all, [SlaveNode1, SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_ab, [node()]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_bc, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_c, []),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_all, [node(), SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_ab, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_all, [SlaveNode1, SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_ab, [node()]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_bc, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_c, []),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_all, [node(), SlaveNode1]),
 
     %% disconnect node 2 (node 1 can still see node 2)
     syn_test_suite_helper:disconnect_node(SlaveNode2),
@@ -218,11 +218,11 @@ three_nodes_discover(Config) ->
     syn_test_suite_helper:assert_cluster(SlaveNode1, [node(), SlaveNode2]),
 
     %% check
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_ab, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_all, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_ab, [node()]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_ab, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_all, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_ab, [node()]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
 
     %% reconnect node 2
     syn_test_suite_helper:connect_node(SlaveNode2),
@@ -231,42 +231,42 @@ three_nodes_discover(Config) ->
     syn_test_suite_helper:assert_cluster(SlaveNode2, [node(), SlaveNode1]),
 
     %% check
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_ab, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_all, [SlaveNode1, SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_ab, [node()]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_bc, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_c, []),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_all, [node(), SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_ab, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_all, [SlaveNode1, SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_ab, [node()]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_bc, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_c, []),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_all, [node(), SlaveNode1]),
 
     %% crash scope processes
     rpc:call(SlaveNode2, syn_test_suite_helper, kill_process, [syn_registry_scope_bc]),
     rpc:call(SlaveNode2, syn_test_suite_helper, wait_process_name_ready, [syn_registry_scope_bc]),
 
     %% check
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_ab, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_all, [SlaveNode1, SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_ab, [node()]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_bc, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_c, []),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_all, [node(), SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_ab, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_all, [SlaveNode1, SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_ab, [node()]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_bc, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_c, []),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_all, [node(), SlaveNode1]),
 
     %% crash scopes supervisor on local
     syn_test_suite_helper:kill_process(syn_scopes_sup),
     syn_test_suite_helper:wait_process_name_ready(syn_registry_scope_all),
 
     %% check
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_ab, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(node(), scope_all, [SlaveNode1, SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_ab, [node()]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_bc, [SlaveNode1]),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_c, []),
-    syn_test_suite_helper:assert_groups_scope_subcluster(SlaveNode2, scope_all, [node(), SlaveNode1]).
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_ab, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(node(), scope_all, [SlaveNode1, SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_ab, [node()]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_bc, [SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode1, scope_all, [node(), SlaveNode2]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_bc, [SlaveNode1]),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_c, []),
+    syn_test_suite_helper:assert_pg_scope_subcluster(SlaveNode2, scope_all, [node(), SlaveNode1]).
 
 three_nodes_join_leave_and_monitor(Config) ->
     %% get slaves

+ 2 - 2
test/syn_test_suite_helper.erl

@@ -35,7 +35,7 @@
 -export([wait_process_name_ready/1, wait_process_name_ready/2]).
 -export([wait_message_queue_empty/0]).
 -export([assert_cluster/2]).
--export([assert_registry_scope_subcluster/3, assert_groups_scope_subcluster/3]).
+-export([assert_registry_scope_subcluster/3, assert_pg_scope_subcluster/3]).
 -export([assert_received_messages/1]).
 -export([assert_empty_queue/1]).
 -export([assert_same_array_with_same_members/2]).
@@ -194,7 +194,7 @@ assert_cluster(Node, ExpectedNodes, StartAt) ->
 assert_registry_scope_subcluster(Node, Scope, ExpectedNodes) ->
     do_assert_scope_subcluster(registry, Node, Scope, ExpectedNodes).
 
-assert_groups_scope_subcluster(Node, Scope, ExpectedNodes) ->
+assert_pg_scope_subcluster(Node, Scope, ExpectedNodes) ->
     do_assert_scope_subcluster(pg, Node, Scope, ExpectedNodes).
 
 assert_received_messages(Messages) ->