Hi Luiz,
> On Tue, Mar 10, 2015 at 6:31 AM, Luiz Augusto von Dentz <luiz.dentz@xxxxxxxxx> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
>
> This checks if crypto is enabled and in case it is not do not run test
> /TP/GAW/CL/BV-02-C.
> ---
> unit/test-gatt.c | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/unit/test-gatt.c b/unit/test-gatt.c
> index 2edcacb..7668e93 100644
> --- a/unit/test-gatt.c
> +++ b/unit/test-gatt.c
> @@ -304,9 +304,12 @@ static gboolean context_quit(gpointer user_data)
> if (step && step->post_func)
> step->post_func(context);
>
> - destroy_context(context);
> + if (context->data->pdu_list[context->pdu_offset].valid)
> + tester_test_abort();
> + else
> + tester_test_passed();
>
> - tester_test_passed();
> + destroy_context(context);
>
> return FALSE;
> }
> @@ -910,6 +913,11 @@ static void test_signed_write(struct context *context)
> uint8_t key[16] = {0xD8, 0x51, 0x59, 0x48, 0x45, 0x1F, 0xEA, 0x32, 0x0D,
> 0xC0, 0x5A, 0x2E, 0x88, 0x30, 0x81, 0x88 };
>
> + if (!bt_att_has_crypto(context->att)) {
Should we assert in this case that
bt_gatt_client_write_without_response, given true for "signed_write"
returns 0 without crashing? We would at least be validating the
current behavior, or perhaps we should add a separate test case for
it.
> + context_quit(context);
> + return;
> + }
> +
> g_assert(bt_att_set_local_key(context->att, key, local_counter,
> context));
>
> --
> 2.1.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
Thanks,
Arman
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
