From 8c4c6fdaef4d7da14b6aa718d058067fc693d0b2 Mon Sep 17 00:00:00 2001 From: Mikhail Uvarov Date: Fri, 3 Dec 2021 13:36:31 +0100 Subject: [PATCH] Run tests in pubsub:basic in parallel We do repeat_all_until_all_ok anyway, so if there is a race condition, it would be good to see it --- big_tests/tests/pubsub_SUITE.erl | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/big_tests/tests/pubsub_SUITE.erl b/big_tests/tests/pubsub_SUITE.erl index bf55dadd6ba..45dc696351c 100644 --- a/big_tests/tests/pubsub_SUITE.erl +++ b/big_tests/tests/pubsub_SUITE.erl @@ -153,11 +153,7 @@ group_is_compatible(hometree_specific, OnlyNodetreeTree) -> OnlyNodetreeTree =:= group_is_compatible(_, _) -> true. base_groups() -> - %% NOTE: basic and collection are placed in sequence because they run bigger - %% sets of tests, which in parallel might cause deadlocks in the - %% transactions for MNESIA and MYSQL. - %% TODO: Optimise pubsub backends (MYSQL! Mnesia?) - G = [{basic, [sequence], basic_tests()}, + G = [{basic, [parallel], basic_tests()}, {service_config, [parallel], service_config_tests()}, {node_config, [parallel], node_config_tests()}, {node_affiliations, [parallel], node_affiliations_tests()}, @@ -482,9 +478,9 @@ subscribe_options_deliver_option_test(Config) -> pubsub_tools:publish(Alice, <<"item1">>, Node, []), - %% Geralt should recive a notification + %% Geralt should receive a notification pubsub_tools:receive_item_notification(Geralt, <<"item1">>, Node, [{expected_result, true}]), - %% Bob should not recive a notification + %% Bob should not receive a notification [] = escalus:wait_for_stanzas(Bob, 1, 5000), pubsub_tools:delete_node(Alice, Node, [])