Update E2ETest #1120
Update E2ETest #1120
Conversation
Bummer. Because I didn't change anything in |
|
@sanmai is it actually run anywhere? |
|
Not during CI, but we can make it, if you insist. Let's get finished with #1106, and I'll help it. Who knows, it may not need any help. |
I've restarted it, but this shouldn't be an issue, because on |
|
Fingers crossed. Anyway, merge as you see fit. |
|
Still no luck, same job :| And now also app veyor is broken (#1110 (comment)) |
|
There's something seriously wrong with this PR. I'll be having a look later. |
| ])); | ||
| } | ||
|
|
||
| $output = $this->runInfection(self::EXPECT_SUCCESS, [ | ||
| '--test-framework-options="--exclude-group=' . self::EXCLUDED_GROUP . '"', | ||
| sprintf('--test-framework-options=--exclude-group=%s', self::EXCLUDED_GROUP), |
There was a problem hiding this comment.
I'm guessing, but probably quotes affects somehow the build. It's now written in a different way
There was a problem hiding this comment.
It didn't help :see_no_evil:
There was a problem hiding this comment.
not many cases left to revert :) next is --no-progress?
There was a problem hiding this comment.
It's more like something isn't added here.
|
Curiously, build are broken only for PCOV. |
|
I'm going to close it with hopes I can revisit the idea later. |
@runInSeparateProcess.