Skip to content

Commit 48ee0f9

Browse files
committed
Serialize diameter_tls_SUITE
1 parent 88e05bf commit 48ee0f9

File tree

1 file changed

+15
-14
lines changed

1 file changed

+15
-14
lines changed

lib/diameter/test/diameter_tls_SUITE.erl

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@
5050
end_per_testcase/2,
5151

5252
%% The test cases
53-
parallel/1
53+
test/1
5454
]).
5555

5656
%% diameter callbacks
@@ -134,15 +134,15 @@ suite() ->
134134
[{timetrap, {seconds, 90}}].
135135

136136
all() ->
137-
[parallel].
137+
[test].
138138

139139
%% Shouldn't really have to know about crypto here but 'ok' from
140140
%% ssl:start() isn't enough to guarantee that TLS is available.
141141
init_per_suite(Config) ->
142142
?TL("init_per_suite -> entry with"
143143
"~n Config: ~p", [Config]),
144144
try
145-
[] == (catch make_certs(dir(Config)))
145+
ok == (catch make_certs(dir(Config)))
146146
orelse throw({?MODULE, no_certs}),
147147
ok == application:start(crypto) orelse throw({?MODULE, no_crypto}),
148148
ok == ssl:start() orelse throw({?MODULE, no_ssl}),
@@ -186,10 +186,10 @@ end_per_testcase(Case, Config) when is_list(Config) ->
186186

187187
%% ===========================================================================
188188

189-
parallel(Config) ->
190-
?TL("parallel -> entry"),
189+
test(Config) ->
190+
?TL("test -> entry"),
191191
Res = run(dir(Config), false),
192-
?TL("parallel -> done when"
192+
?TL("test -> done when"
193193
"~n Res: ~p", [Res]),
194194
Res.
195195

@@ -214,7 +214,7 @@ run(Dir, B) ->
214214
ssl:start(),
215215
try
216216
?TL("run -> try run traffic"),
217-
?RUN([{[fun traffic/2, Dir, B], 60000}])
217+
traffic(Dir, B)
218218
after
219219
?TL("run(after) -> stop diameter"),
220220
diameter:stop(),
@@ -239,7 +239,7 @@ traffic(Dir, false) ->
239239
?TL("traffic(false) -> add transports"),
240240
Connections = add_transports(Dir, Servers),
241241
?TL("traffic(false) -> calls"),
242-
[] = ?RUN([[fun call/1, S] || S <- ?SCRAMBLE(?SERVERS)]),
242+
[ok = call(S) || S <- ?SCRAMBLE(?SERVERS)],
243243
?TL("traffic(false) -> remove transports"),
244244
[] = remove_transports(Connections),
245245
?TL("traffic(false) -> stop (diameter) services"),
@@ -248,11 +248,12 @@ traffic(Dir, false) ->
248248
ok.
249249

250250
make_certs(Dir) ->
251-
?RUN([[fun make_cert/2, Dir, B] || B <- ["server1",
252-
"server2",
253-
"server4",
254-
"server5",
255-
"client"]]).
251+
[make_cert(Dir, B) || B <- ["server1",
252+
"server2",
253+
"server4",
254+
"server5",
255+
"client"]],
256+
ok.
256257

257258
start_services(Dir) ->
258259
lists:foreach(fun(S) -> ?DEL_REG(S) end, ?SERVERS ++ [?CLIENT]),
@@ -269,7 +270,7 @@ add_transports(Dir, Servers) ->
269270
%% transport to go down.
270271
remove_transports(Connections) ->
271272
[] = [T || S <- ?SERVERS, T <- [diameter:subscribe(S)], T /= true],
272-
[] = ?RUN([[fun disconnect/1, T] || T <- Connections]),
273+
[disconnect(T) || T <- Connections],
273274
[S || S <- ?SERVERS,
274275
I <- [receive #diameter_event{service = S, info = I} -> I end],
275276
down /= catch element(1, I)].

0 commit comments

Comments
 (0)