Re: [PATCH] unit/avctp: Add TP/NFR/BV-04-C test

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andrei,

On Tue, Feb 4, 2014 at 3:47 PM, Andrei Emeltchenko
<Andrei.Emeltchenko.news@xxxxxxxxx> wrote:
> From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
>
> Add test TP/NFR/BV-04-C for AVCTP.
> ---
>  unit/test-avctp.c | 40 +++++++++++++++++++++++++++++++---------
>  1 file changed, 31 insertions(+), 9 deletions(-)
>
> diff --git a/unit/test-avctp.c b/unit/test-avctp.c
> index 6d7e34a..581f88c 100644
> --- a/unit/test-avctp.c
> +++ b/unit/test-avctp.c
> @@ -240,15 +240,6 @@ static void execute_context(struct context *context)
>         destroy_context(context);
>  }
>
> -static void test_client(gconstpointer data)
> -{
> -       struct context *context = create_context(0x0100, data);
> -
> -       avctp_send_vendordep_req(context->session, 0, 0, NULL, 0, NULL, NULL);
> -
> -       execute_context(context);
> -}
> -
>  static size_t handler(struct avctp *session,
>                                         uint8_t transaction, uint8_t *code,
>                                         uint8_t *subunit, uint8_t *operands,
> @@ -265,6 +256,33 @@ static size_t handler(struct avctp *session,
>         return operand_count;
>  }
>
> +static gboolean handler_response(struct avctp *session,
> +                                       uint8_t code, uint8_t subunit,
> +                                       uint8_t *operands, size_t operand_count,
> +                                       void *user_data)
> +{
> +       struct context *context = user_data;
> +
> +       DBG("code 0x%02x subunit %d operand_count %zu", code, subunit,
> +                                                               operand_count);
> +
> +       g_assert_cmpint(code, ==, 0x0a);
> +       g_assert_cmpint(subunit, ==, 0);
> +       g_assert_cmpint(operand_count, ==, 0);
> +
> +       return context_quit(context);
> +}
> +
> +static void test_client(gconstpointer data)
> +{
> +       struct context *context = create_context(0x0100, data);
> +
> +       avctp_send_vendordep_req(context->session, 0, 0, NULL, 0,
> +                                               handler_response, context);
> +
> +       execute_context(context);
> +}
> +
>  static void test_server(gconstpointer data)
>  {
>         struct context *context = create_context(0x0100, data);
> @@ -322,6 +340,10 @@ int main(int argc, char *argv[])
>                                 raw_pdu(0x00, 0x11, 0x0e, 0x00, 0x00, 0x00),
>                                 raw_pdu(0x02, 0x11, 0x0e, 0x00, 0x00, 0x00));
>
> +       define_test("/TP/NFR/BV-04-C", test_client,
> +                               raw_pdu(0x00, 0x11, 0x0e, 0x00, 0x00, 0x00),
> +                               raw_pdu(0x02, 0x11, 0x0e, 0x0a, 0x00, 0x00));
> +
>         define_test("/TP/NFR/BI-01-C", test_server,
>                                 raw_pdu(0x00, 0xff, 0xff, 0x00, 0x00, 0x00),
>                                 raw_pdu(0x03, 0xff, 0xff));
> --
> 1.8.3.2

Applied, thanks.


-- 
Luiz Augusto von Dentz
--
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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux