Skip to content

Commit c109bb5

Browse files
committed
minor #1230 Do not skip user list command test because of stty (bocharsky-bw)
This PR was merged into the main branch. Discussion ---------- Do not skip user list command test because of stty Fixes an issue in this discussion: https://github.com/symfony/demo/pull/1133/files#r665459095 Commits ------- 370f9f0 Do not skip user list command test because of stty
2 parents 4d39a62 + 370f9f0 commit c109bb5

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

tests/Command/AbstractCommandTest.php

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,6 @@
1717

1818
abstract class AbstractCommandTest extends KernelTestCase
1919
{
20-
protected function setUp(): void
21-
{
22-
if ('Windows' === \PHP_OS_FAMILY) {
23-
$this->markTestSkipped('`stty` is required to test this command.');
24-
}
25-
}
26-
2720
/**
2821
* This helper method abstracts the boilerplate code needed to test the
2922
* execution of a command.

tests/Command/AddUserCommandTest.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,13 @@ class AddUserCommandTest extends AbstractCommandTest
2323
'full-name' => 'Chuck Norris',
2424
];
2525

26+
protected function setUp(): void
27+
{
28+
if ('Windows' === \PHP_OS_FAMILY) {
29+
$this->markTestSkipped('`stty` is required to test this command.');
30+
}
31+
}
32+
2633
/**
2734
* @dataProvider isAdminDataProvider
2835
*

0 commit comments

Comments
 (0)