qemu-trivial.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-trivial] [PATCH] rtc-test: Fix test failures with recent glib
@ 2013-02-27  0:31 Cole Robinson
  2013-03-08  9:17 ` Stefan Hajnoczi
  0 siblings, 1 reply; 2+ messages in thread
From: Cole Robinson @ 2013-02-27  0:31 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, qemu-stable, Cole Robinson

As of glib 2.35.4, glib changed its logic for ordering test cases:

https://bugzilla.gnome.org/show_bug.cgi?id=694487

This was causing failures in rtc-test. Group the reordered test
cases into their own suite, which maintains the original ordering.

CC: qemu-stable@nongnu.org
Signed-off-by: Cole Robinson <crobinso@redhat.com>
---
 tests/rtc-test.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/rtc-test.c b/tests/rtc-test.c
index c5fd042..9ab583b 100644
--- a/tests/rtc-test.c
+++ b/tests/rtc-test.c
@@ -565,8 +565,8 @@ int main(int argc, char **argv)
     qtest_add_func("/rtc/basic/bcd-12h", basic_12h_bcd);
     qtest_add_func("/rtc/set-year/20xx", set_year_20xx);
     qtest_add_func("/rtc/set-year/1980", set_year_1980);
-    qtest_add_func("/rtc/register_b_set_flag", register_b_set_flag);
-    qtest_add_func("/rtc/fuzz-registers", fuzz_registers);
+    qtest_add_func("/rtc/misc/register_b_set_flag", register_b_set_flag);
+    qtest_add_func("/rtc/misc/fuzz-registers", fuzz_registers);
     ret = g_test_run();
 
     if (s) {
-- 
1.8.1.2



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

* Re: [Qemu-trivial] [PATCH] rtc-test: Fix test failures with recent glib
  2013-02-27  0:31 [Qemu-trivial] [PATCH] rtc-test: Fix test failures with recent glib Cole Robinson
@ 2013-03-08  9:17 ` Stefan Hajnoczi
  0 siblings, 0 replies; 2+ messages in thread
From: Stefan Hajnoczi @ 2013-03-08  9:17 UTC (permalink / raw)
  To: Cole Robinson; +Cc: qemu-trivial, qemu-devel, qemu-stable

On Tue, Feb 26, 2013 at 07:31:32PM -0500, Cole Robinson wrote:
> As of glib 2.35.4, glib changed its logic for ordering test cases:
> 
> https://bugzilla.gnome.org/show_bug.cgi?id=694487
> 
> This was causing failures in rtc-test. Group the reordered test
> cases into their own suite, which maintains the original ordering.
> 
> CC: qemu-stable@nongnu.org
> Signed-off-by: Cole Robinson <crobinso@redhat.com>
> ---
>  tests/rtc-test.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

It's best to make test cases independent of order (no side-effects), but
this small patch fixes the problem.

Thanks, applied to the trivial patches tree:
https://github.com/stefanha/qemu/commits/trivial-patches

Stefan


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

end of thread, other threads:[~2013-03-08  9:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-02-27  0:31 [Qemu-trivial] [PATCH] rtc-test: Fix test failures with recent glib Cole Robinson
2013-03-08  9:17 ` Stefan Hajnoczi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).