-
Notifications
You must be signed in to change notification settings - Fork 422
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run tests in pubsub:basic in parallel #3432
Conversation
We do repeat_all_until_all_ok anyway, so if there is a race condition, it would be good to see it
Codecov Report
@@ Coverage Diff @@
## master #3432 +/- ##
==========================================
+ Coverage 80.79% 80.87% +0.07%
==========================================
Files 414 414
Lines 32336 32336
==========================================
+ Hits 26126 26151 +25
+ Misses 6210 6185 -25
Continue to review full report at Codecov.
|
small_tests_24 / small_tests / 8c4c6fd small_tests_23 / small_tests / 8c4c6fd dynamic_domains_pgsql_mnesia_23 / pgsql_mnesia / 8c4c6fd dynamic_domains_mssql_mnesia_24 / odbc_mssql_mnesia / 8c4c6fd dynamic_domains_mysql_redis_24 / mysql_redis / 8c4c6fd dynamic_domains_pgsql_mnesia_24 / pgsql_mnesia / 8c4c6fd ldap_mnesia_23 / ldap_mnesia / 8c4c6fd ldap_mnesia_24 / ldap_mnesia / 8c4c6fd internal_mnesia_24 / internal_mnesia / 8c4c6fd pgsql_mnesia_24 / pgsql_mnesia / 8c4c6fd elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / 8c4c6fd pgsql_mnesia_23 / pgsql_mnesia / 8c4c6fd mssql_mnesia_24 / odbc_mssql_mnesia / 8c4c6fd disco_and_caps_SUITE:disco_with_caps:user_can_query_friend_features{error,{{assertion_failed,assert_many,false,[is_roster_set],[],[]},
[{escalus_new_assert,assert_true,2,
[{file,"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_new_assert.erl"},
{line,84}]},
{escalus_story,'-make_all_clients_friends/1-fun-0-',2,
[{file,"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_story.erl"},
{line,108}]},
{escalus_utils,'-each_with_index/3-fun-0-',3,
[{file,"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_utils.erl"},
{line,87}]},
{lists,foldl,3,[{file,"lists.erl"},{line,1267}]},
{escalus_utils,'-each_with_index/3-fun-0-',3,
[{file,"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_utils.erl"},
{line,87}]},
{lists,foldl,3,[{file,"lists.erl"},{line,1267}]},
{escalus_utils,distinct_pairs,2,
[{file,"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_utils.erl"},
{line,60}]},
{escalus_story,make_all_clients_friends,1,
[{file,"/home/circleci/project/big_tests/_build/default/lib/escalus/src/escalus_story.erl"},
{line,106}]}]}} mysql_redis_24 / mysql_redis / 8c4c6fd riak_mnesia_24 / riak_mnesia / 8c4c6fd sm_SUITE:parallel_manual_ack_freq_1:resume_session_state_stop_c2s{error,{thrown,{timeout,msg}}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, ok, let's see if it crashes 🤔
Proposed changes include:
The main motivation is that it has one test that sleeps 5 seconds (i.e. subscribe_options_deliver_option_test). But I probably need to read some pubsub to ensure we can check that notification would not come without sleeping :) - that would come separately