Commit 9cccaa3
test: fix component printing on windows
Commit 084b2ec ("test: include component in tap output") introduced
an in hindsight glaringly obvious but fortunately not very critical
Windows-specific bug by failing to take the path separator into account.
This commit rectifies that, the prefix is now correctly stripped.
PR-URL: #6915
Refs: #6653
Reviewed-By: Anna Henningsen <anna@addaleax.net>1 parent 373ffc5 commit 9cccaa3
1 file changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -273,10 +273,11 @@ def HasRun(self, output): | |||
| 273 | 273 | # Print test name as (for example) "parallel/test-assert". Tests that are | |
| 274 | 274 | # scraped from the addons documentation are all named test.js, making it | |
| 275 | 275 | # hard to decipher what test is running when only the filename is printed. | |
| 276 | - prefix = abspath(join(dirname(__file__), '../test')) + '/' | ||
| 276 | + prefix = abspath(join(dirname(__file__), '../test')) + os.sep | ||
| 277 | 277 | command = output.command[-1] | |
| 278 | 278 | if command.endswith('.js'): command = command[:-3] | |
| 279 | 279 | if command.startswith(prefix): command = command[len(prefix):] | |
| 280 | + command = command.replace('\\', '/') | ||
| 280 | 281 | ||
| 281 | 282 | if output.UnexpectedOutput(): | |
| 282 | 283 | status_line = 'not ok %i %s' % (self._done, command) | |
0 commit comments