diff options
| author | Michael Klishin <michael@clojurewerkz.org> | 2021-08-26 17:25:07 +0300 |
|---|---|---|
| committer | Michael Klishin <michael@clojurewerkz.org> | 2021-08-26 17:25:07 +0300 |
| commit | 42a3dfa81b3c532ccdcf2fb69d192fb7a2db495b (patch) | |
| tree | 19cb15c3c0fa1ba0bd61ad26d5c330a8d42b96ee | |
| parent | a809e55dbb65be2fef6f9193ffb5176a956f703d (diff) | |
| download | rabbitmq-server-git-42a3dfa81b3c532ccdcf2fb69d192fb7a2db495b.tar.gz | |
Exclude the #3333 test case from mixed version runs
| -rw-r--r-- | deps/rabbit/test/definition_import_SUITE.erl | 44 |
1 files changed, 25 insertions, 19 deletions
diff --git a/deps/rabbit/test/definition_import_SUITE.erl b/deps/rabbit/test/definition_import_SUITE.erl index 216402ef1f..a589018164 100644 --- a/deps/rabbit/test/definition_import_SUITE.erl +++ b/deps/rabbit/test/definition_import_SUITE.erl @@ -209,25 +209,31 @@ import_case14(Config) -> import_file_case(Config, "case14"). import_case15(Config) -> import_file_case(Config, "case15"). %% contains a virtual host with tags import_case16(Config) -> - case rabbit_ct_broker_helpers:enable_feature_flag(Config, virtual_host_metadata) of - ok -> - import_file_case(Config, "case16"), - VHost = <<"tagged">>, - VHostIsImported = - fun () -> - case vhost_lookup(Config, VHost) of - {error, {no_such_vhosts, _}} -> false; - _ -> true - end - end, - rabbit_ct_helpers:await_condition(VHostIsImported, 20000), - VHostRec = vhost_lookup(Config, VHost), - ?assertEqual(<<"A case16 description">>, vhost:get_description(VHostRec)), - ?assertEqual([multi_dc_replication,ab,cde], vhost:get_tags(VHostRec)), - - ok; - Skip -> - Skip + case rabbit_ct_helpers:is_mixed_versions() of + false -> + case rabbit_ct_broker_helpers:enable_feature_flag(Config, virtual_host_metadata) of + ok -> + import_file_case(Config, "case16"), + VHost = <<"tagged">>, + VHostIsImported = + fun () -> + case vhost_lookup(Config, VHost) of + {error, {no_such_vhosts, _}} -> false; + _ -> true + end + end, + rabbit_ct_helpers:await_condition(VHostIsImported, 20000), + VHostRec = vhost_lookup(Config, VHost), + ?assertEqual(<<"A case16 description">>, vhost:get_description(VHostRec)), + ?assertEqual([multi_dc_replication,ab,cde], vhost:get_tags(VHostRec)), + + ok; + Skip -> + Skip + end; + _ -> + %% skip the test in mixed version mode + {skip, "Should not run in mixed version environments"} end. export_import_round_trip_case1(Config) -> |