aboutsummaryrefslogtreecommitdiffstats
path: root/lib/diameter
diff options
context:
space:
mode:
authorAnders Svensson <[email protected]>2015-05-05 09:48:40 +0200
committerAnders Svensson <[email protected]>2015-05-05 12:42:55 +0200
commit3d6446cd9949e188bdfcacb28800e9db4ecd3dab (patch)
tree647107fe88c78bd1e6ca4e8824f0443308d078f8 /lib/diameter
parent2d9c68b22e352edbdd2a6b1ff823293a258d9493 (diff)
downloadotp-3d6446cd9949e188bdfcacb28800e9db4ecd3dab.tar.gz
otp-3d6446cd9949e188bdfcacb28800e9db4ecd3dab.tar.bz2
otp-3d6446cd9949e188bdfcacb28800e9db4ecd3dab.zip
Fix broken traffic testcase
The send_error testcase tested that Session-Id in an answer-message was not undefined, but that's always the case since the AVP has arity 0 or 1. The correct test is that it's a list of length 1, to ensure that diameter has inserted the session id as expected.
Diffstat (limited to 'lib/diameter')
-rw-r--r--lib/diameter/test/diameter_traffic_SUITE.erl5
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/diameter/test/diameter_traffic_SUITE.erl b/lib/diameter/test/diameter_traffic_SUITE.erl
index f223c9a7d0..7ff6ba7ab9 100644
--- a/lib/diameter/test/diameter_traffic_SUITE.erl
+++ b/lib/diameter/test/diameter_traffic_SUITE.erl
@@ -707,9 +707,8 @@ send_timeout(Config) ->
%% received the Session-Id.
send_error(Config) ->
Req = ['RAR', {'Re-Auth-Request-Type', ?AUTHORIZE_AUTHENTICATE}],
- ?answer_message(SId, ?TOO_BUSY)
- = call(Config, Req),
- true = undefined /= SId.
+ ?answer_message([_], ?TOO_BUSY)
+ = call(Config, Req).
%% Send a request with the detached option and receive it as a message
%% from handle_answer instead.