[PATCH] lib: framework: add in test name before test description

IvanHu ivan.hu at canonical.com
Thu Mar 7 07:40:48 UTC 2013


On 03/06/2013 05:00 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> As suggested by John Brier, it makes sense to include the name
> of the test, so lets prefix that before the test description.
> This allows users to find the test results in the output log
> by just looking for the test name.
>
> An example of the output goes from:
>
> Gather BIOS DMI information.
>
> to:
>
> bios_info: Gather BIOS DMI information.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/lib/src/fwts_framework.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/lib/src/fwts_framework.c b/src/lib/src/fwts_framework.c
> index 25ccc41..57079e6 100644
> --- a/src/lib/src/fwts_framework.c
> +++ b/src/lib/src/fwts_framework.c
> @@ -496,7 +496,7 @@ static int fwts_framework_run_test(fwts_framework *fw, fwts_framework_test *test
>   		fw->print_summary = true;
>
>   	if (test->os->description) {
> -		fwts_log_heading(fw, "%s", test->ops->description);
> +		fwts_log_heading(fw, "%s: %s", test->name, test->ops->description);
>   		fwts_framework_underline(fw,'-');
>   		if (fw->show_progress) {
>   			char buf[70];
>


Acked-by: Ivan Hu <ivan.hu at canonical.com>



More information about the fwts-devel mailing list