diff options
| author | Neels Hofmeyr <neels@hofmeyr.de> | 2018-04-02 22:46:55 +0200 |
|---|---|---|
| committer | Neels Hofmeyr <neels@hofmeyr.de> | 2018-04-03 02:09:39 +0200 |
| commit | 9fe52e4af3b9d99e23ca8e559c5e6f36c70cee14 (patch) | |
| tree | a8a98580ce6dbcddd1be98d352728256914ad288 /tests | |
| parent | 80d1101153b97a031b6eeed317016c7fdce72ae0 (diff) | |
test_reject_concurrency: missing assert
I broke this test during dev and saw the failure being noticed only in the next
test when DTAP is expected again. Verify success right there, instead.
Change-Id: Ifdde3a6fa5835203c34c40db77761f2e90c0d5ff
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/msc_vlr/msc_vlr_test_reject_concurrency.c | 1 | ||||
| -rw-r--r-- | tests/msc_vlr/msc_vlr_test_reject_concurrency.err | 1 |
2 files changed, 2 insertions, 0 deletions
diff --git a/tests/msc_vlr/msc_vlr_test_reject_concurrency.c b/tests/msc_vlr/msc_vlr_test_reject_concurrency.c index b6f6558e2..08fc9e7a3 100644 --- a/tests/msc_vlr/msc_vlr_test_reject_concurrency.c +++ b/tests/msc_vlr/msc_vlr_test_reject_concurrency.c @@ -219,6 +219,7 @@ static void test_reject_cm_during_lu() ms_sends_msg("05247803305886089910070000006402"); VERBOSE_ASSERT(lu_result_sent, == RES_NONE, "%d"); VERBOSE_ASSERT(cm_service_result_sent, == RES_NONE, "%d"); + VERBOSE_ASSERT(dtap_tx_confirmed, == true, "%d"); EXPECT_CONN_COUNT(1); BTW("The first LU can still complete"); diff --git a/tests/msc_vlr/msc_vlr_test_reject_concurrency.err b/tests/msc_vlr/msc_vlr_test_reject_concurrency.err index a045e6726..c58d099bc 100644 --- a/tests/msc_vlr/msc_vlr_test_reject_concurrency.err +++ b/tests/msc_vlr/msc_vlr_test_reject_concurrency.err @@ -333,6 +333,7 @@ DMM IMSI:901700000004620: subscr_conn_release_when_unused: conn still being esta DREF IMSI:901700000004620: MSC conn use - dtap == 1 (0x4) lu_result_sent == 0 cm_service_result_sent == 0 + dtap_tx_confirmed == 1 llist_count(&net->subscr_conns) == 1 --- - The first LU can still complete |