Skip to content

Commit 4a925f9

Browse files
committed
Tests: system_SUITE return configuration from run steps
1 parent 106012d commit 4a925f9

File tree

1 file changed

+4
-8
lines changed

1 file changed

+4
-8
lines changed

deps/oauth2_client/test/system_SUITE.erl

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,6 @@ init_per_group(https, Config) ->
105105
CaCertFile = filename:join([CertsDir, "testca", "cacert.pem"]),
106106
WrongCaCertFile = filename:join([CertsDir, "server", "server.pem"]),
107107
[{group, https},
108-
{certsdir, CertsDir},
109108
{oauth_provider_id, <<"uaa">>},
110109
{oauth_provider, build_https_oauth_provider(<<"uaa">>, CaCertFile)},
111110
{oauth_provider_with_issuer, keep_only_issuer_and_ssl_options(
@@ -122,9 +121,8 @@ init_per_group(https_down, Config) ->
122121
CaCertFile = filename:join([CertsDir, "testca", "cacert.pem"]),
123122

124123
[{issuer, build_issuer("https")},
125-
{certsdir, CertsDir},
126124
{oauth_provider_id, <<"uaa">>},
127-
{oauth_provider, build_https_oauth_provider(<<"uaa">>, CaCertFile)} | Config];
125+
{oauth_provider, build_https_oauth_provider(<<"uaa">>, CaCertFile)} | Config0];
128126

129127
init_per_group(openid_configuration_with_path, Config) ->
130128
[{use_openid_configuration_with_path, true} | Config];
@@ -135,20 +133,18 @@ init_per_group(with_all_oauth_provider_settings, Config) ->
135133
CaCertFile = filename:join([CertsDir, "testca", "cacert.pem"]),
136134

137135
[{with_all_oauth_provider_settings, true},
138-
{certsdir, CertsDir},
139136
{oauth_provider_id, <<"uaa">>},
140-
{oauth_provider, build_https_oauth_provider(<<"uaa">>, CaCertFile)} | Config];
137+
{oauth_provider, build_https_oauth_provider(<<"uaa">>, CaCertFile)} | Config0];
141138

142139
init_per_group(without_all_oauth_providers_settings, Config) ->
143140
Config0 = rabbit_ct_helpers:run_setup_steps(Config),
144141
CertsDir = ?config(rmq_certsdir, Config0),
145142
CaCertFile = filename:join([CertsDir, "testca", "cacert.pem"]),
146143

147144
[{with_all_oauth_provider_settings, false},
148-
{certsdir, CertsDir},
149145
{oauth_provider_id, <<"uaa">>},
150146
{oauth_provider, keep_only_issuer_and_ssl_options(
151-
build_https_oauth_provider(<<"uaa">>, CaCertFile))} | Config];
147+
build_https_oauth_provider(<<"uaa">>, CaCertFile))} | Config0];
152148

153149
init_per_group(with_default_oauth_provider, Config) ->
154150
OAuthProvider = ?config(oauth_provider, Config),
@@ -285,7 +281,7 @@ init_per_testcase(TestCase, Config0) ->
285281

286282
case ?config(group, Config) of
287283
https ->
288-
start_https_oauth_server(?AUTH_PORT, ?config(certsdir, Config),
284+
start_https_oauth_server(?AUTH_PORT, ?config(rmq_certsdir, Config),
289285
ListOfExpectations);
290286
_ ->
291287
do_nothing

0 commit comments

Comments
 (0)