Patchwork [2/2] libqtest: New qtest_end() to go with qtest_start()

login
register
mail settings
Submitter Markus Armbruster
Date June 19, 2013, 7:44 a.m.
Message ID <1371627895-27046-3-git-send-email-armbru@redhat.com>
Download mbox | patch
Permalink /patch/252512/
State New
Headers show

Comments

Markus Armbruster - June 19, 2013, 7:44 a.m.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 tests/fdc-test.c    | 2 +-
 tests/hd-geo-test.c | 8 ++++----
 tests/ide-test.c    | 2 +-
 tests/libqtest.h    | 7 +++++++
 4 files changed, 13 insertions(+), 6 deletions(-)
Andreas Färber - June 19, 2013, 12:36 p.m.
Am 19.06.2013 09:44, schrieb Markus Armbruster:
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  tests/fdc-test.c    | 2 +-
>  tests/hd-geo-test.c | 8 ++++----
>  tests/ide-test.c    | 2 +-
>  tests/libqtest.h    | 7 +++++++
>  4 files changed, 13 insertions(+), 6 deletions(-)
> 
> diff --git a/tests/fdc-test.c b/tests/fdc-test.c
> index 4b0301d..fd198dc 100644
> --- a/tests/fdc-test.c
> +++ b/tests/fdc-test.c
> @@ -556,7 +556,7 @@ int main(int argc, char **argv)
>      ret = g_test_run();
>  
>      /* Cleanup */
> -    qtest_quit(global_qtest);
> +    qtest_end();
>      unlink(test_image);
>  
>      return ret;
> diff --git a/tests/hd-geo-test.c b/tests/hd-geo-test.c
> index 9a31e85..b72042e 100644
> --- a/tests/hd-geo-test.c
> +++ b/tests/hd-geo-test.c
> @@ -244,7 +244,7 @@ static void test_ide_none(void)
>      setup_common(argv, ARRAY_SIZE(argv));
>      qtest_start(g_strjoinv(" ", argv));
>      test_cmos();
> -    qtest_quit(global_qtest);
> +    qtest_end();
>  }
>  
>  static void test_ide_mbr(bool use_device, MBRcontents mbr)
> @@ -262,7 +262,7 @@ static void test_ide_mbr(bool use_device, MBRcontents mbr)
>      }
>      qtest_start(g_strjoinv(" ", argv));
>      test_cmos();
> -    qtest_quit(global_qtest);
> +    qtest_end();
>  }
>  
>  /*
> @@ -334,7 +334,7 @@ static void test_ide_drive_user(const char *dev, bool trans)
>      g_free(opts);
>      qtest_start(g_strjoinv(" ", argv));
>      test_cmos();
> -    qtest_quit(global_qtest);
> +    qtest_end();
>  }
>  
>  /*
> @@ -387,7 +387,7 @@ static void test_ide_drive_cd_0(void)
>      }
>      qtest_start(g_strjoinv(" ", argv));
>      test_cmos();
> -    qtest_quit(global_qtest);
> +    qtest_end();
>  }
>  
>  int main(int argc, char **argv)
> diff --git a/tests/ide-test.c b/tests/ide-test.c
> index 7e2eb94..7307f1d 100644
> --- a/tests/ide-test.c
> +++ b/tests/ide-test.c
> @@ -122,7 +122,7 @@ static void ide_test_start(const char *cmdline_fmt, ...)
>  
>  static void ide_test_quit(void)
>  {
> -    qtest_quit(global_qtest);
> +    qtest_end();
>  }
>  
>  static QPCIDevice *get_pci_device(uint16_t *bmdma_base)
> diff --git a/tests/libqtest.h b/tests/libqtest.h
> index 437bda3..329325b 100644
> --- a/tests/libqtest.h
> +++ b/tests/libqtest.h
> @@ -17,6 +17,7 @@
>  #ifndef LIBQTEST_H
>  #define LIBQTEST_H
>  
> +#include <stddef.h>
>  #include <stdint.h>
>  #include <stdbool.h>
>  #include <stdarg.h>
> @@ -318,6 +319,12 @@ static inline QTestState *qtest_start(const char *args)
>      return global_qtest;
>  }
>  
> +static inline void qtest_end(void)
> +{
> +    qtest_quit(global_qtest);
> +    global_qtest = NULL;
> +}
> +
>  /**
>   * qmp:
>   * @fmt...: QMP message to send to qemu
> 

Function and usage looks fine, but please add a gtk-doc comment.
qtest_start() does have one.

Thanks,
Andreas
Markus Armbruster - June 19, 2013, 1:21 p.m.
Andreas Färber <afaerber@suse.de> writes:

> Am 19.06.2013 09:44, schrieb Markus Armbruster:
[...]
>> diff --git a/tests/libqtest.h b/tests/libqtest.h
>> index 437bda3..329325b 100644
>> --- a/tests/libqtest.h
>> +++ b/tests/libqtest.h
>> @@ -17,6 +17,7 @@
>>  #ifndef LIBQTEST_H
>>  #define LIBQTEST_H
>>  
>> +#include <stddef.h>
>>  #include <stdint.h>
>>  #include <stdbool.h>
>>  #include <stdarg.h>
>> @@ -318,6 +319,12 @@ static inline QTestState *qtest_start(const char *args)
>>      return global_qtest;
>>  }
>>  
>> +static inline void qtest_end(void)
>> +{
>> +    qtest_quit(global_qtest);
>> +    global_qtest = NULL;
>> +}
>> +
>>  /**
>>   * qmp:
>>   * @fmt...: QMP message to send to qemu
>> 
>
> Function and usage looks fine, but please add a gtk-doc comment.
> qtest_start() does have one.

Oversight, will fix, thanks!

Patch

diff --git a/tests/fdc-test.c b/tests/fdc-test.c
index 4b0301d..fd198dc 100644
--- a/tests/fdc-test.c
+++ b/tests/fdc-test.c
@@ -556,7 +556,7 @@  int main(int argc, char **argv)
     ret = g_test_run();
 
     /* Cleanup */
-    qtest_quit(global_qtest);
+    qtest_end();
     unlink(test_image);
 
     return ret;
diff --git a/tests/hd-geo-test.c b/tests/hd-geo-test.c
index 9a31e85..b72042e 100644
--- a/tests/hd-geo-test.c
+++ b/tests/hd-geo-test.c
@@ -244,7 +244,7 @@  static void test_ide_none(void)
     setup_common(argv, ARRAY_SIZE(argv));
     qtest_start(g_strjoinv(" ", argv));
     test_cmos();
-    qtest_quit(global_qtest);
+    qtest_end();
 }
 
 static void test_ide_mbr(bool use_device, MBRcontents mbr)
@@ -262,7 +262,7 @@  static void test_ide_mbr(bool use_device, MBRcontents mbr)
     }
     qtest_start(g_strjoinv(" ", argv));
     test_cmos();
-    qtest_quit(global_qtest);
+    qtest_end();
 }
 
 /*
@@ -334,7 +334,7 @@  static void test_ide_drive_user(const char *dev, bool trans)
     g_free(opts);
     qtest_start(g_strjoinv(" ", argv));
     test_cmos();
-    qtest_quit(global_qtest);
+    qtest_end();
 }
 
 /*
@@ -387,7 +387,7 @@  static void test_ide_drive_cd_0(void)
     }
     qtest_start(g_strjoinv(" ", argv));
     test_cmos();
-    qtest_quit(global_qtest);
+    qtest_end();
 }
 
 int main(int argc, char **argv)
diff --git a/tests/ide-test.c b/tests/ide-test.c
index 7e2eb94..7307f1d 100644
--- a/tests/ide-test.c
+++ b/tests/ide-test.c
@@ -122,7 +122,7 @@  static void ide_test_start(const char *cmdline_fmt, ...)
 
 static void ide_test_quit(void)
 {
-    qtest_quit(global_qtest);
+    qtest_end();
 }
 
 static QPCIDevice *get_pci_device(uint16_t *bmdma_base)
diff --git a/tests/libqtest.h b/tests/libqtest.h
index 437bda3..329325b 100644
--- a/tests/libqtest.h
+++ b/tests/libqtest.h
@@ -17,6 +17,7 @@ 
 #ifndef LIBQTEST_H
 #define LIBQTEST_H
 
+#include <stddef.h>
 #include <stdint.h>
 #include <stdbool.h>
 #include <stdarg.h>
@@ -318,6 +319,12 @@  static inline QTestState *qtest_start(const char *args)
     return global_qtest;
 }
 
+static inline void qtest_end(void)
+{
+    qtest_quit(global_qtest);
+    global_qtest = NULL;
+}
+
 /**
  * qmp:
  * @fmt...: QMP message to send to qemu