[CRIU] [PATCH v2] zdtm: print a summary of failed tests if using --keep-going

[Andrei Vagin]

On Tue, Nov 29, 2016 at 08:45:07AM +0100, Adrian Reber wrote:
> From: Adrian Reber <areber at redhat.com>
> 
> When using the option --keep-going to get a quick overview of the test
> results there is now a summary which tests have failed like:
> 
>  ############# 7 TEST(S) FAILED (TOTAL 268/SKIPPED 83) ##############
>   * zdtm/static/sched_policy00(unknown)
>   * zdtm/static/pthread00(unknown)
>   * zdtm/static/pthread01(unknown)
>   * zdtm/static/maps01(unknown)
>   * zdtm/static/maps00(unknown)
>   * zdtm/transition/maps008(unknown)
>   * zdtm/transition/shmem(unknown)
>  ############################### FAIL ###############################
>

Acked-by: Andrei Vagin <avagin at virtuozzo.com>

Thanks!

> Signed-off-by: Adrian Reber <areber at redhat.com>
> ---
>  test/zdtm.py | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/test/zdtm.py b/test/zdtm.py
> index 9d682dd..a17cce4 100755
> --- a/test/zdtm.py
> +++ b/test/zdtm.py
> @@ -1325,6 +1325,8 @@ class launcher:
>  		self.__subs = {}
>  		self.__fail = False
>  		self.__file_report = None
> +		self.__failed = []
> +		self.__nr_skip = 0
>  		if self.__max > 1 and self.__total > 1:
>  			self.__use_log = True
>  		elif opts['report']:
> @@ -1355,6 +1357,7 @@ class launcher:
>  		print "Skipping %s (%s)" % (name, reason)
>  		self.__nr += 1
>  		self.__runtest += 1
> +		self.__nr_skip += 1
>  		if self.__file_report:
>  			testline = "ok %d - %s # SKIP %s" % (self.__runtest, name, reason)
>  			print >> self.__file_report, testline
> @@ -1398,6 +1401,7 @@ class launcher:
>  			if status != 0:
>  				self.__fail = True
>  				failed_flavor = decode_flav(os.WEXITSTATUS(status))
> +				self.__failed.append([sub['name'], failed_flavor])
>  				if self.__file_report:
>  					testline = "not ok %d - %s # flavor %s" % (self.__runtest, sub['name'], failed_flavor)
>  					details = {'output': open(sub['log']).read()}
> @@ -1442,6 +1446,11 @@ class launcher:
>  		if self.__file_report:
>  			self.__file_report.close()
>  		if self.__fail:
> +			if opts['keep_going']:
> +				print_sep("%d TEST(S) FAILED (TOTAL %d/SKIPPED %d)"
> +						% (len(self.__failed), self.__total, self.__nr_skip), "#")
> +				for failed in self.__failed:
> +					print " * %s(%s)" % (failed[0], failed[1])
>  			print_sep("FAIL", "#")
>  			sys.exit(1)
>  
> -- 
> 2.9.3
>