* [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).