All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH BlueZ] unit/test-gdbus-client: Fix hanging if D-Bus fails to launch
@ 2015-04-22  8:39 Luiz Augusto von Dentz
  2015-04-22 21:12 ` Arman Uguray
  0 siblings, 1 reply; 2+ messages in thread
From: Luiz Augusto von Dentz @ 2015-04-22  8:39 UTC (permalink / raw)
  To: linux-bluetooth

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

In case D-Bus fails to launch make the test fails so it does not hang
other tests.
---
 unit/test-gdbus-client.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/unit/test-gdbus-client.c b/unit/test-gdbus-client.c
index 5a4b974..7e16d16 100644
--- a/unit/test-gdbus-client.c
+++ b/unit/test-gdbus-client.c
@@ -72,6 +72,7 @@ static struct context *create_context(void)
 		}
 
 		g_free(context);
+		tester_test_failed();
 		return NULL;
 	}
 
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH BlueZ] unit/test-gdbus-client: Fix hanging if D-Bus fails to launch
  2015-04-22  8:39 [PATCH BlueZ] unit/test-gdbus-client: Fix hanging if D-Bus fails to launch Luiz Augusto von Dentz
@ 2015-04-22 21:12 ` Arman Uguray
  0 siblings, 0 replies; 2+ messages in thread
From: Arman Uguray @ 2015-04-22 21:12 UTC (permalink / raw)
  To: Luiz Augusto von Dentz; +Cc: BlueZ development

Hi Luiz,

> On Wed, Apr 22, 2015 at 1:39 AM, Luiz Augusto von Dentz <luiz.dentz@gmail.com> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
>
> In case D-Bus fails to launch make the test fails so it does not hang
> other tests.
> ---
>  unit/test-gdbus-client.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/unit/test-gdbus-client.c b/unit/test-gdbus-client.c
> index 5a4b974..7e16d16 100644
> --- a/unit/test-gdbus-client.c
> +++ b/unit/test-gdbus-client.c
> @@ -72,6 +72,7 @@ static struct context *create_context(void)
>                 }
>
>                 g_free(context);
> +               tester_test_failed();
>                 return NULL;
>         }
>
> --
> 2.1.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Verified and pushed.

Thanks,
Arman

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-04-22 21:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-22  8:39 [PATCH BlueZ] unit/test-gdbus-client: Fix hanging if D-Bus fails to launch Luiz Augusto von Dentz
2015-04-22 21:12 ` Arman Uguray

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.