Skip to content
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

Enable metrics_c2s_SUITE for dynamic domains #3277

Merged
merged 1 commit into from
Sep 17, 2021

Conversation

Premwoik
Copy link
Contributor

Enable metrics_c2s_SUITE for dynamic domains.

@mongoose-im
Copy link
Collaborator

mongoose-im commented Sep 16, 2021

small_tests_24 / small_tests / 7f01406
Reports root / small


internal_mnesia_24 / internal_mnesia / 7f01406
Reports root/ big
OK: 1589 / Failed: 0 / User-skipped: 290 / Auto-skipped: 0


small_tests_22 / small_tests / 7f01406
Reports root / small


small_tests_23 / small_tests / 7f01406
Reports root / small


dynamic_domains_24 / pgsql_mnesia / 7f01406
Reports root/ big
OK: 2445 / Failed: 0 / User-skipped: 141 / Auto-skipped: 0


ldap_mnesia_22 / ldap_mnesia / 7f01406
Reports root/ big
OK: 1486 / Failed: 0 / User-skipped: 393 / Auto-skipped: 0


dynamic_domains_23 / pgsql_mnesia / 7f01406
Reports root/ big
OK: 2445 / Failed: 0 / User-skipped: 141 / Auto-skipped: 0


pgsql_mnesia_22 / pgsql_mnesia / 7f01406
Reports root/ big
OK: 3064 / Failed: 0 / User-skipped: 211 / Auto-skipped: 0


ldap_mnesia_24 / ldap_mnesia / 7f01406
Reports root/ big
OK: 1486 / Failed: 0 / User-skipped: 393 / Auto-skipped: 0


ldap_mnesia_23 / ldap_mnesia / 7f01406
Reports root/ big
OK: 1486 / Failed: 0 / User-skipped: 393 / Auto-skipped: 0


mssql_mnesia_24 / odbc_mssql_mnesia / 7f01406
Reports root/ big
OK: 3064 / Failed: 0 / User-skipped: 211 / Auto-skipped: 0


pgsql_mnesia_24 / pgsql_mnesia / 7f01406
Reports root/ big
OK: 3064 / Failed: 0 / User-skipped: 211 / Auto-skipped: 0


elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / 7f01406
Reports root/ big
OK: 1862 / Failed: 0 / User-skipped: 316 / Auto-skipped: 0


pgsql_mnesia_23 / pgsql_mnesia / 7f01406
Reports root/ big
OK: 3064 / Failed: 0 / User-skipped: 211 / Auto-skipped: 0


mysql_redis_24 / mysql_redis / 7f01406
Reports root/ big
OK: 3059 / Failed: 2 / User-skipped: 228 / Auto-skipped: 0

mam_SUITE:rdbms_cache_prefs_cases:messages_filtered_when_prefs_default_policy_is_never
{error,{test_case_failed,"ASSERT EQUAL\n\tExpected []\n\tValue [ok]\n"}}

Report log

mam_SUITE:rdbms_cache_prefs_cases:messages_filtered_when_prefs_default_policy_is_roster
{error,{test_case_failed,"ASSERT EQUAL\n\tExpected []\n\tValue [ok]\n"}}

Report log


riak_mnesia_24 / riak_mnesia / 7f01406
Reports root/ big
OK: 1709 / Failed: 0 / User-skipped: 319 / Auto-skipped: 0


mysql_redis_24 / mysql_redis / 7f01406
Reports root/ big
OK: 3054 / Failed: 1 / User-skipped: 228 / Auto-skipped: 0

sm_SUITE:parallel_manual_ack_freq_1:resume_session_state_send_message
{error,
  {{assertion_failed,assert,is_presence,
     {xmlel,<<"message">>,
       [{<<"from">>,
         <<"bOb_resume_session_state_send_message_63.842498@localhost/escalus-default-resource">>},
        {<<"to">>,
         <<"alicE_resume_session_state_send_message_63.960437@localhost">>},
        {<<"xml:lang">>,<<"en">>},
        {<<"type">>,<<"chat">>}],
       [{xmlel,<<"body">>,[],[{xmlcdata,<<"msg-3">>}]}]},
     "<message from='bOb_resume_session_state_send_message_63.842498@localhost/escalus-default-resource' to='alicE_resume_session_state_send_message_63.960437@localhost' xml:lang='en' type='chat'><body>msg-3</body></message>"},
   [{escalus_new_assert,assert_true,2,
      [{file,
         "/home/circleci/app/big_tests/_build/default/lib/escalus/src/escalus_new_assert.erl"},
       {line,84}]},
    {sm_SUITE,resume_session_state_send_message,1,
      [{file,"/home/circleci/app/big_tests/tests/sm_SUITE.erl"},
       {line,727}]},
    {test_server,ts_tc,3,[{file,"test_server.erl"},{line,1783}]},
    {test_server,run_test_case_eval1,6,
      [{file,"test_server.erl"},{line,1292}]},
    {test_server,run_test_case_eval,9,
      [{file,"test_server.erl"},{line,1224}]}]}}

Report log

@codecov
Copy link

codecov bot commented Sep 16, 2021

Codecov Report

Merging #3277 (a3587a2) into master (40943d6) will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3277      +/-   ##
==========================================
- Coverage   80.27%   80.23%   -0.04%     
==========================================
  Files         398      398              
  Lines       32560    32560              
==========================================
- Hits        26137    26125      -12     
- Misses       6423     6435      +12     
Impacted Files Coverage Δ
src/mod_roster_riak.erl 81.53% <0.00%> (-15.39%) ⬇️
src/mam/mod_mam_rdbms_async_pool_writer.erl 66.66% <0.00%> (-3.93%) ⬇️
src/global_distrib/mod_global_distrib_receiver.erl 80.00% <0.00%> (-1.12%) ⬇️
src/ejabberd_sm.erl 83.93% <0.00%> (-0.66%) ⬇️
src/mod_muc_room.erl 77.18% <0.00%> (+0.05%) ⬆️
src/ejabberd_c2s.erl 89.29% <0.00%> (+0.07%) ⬆️
src/domain/mongoose_domain_loader.erl 79.24% <0.00%> (+1.88%) ⬆️
src/ejabberd.erl 55.00% <0.00%> (+10.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40943d6...a3587a2. Read the comment docs.

@Premwoik Premwoik marked this pull request as ready for review September 17, 2021 07:22
Copy link
Contributor

@vkatsuba vkatsuba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please resolve conflicts in big_tests/dynamic_domains.spec.

@Premwoik Premwoik force-pushed the dynamic-domains-metrics-c2s-tests branch from 7f01406 to a3587a2 Compare September 17, 2021 08:02
@mongoose-im
Copy link
Collaborator

mongoose-im commented Sep 17, 2021

small_tests_24 / small_tests / a3587a2
Reports root / small


internal_mnesia_24 / internal_mnesia / a3587a2
Reports root/ big
OK: 1589 / Failed: 0 / User-skipped: 290 / Auto-skipped: 0


small_tests_22 / small_tests / a3587a2
Reports root / small


small_tests_23 / small_tests / a3587a2
Reports root / small


dynamic_domains_23 / pgsql_mnesia / a3587a2
Reports root/ big
OK: 2524 / Failed: 0 / User-skipped: 147 / Auto-skipped: 0


dynamic_domains_24 / pgsql_mnesia / a3587a2
Reports root/ big
OK: 2524 / Failed: 0 / User-skipped: 147 / Auto-skipped: 0


ldap_mnesia_23 / ldap_mnesia / a3587a2
Reports root/ big
OK: 1486 / Failed: 0 / User-skipped: 393 / Auto-skipped: 0


ldap_mnesia_22 / ldap_mnesia / a3587a2
Reports root/ big
OK: 1486 / Failed: 0 / User-skipped: 393 / Auto-skipped: 0


ldap_mnesia_24 / ldap_mnesia / a3587a2
Reports root/ big
OK: 1486 / Failed: 0 / User-skipped: 393 / Auto-skipped: 0


pgsql_mnesia_24 / pgsql_mnesia / a3587a2
Reports root/ big
OK: 3064 / Failed: 0 / User-skipped: 211 / Auto-skipped: 0


pgsql_mnesia_22 / pgsql_mnesia / a3587a2
Reports root/ big
OK: 3064 / Failed: 0 / User-skipped: 211 / Auto-skipped: 0


mysql_redis_24 / mysql_redis / a3587a2
Reports root/ big
OK: 3053 / Failed: 1 / User-skipped: 228 / Auto-skipped: 0

mam_SUITE:rdbms_async_cache_prefs_cases:messages_filtered_when_prefs_default_policy_is_never
{error,{test_case_failed,"ASSERT EQUAL\n\tExpected []\n\tValue [ok]\n"}}

Report log


pgsql_mnesia_23 / pgsql_mnesia / a3587a2
Reports root/ big
OK: 3064 / Failed: 0 / User-skipped: 211 / Auto-skipped: 0


elasticsearch_and_cassandra_24 / elasticsearch_and_cassandra_mnesia / a3587a2
Reports root/ big
OK: 1862 / Failed: 0 / User-skipped: 316 / Auto-skipped: 0


mssql_mnesia_24 / odbc_mssql_mnesia / a3587a2
Reports root/ big
OK: 3064 / Failed: 0 / User-skipped: 211 / Auto-skipped: 0


riak_mnesia_24 / riak_mnesia / a3587a2
Reports root/ big
OK: 1709 / Failed: 0 / User-skipped: 319 / Auto-skipped: 0

Copy link
Member

@chrzaszcz chrzaszcz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@chrzaszcz chrzaszcz merged commit aa671e8 into master Sep 17, 2021
@chrzaszcz chrzaszcz deleted the dynamic-domains-metrics-c2s-tests branch September 17, 2021 08:54
@Premwoik Premwoik added this to the 5.0.0 milestone Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants