diff mbox

tests: Rename qtests which have names ending "error"

Message ID 1470307178-22848-1-git-send-email-peter.maydell@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Peter Maydell Aug. 4, 2016, 10:39 a.m. UTC
We have three qtest tests which have test names ending with "error".
This is awkward because the output of verbose test runs looks like
  /crypto/task/error:                                                  OK
  /crypto/task/thread_error:                                           OK

which gives false positives if you are grepping build logs for
errors by looking for "error:". Since there are only three tests
with this problem, just rename them all to 'failure' instead.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
Per discussion on IRC yesterday. I might throw this one into 2.7,
I dunno.

In particular the grep rune I run over build logs for merges
looks for "error:" among other things, so these false positives
are irritating.
---
 tests/test-io-task.c      | 8 ++++----
 tests/test-qmp-commands.c | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

Comments

Daniel P. Berrangé Aug. 4, 2016, 11:31 a.m. UTC | #1
On Thu, Aug 04, 2016 at 11:39:38AM +0100, Peter Maydell wrote:
> We have three qtest tests which have test names ending with "error".
> This is awkward because the output of verbose test runs looks like
>   /crypto/task/error:                                                  OK
>   /crypto/task/thread_error:                                           OK
> 
> which gives false positives if you are grepping build logs for
> errors by looking for "error:". Since there are only three tests
> with this problem, just rename them all to 'failure' instead.
> 
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

Reviewed-by: Daniel P. Berrange <berrange@redhat.com>

> ---
> Per discussion on IRC yesterday. I might throw this one into 2.7,
> I dunno.

It has no functional change, so low risk & thus fine to put in 2.7 IMHO

> 
> In particular the grep rune I run over build logs for merges
> looks for "error:" among other things, so these false positives
> are irritating.
> ---
>  tests/test-io-task.c      | 8 ++++----
>  tests/test-qmp-commands.c | 4 ++--
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/tests/test-io-task.c b/tests/test-io-task.c
> index a36cb82..e091c12 100644
> --- a/tests/test-io-task.c
> +++ b/tests/test-io-task.c
> @@ -110,7 +110,7 @@ static void test_task_data_free(void)
>  }
>  
>  
> -static void test_task_error(void)
> +static void test_task_failure(void)
>  {
>      QIOTask *task;
>      Object *obj = object_new(TYPE_DUMMY);
> @@ -214,7 +214,7 @@ static void test_task_thread_complete(void)
>  }
>  
>  
> -static void test_task_thread_error(void)
> +static void test_task_thread_failure(void)
>  {
>      QIOTask *task;
>      Object *obj = object_new(TYPE_DUMMY);
> @@ -262,8 +262,8 @@ int main(int argc, char **argv)
>      type_register_static(&dummy_info);
>      g_test_add_func("/crypto/task/complete", test_task_complete);
>      g_test_add_func("/crypto/task/datafree", test_task_data_free);
> -    g_test_add_func("/crypto/task/error", test_task_error);
> +    g_test_add_func("/crypto/task/failure", test_task_failure);
>      g_test_add_func("/crypto/task/thread_complete", test_task_thread_complete);
> -    g_test_add_func("/crypto/task/thread_error", test_task_thread_error);
> +    g_test_add_func("/crypto/task/thread_failure", test_task_thread_failure);
>      return g_test_run();
>  }
> diff --git a/tests/test-qmp-commands.c b/tests/test-qmp-commands.c
> index 5af1a46..261fd9e 100644
> --- a/tests/test-qmp-commands.c
> +++ b/tests/test-qmp-commands.c
> @@ -103,7 +103,7 @@ static void test_dispatch_cmd(void)
>  }
>  
>  /* test commands that return an error due to invalid parameters */
> -static void test_dispatch_cmd_error(void)
> +static void test_dispatch_cmd_failure(void)
>  {
>      QDict *req = qdict_new();
>      QObject *resp;
> @@ -253,7 +253,7 @@ int main(int argc, char **argv)
>      g_test_init(&argc, &argv, NULL);
>  
>      g_test_add_func("/0.15/dispatch_cmd", test_dispatch_cmd);
> -    g_test_add_func("/0.15/dispatch_cmd_error", test_dispatch_cmd_error);
> +    g_test_add_func("/0.15/dispatch_cmd_failure", test_dispatch_cmd_failure);
>      g_test_add_func("/0.15/dispatch_cmd_io", test_dispatch_cmd_io);
>      g_test_add_func("/0.15/dealloc_types", test_dealloc_types);
>      g_test_add_func("/0.15/dealloc_partial", test_dealloc_partial);
> -- 
> 1.9.1
> 

Regards,
Daniel
Peter Maydell Aug. 5, 2016, 3:39 p.m. UTC | #2
On 4 August 2016 at 12:31, Daniel P. Berrange <berrange@redhat.com> wrote:
> On Thu, Aug 04, 2016 at 11:39:38AM +0100, Peter Maydell wrote:
>> We have three qtest tests which have test names ending with "error".
>> This is awkward because the output of verbose test runs looks like
>>   /crypto/task/error:                                                  OK
>>   /crypto/task/thread_error:                                           OK
>>
>> which gives false positives if you are grepping build logs for
>> errors by looking for "error:". Since there are only three tests
>> with this problem, just rename them all to 'failure' instead.
>>
>> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
>
> Reviewed-by: Daniel P. Berrange <berrange@redhat.com>
>
>> ---
>> Per discussion on IRC yesterday. I might throw this one into 2.7,
>> I dunno.
>
> It has no functional change, so low risk & thus fine to put in 2.7 IMHO

Applied to master, thanks.

-- PMM
diff mbox

Patch

diff --git a/tests/test-io-task.c b/tests/test-io-task.c
index a36cb82..e091c12 100644
--- a/tests/test-io-task.c
+++ b/tests/test-io-task.c
@@ -110,7 +110,7 @@  static void test_task_data_free(void)
 }
 
 
-static void test_task_error(void)
+static void test_task_failure(void)
 {
     QIOTask *task;
     Object *obj = object_new(TYPE_DUMMY);
@@ -214,7 +214,7 @@  static void test_task_thread_complete(void)
 }
 
 
-static void test_task_thread_error(void)
+static void test_task_thread_failure(void)
 {
     QIOTask *task;
     Object *obj = object_new(TYPE_DUMMY);
@@ -262,8 +262,8 @@  int main(int argc, char **argv)
     type_register_static(&dummy_info);
     g_test_add_func("/crypto/task/complete", test_task_complete);
     g_test_add_func("/crypto/task/datafree", test_task_data_free);
-    g_test_add_func("/crypto/task/error", test_task_error);
+    g_test_add_func("/crypto/task/failure", test_task_failure);
     g_test_add_func("/crypto/task/thread_complete", test_task_thread_complete);
-    g_test_add_func("/crypto/task/thread_error", test_task_thread_error);
+    g_test_add_func("/crypto/task/thread_failure", test_task_thread_failure);
     return g_test_run();
 }
diff --git a/tests/test-qmp-commands.c b/tests/test-qmp-commands.c
index 5af1a46..261fd9e 100644
--- a/tests/test-qmp-commands.c
+++ b/tests/test-qmp-commands.c
@@ -103,7 +103,7 @@  static void test_dispatch_cmd(void)
 }
 
 /* test commands that return an error due to invalid parameters */
-static void test_dispatch_cmd_error(void)
+static void test_dispatch_cmd_failure(void)
 {
     QDict *req = qdict_new();
     QObject *resp;
@@ -253,7 +253,7 @@  int main(int argc, char **argv)
     g_test_init(&argc, &argv, NULL);
 
     g_test_add_func("/0.15/dispatch_cmd", test_dispatch_cmd);
-    g_test_add_func("/0.15/dispatch_cmd_error", test_dispatch_cmd_error);
+    g_test_add_func("/0.15/dispatch_cmd_failure", test_dispatch_cmd_failure);
     g_test_add_func("/0.15/dispatch_cmd_io", test_dispatch_cmd_io);
     g_test_add_func("/0.15/dealloc_types", test_dealloc_types);
     g_test_add_func("/0.15/dealloc_partial", test_dealloc_partial);