diff options
| author | Neels Hofmeyr <nhofmeyr@sysmocom.de> | 2016-03-30 11:22:27 +0200 |
|---|---|---|
| committer | Harald Welte <laforge@gnumonks.org> | 2016-03-31 11:56:29 +0200 |
| commit | f9b212fabd0d5c37dac9639fc9d9ecd73688e3a3 (patch) | |
| tree | 39b0fe83d18acdb2d643ba0f86552c425c52328d /openbsc/tests | |
| parent | d617c5d3ac65b5090f787da3e2911fb6742f515c (diff) | |
MM Auth: introduce AUTH_ERROR constant.
Instead of using hardcoded -1 for errors, include -1 in the enum auth_action
type; apply its use.
In the mm_auth test, the string output changes from '(internal error)' to
'AUTH_ERROR', since now the proper enum value is used in auth_action_names[].
Diffstat (limited to 'openbsc/tests')
| -rw-r--r-- | openbsc/tests/mm_auth/mm_auth_test.c | 2 | ||||
| -rw-r--r-- | openbsc/tests/mm_auth/mm_auth_test.ok | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/openbsc/tests/mm_auth/mm_auth_test.c b/openbsc/tests/mm_auth/mm_auth_test.c index e54189834..1d65984cb 100644 --- a/openbsc/tests/mm_auth/mm_auth_test.c +++ b/openbsc/tests/mm_auth/mm_auth_test.c @@ -141,7 +141,7 @@ static void test_error() test_get_authinfo_rc = -EIO; auth_action = auth_get_tuple_for_subscr_verbose(&atuple, &subscr, key_seq); - OSMO_ASSERT(auth_action == -1); + OSMO_ASSERT(auth_action == AUTH_ERROR); } static void test_auth_not_avail() diff --git a/openbsc/tests/mm_auth/mm_auth_test.ok b/openbsc/tests/mm_auth/mm_auth_test.ok index cc0e76960..7dedadc38 100644 --- a/openbsc/tests/mm_auth/mm_auth_test.ok +++ b/openbsc/tests/mm_auth/mm_auth_test.ok @@ -1,7 +1,7 @@ * test_error() wrapped: db_get_authinfo_for_subscr(): rc = -5 -auth_get_tuple_for_subscr(key_seq=0) --> auth_action == (internal error) +auth_get_tuple_for_subscr(key_seq=0) --> auth_action == AUTH_ERROR * test_auth_not_avail() wrapped: db_get_authinfo_for_subscr(): rc = -2 |