Commit 4b19439
tools: print appropriate output when test aborted
distinguish test abort from all tests passed.
PR-URL: #59794
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent cf5d91e commit 4b19439
1 file changed
Lines changed: 4 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -174,7 +174,7 @@ def Run(self, tasks) -> Dict: | |||
| 174 | 174 | raise | |
| 175 | 175 | self.Done() | |
| 176 | 176 | return { | |
| 177 | - 'allPassed': not self.failed, | ||
| 177 | + 'allPassed': not self.failed and not self.shutdown_event.is_set(), | ||
| 178 | 178 | 'failed': self.failed, | |
| 179 | 179 | } | |
| 180 | 180 | ||
@@ -1834,11 +1834,12 @@ def should_keep(case): | |||
| 1834 | 1834 | ||
| 1835 | 1835 | if result['allPassed']: | |
| 1836 | 1836 | print("\nAll tests passed.") | |
| 1837 | - else: | ||
| 1837 | + elif result['failed']: | ||
| 1838 | 1838 | print("\nFailed tests:") | |
| 1839 | 1839 | for failure in result['failed']: | |
| 1840 | 1840 | print(EscapeCommand(failure.command)) | |
| 1841 | - | ||
| 1841 | + else: | ||
| 1842 | + print("\nTest aborted.") | ||
| 1842 | 1843 | return exitcode | |
| 1843 | 1844 | ||
| 1844 | 1845 | ||
0 commit comments