From bb8677549aa6ee9c1994c4ed5e36c451af88c740 Mon Sep 17 00:00:00 2001 From: Nuno Maduro Date: Mon, 2 Sep 2024 18:30:12 +0100 Subject: [PATCH] chore: fixes snapshots --- tests/.pest/snapshots/Visual/Todo/todo.snap | 6 +++--- .../.pest/snapshots/Visual/Todo/todo_in_parallel.snap | 6 +++--- tests/.pest/snapshots/Visual/Todo/todos.snap | 6 +++--- .../.pest/snapshots/Visual/Todo/todos_in_parallel.snap | 6 +++--- tests/.snapshots/success.txt | 10 +++++----- tests/Visual/Success.php | 2 +- tests/Visual/Todo.php | 10 +++++----- 7 files changed, 23 insertions(+), 23 deletions(-) diff --git a/tests/.pest/snapshots/Visual/Todo/todo.snap b/tests/.pest/snapshots/Visual/Todo/todo.snap index 6b944036..f7a85c93 100644 --- a/tests/.pest/snapshots/Visual/Todo/todo.snap +++ b/tests/.pest/snapshots/Visual/Todo/todo.snap @@ -19,9 +19,9 @@ ↓ something todo later ↓ something todo later chained ↓ something todo later chained and with function body - ↓ it may have an associated assignee []8;;https://github.com/nunomaduro\@nunomaduro]8;;\] - ↓ it may have an associated issue ]8;;https://github.com/pestphp/pest/issues/1\#1]8;;\ - ↓ it may have an associated PR ]8;;https://github.com/pestphp/pest/pull/1\#1]8;;\ + ↓ it may have an associated assignee [@nunomaduro] + ↓ it may have an associated issue #1 + ↓ it may have an associated PR #1 ↓ it may have an associated note // a note diff --git a/tests/.pest/snapshots/Visual/Todo/todo_in_parallel.snap b/tests/.pest/snapshots/Visual/Todo/todo_in_parallel.snap index 6b944036..f7a85c93 100644 --- a/tests/.pest/snapshots/Visual/Todo/todo_in_parallel.snap +++ b/tests/.pest/snapshots/Visual/Todo/todo_in_parallel.snap @@ -19,9 +19,9 @@ ↓ something todo later ↓ something todo later chained ↓ something todo later chained and with function body - ↓ it may have an associated assignee []8;;https://github.com/nunomaduro\@nunomaduro]8;;\] - ↓ it may have an associated issue ]8;;https://github.com/pestphp/pest/issues/1\#1]8;;\ - ↓ it may have an associated PR ]8;;https://github.com/pestphp/pest/pull/1\#1]8;;\ + ↓ it may have an associated assignee [@nunomaduro] + ↓ it may have an associated issue #1 + ↓ it may have an associated PR #1 ↓ it may have an associated note // a note diff --git a/tests/.pest/snapshots/Visual/Todo/todos.snap b/tests/.pest/snapshots/Visual/Todo/todos.snap index 6b944036..f7a85c93 100644 --- a/tests/.pest/snapshots/Visual/Todo/todos.snap +++ b/tests/.pest/snapshots/Visual/Todo/todos.snap @@ -19,9 +19,9 @@ ↓ something todo later ↓ something todo later chained ↓ something todo later chained and with function body - ↓ it may have an associated assignee []8;;https://github.com/nunomaduro\@nunomaduro]8;;\] - ↓ it may have an associated issue ]8;;https://github.com/pestphp/pest/issues/1\#1]8;;\ - ↓ it may have an associated PR ]8;;https://github.com/pestphp/pest/pull/1\#1]8;;\ + ↓ it may have an associated assignee [@nunomaduro] + ↓ it may have an associated issue #1 + ↓ it may have an associated PR #1 ↓ it may have an associated note // a note diff --git a/tests/.pest/snapshots/Visual/Todo/todos_in_parallel.snap b/tests/.pest/snapshots/Visual/Todo/todos_in_parallel.snap index 6b944036..f7a85c93 100644 --- a/tests/.pest/snapshots/Visual/Todo/todos_in_parallel.snap +++ b/tests/.pest/snapshots/Visual/Todo/todos_in_parallel.snap @@ -19,9 +19,9 @@ ↓ something todo later ↓ something todo later chained ↓ something todo later chained and with function body - ↓ it may have an associated assignee []8;;https://github.com/nunomaduro\@nunomaduro]8;;\] - ↓ it may have an associated issue ]8;;https://github.com/pestphp/pest/issues/1\#1]8;;\ - ↓ it may have an associated PR ]8;;https://github.com/pestphp/pest/pull/1\#1]8;;\ + ↓ it may have an associated assignee [@nunomaduro] + ↓ it may have an associated issue #1 + ↓ it may have an associated PR #1 ↓ it may have an associated note // a note diff --git a/tests/.snapshots/success.txt b/tests/.snapshots/success.txt index 6db9b207..a619ca9e 100644 --- a/tests/.snapshots/success.txt +++ b/tests/.snapshots/success.txt @@ -1551,12 +1551,12 @@ - visual snapshot of team city with ('SuccessOnly.php') WARN Tests\Visual\Todo - … todos → Snapshot created at [tests/.pest/snapshots/Visual/Todo/todos.snap] - … todos in parallel → Snapshot created at [tests/.pest/snapshots/Visual/Todo/todos_in_parallel.snap] - … todo → Snapshot created at [tests/.pest/snapshots/Visual/Todo/todo.snap] - … todo in parallel → Snapshot created at [tests/.pest/snapshots/Visual/Todo/todo_in_parallel.snap] + - todos + - todos in parallel + - todo + - todo in parallel WARN Tests\Visual\Version - visual snapshot of help command output - Tests: 2 deprecated, 4 warnings, 9 incomplete, 2 notices, 17 todos, 24 skipped, 1079 passed (2613 assertions) \ No newline at end of file + Tests: 2 deprecated, 4 warnings, 5 incomplete, 2 notices, 17 todos, 28 skipped, 1079 passed (2609 assertions) \ No newline at end of file diff --git a/tests/Visual/Success.php b/tests/Visual/Success.php index ac0db670..d87ad304 100644 --- a/tests/Visual/Success.php +++ b/tests/Visual/Success.php @@ -12,7 +12,7 @@ test('visual snapshot of test suite on success', function () { $process = (new Symfony\Component\Process\Process( ['php', 'bin/pest'], dirname($testsPath), - ['EXCLUDE' => 'integration', 'REBUILD_SNAPSHOTS' => false, 'PARATEST' => 0, 'COLLISION_PRINTER' => 'DefaultPrinter', 'COLLISION_IGNORE_DURATION' => 'true'], + ['EXCLUDE' => 'integration', '--exclude-group' => 'integration', 'REBUILD_SNAPSHOTS' => false, 'PARATEST' => 0, 'COLLISION_PRINTER' => 'DefaultPrinter', 'COLLISION_IGNORE_DURATION' => 'true'], )); $process->run(); diff --git a/tests/Visual/Todo.php b/tests/Visual/Todo.php index 945291ef..21b8963d 100644 --- a/tests/Visual/Todo.php +++ b/tests/Visual/Todo.php @@ -3,7 +3,7 @@ use Symfony\Component\Process\Process; $run = function (string $target, bool $parallel) { - $process = new Process(['php', 'bin/pest', $target, $parallel ? '--parallel' : '', '--colors=always'], dirname(__DIR__, 2), + $process = new Process(['php', 'bin/pest', $target, $parallel ? '--parallel' : ''], dirname(__DIR__, 2), ['COLLISION_PRINTER' => 'DefaultPrinter', 'COLLISION_IGNORE_DURATION' => 'true'], ); @@ -18,16 +18,16 @@ $run = function (string $target, bool $parallel) { test('todos', function () use ($run) { expect($run('--todos', false))->toMatchSnapshot(); -})->skipOnWindows(); +})->skipOnWindows()->skip(! getenv('REBUILD_SNAPSHOTS') && getenv('EXCLUDE')); test('todos in parallel', function () use ($run) { expect($run('--todos', true))->toMatchSnapshot(); -})->skipOnWindows(); +})->skipOnWindows()->skip(! getenv('REBUILD_SNAPSHOTS') && getenv('EXCLUDE')); test('todo', function () use ($run) { expect($run('--todo', false))->toMatchSnapshot(); -})->skipOnWindows(); +})->skipOnWindows()->skip(! getenv('REBUILD_SNAPSHOTS') && getenv('EXCLUDE')); test('todo in parallel', function () use ($run) { expect($run('--todo', true))->toMatchSnapshot(); -})->skipOnWindows(); +})->skipOnWindows()->skip(! getenv('REBUILD_SNAPSHOTS') && getenv('EXCLUDE'));