diff options
| author | Anders Svensson <[email protected]> | 2015-05-05 09:48:40 +0200 |
|---|---|---|
| committer | Anders Svensson <[email protected]> | 2015-05-05 12:42:55 +0200 |
| commit | 3d6446cd9949e188bdfcacb28800e9db4ecd3dab (patch) | |
| tree | 647107fe88c78bd1e6ca4e8824f0443308d078f8 | |
| parent | 2d9c68b22e352edbdd2a6b1ff823293a258d9493 (diff) | |
| download | otp-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.
| -rw-r--r-- | lib/diameter/test/diameter_traffic_SUITE.erl | 5 |
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. |