Skip to content

Commit c39d76e

Browse files
committed
docs: put actual first in Assert::same() examples
Testo's Assert::same() takes ($actual, $expected), the reverse of PHPUnit, yet 42 examples across EN and RU passed the expected value first. Assisted-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 529787b commit c39d76e

17 files changed

Lines changed: 40 additions & 40 deletions

File tree

‎docs/ai/best-practices.md‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ final class UserServiceTest
3737

3838
$user = $service->create('Alice', 'alice@example.com');
3939

40-
Assert::same('Alice', $user->name);
40+
Assert::same($user->name, 'Alice');
4141
}
4242

4343
/**
@@ -66,7 +66,7 @@ public function rejectsAgesOutsideRange(int $age, bool $accepted): void
6666
{
6767
$service = new UserService(new InMemoryRepository());
6868

69-
Assert::same($accepted, $service->canRegister($age));
69+
Assert::same($service->canRegister($age), $accepted);
7070
}
7171
```
7272

@@ -76,7 +76,7 @@ Reach for `\Testo\Data\DataProvider` when the cases need code to produce — gen
7676
#[DataProvider('emailFixtures')]
7777
public function validatesEmail(string $email, bool $expected): void
7878
{
79-
Assert::same($expected, UserService::isValidEmail($email));
79+
Assert::same(UserService::isValidEmail($email), $expected);
8080
}
8181

8282
public static function emailFixtures(): iterable
@@ -116,7 +116,7 @@ When positive and negative cases come from separate providers, pair each with it
116116
#[Covers(UserService::class, 'isValidEmail')]
117117
public function validatesEmail(string $email, bool $expected): void
118118
{
119-
Assert::same($expected, UserService::isValidEmail($email));
119+
Assert::same(UserService::isValidEmail($email), $expected);
120120
}
121121
```
122122

‎docs/bridge/revolt.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public function resolvesAfterDelay(): void
4242
$suspension = EventLoop::getSuspension();
4343
EventLoop::delay(0.1, static fn() => $suspension->resume('ready'));
4444

45-
Assert::same('ready', $suspension->suspend()); // the loop keeps spinning while we wait
45+
Assert::same($suspension->suspend(), 'ready'); // the loop keeps spinning while we wait
4646
}
4747
```
4848

‎docs/plugins/codecov.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,7 @@ public function smokeTest(): void
304304
{
305305
// Test runs, but coverage is not collected
306306
$response = $this->app->get('/health');
307-
Assert::same(200, $response->statusCode);
307+
Assert::same($response->statusCode, 200);
308308
}
309309
```
310310
</example>

‎docs/plugins/data.md‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ Data providers let you run one test with different sets of input data. Each set
2020
#[DataSet([0, 0, 0])]
2121
public function testSum(int $a, int $b, int $expected): void
2222
{
23-
Assert::same($expected, $a + $b);
23+
Assert::same($a + $b, $expected);
2424
}
2525
```
2626
</example>
@@ -46,7 +46,7 @@ public function testSum(int $a, int $b, int $expected): void { ... }
4646
public function testUserValidation(string $email, bool $expected): void
4747
{
4848
$isValid = $this->validator->validate($email);
49-
Assert::same($expected, $isValid);
49+
Assert::same($isValid, $expected);
5050
}
5151

5252
public function userDataProvider(): iterable
@@ -122,7 +122,7 @@ The first item from the first provider joins with the first from the second, sec
122122
public function testUserPermissions(string $login, string $password, array $permissions): void
123123
{
124124
$user = $this->auth->login($login, $password);
125-
Assert::same($permissions, $user->getPermissions());
125+
Assert::same($user->getPermissions(), $permissions);
126126
}
127127

128128
// credentials: [['admin', 'secret'], ['guest', '1234']]
@@ -261,7 +261,7 @@ For complex scenarios, providers can be nested:
261261
public function testAccessControl(string $user, string $role, bool $expected): void
262262
{
263263
$this->actAs($user)->withRole($role);
264-
Assert::same($expected, $this->canAccess('/admin'));
264+
Assert::same($this->canAccess('/admin'), $expected);
265265
}
266266

267267
// users × roles produces all user-role combinations,

‎docs/plugins/inline.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ For complex checks, pass a closure as the second parameter, where you can perfor
7979
#[TestInline(
8080
arguments: ['john.doe@example.com'],
8181
result: static function (User $user) {
82-
Assert::same('john.doe@example.com', $user->email);
82+
Assert::same($user->email, 'john.doe@example.com');
8383
Assert::true($user->isActive);
8484
Assert::notNull($user->createdAt);
8585
}

‎docs/plugins/lifecycle.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ final class DatabaseTest
123123
public function insertsRecord(): void
124124
{
125125
self::$connection->insert('users', ['name' => 'John']);
126-
Assert::same(1, self::$connection->count('users'));
126+
Assert::same(self::$connection->count('users'), 1);
127127
}
128128
}
129129
```

‎docs/plugins/repeat.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ Run the same test 5 times — any single failure fails the whole test:
2424
public function orderCalculationIsStable(): void
2525
{
2626
$order = new Order([new Item('A', 10), new Item('B', 20)]);
27-
Assert::same(30, $order->total());
27+
Assert::same($order->total(), 30);
2828
}
2929
```
3030
</example>
@@ -36,7 +36,7 @@ Tolerate up to two failures out of ten — useful when the underlying system is
3636
public function externalServiceReturnsExpectedShape(): void
3737
{
3838
$response = HttpClient::get('https://api.example.com/users/1');
39-
Assert::same(200, $response->statusCode);
39+
Assert::same($response->statusCode, 200);
4040
}
4141
```
4242
</example>

‎docs/plugins/retry.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ Retry a test up to 3 times:
2323
public function flakyExternalService(): void
2424
{
2525
$response = HttpClient::get('https://api.example.com/health');
26-
Assert::same(200, $response->statusCode);
26+
Assert::same($response->statusCode, 200);
2727
}
2828
```
2929
</example>

‎docs/theory/mutation-testing.md‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -171,10 +171,10 @@ Now write the "obvious" test suite, one test per branch:
171171
```php
172172
public function testGrades(): void
173173
{
174-
Assert::same('A', gradeScore(95));
175-
Assert::same('B', gradeScore(85));
176-
Assert::same('C', gradeScore(75));
177-
Assert::same('F', gradeScore(50));
174+
Assert::same(gradeScore(95), 'A');
175+
Assert::same(gradeScore(85), 'B');
176+
Assert::same(gradeScore(75), 'C');
177+
Assert::same(gradeScore(50), 'F');
178178
}
179179
```
180180

@@ -197,9 +197,9 @@ Let's close it:
197197
```php
198198
public function testGradeBoundaries(): void
199199
{
200-
Assert::same('A', gradeScore(90)); // lower bound for A
201-
Assert::same('B', gradeScore(80)); // lower bound for B
202-
Assert::same('C', gradeScore(70)); // lower bound for C
200+
Assert::same(gradeScore(90), 'A'); // lower bound for A
201+
Assert::same(gradeScore(80), 'B'); // lower bound for B
202+
Assert::same(gradeScore(70), 'C'); // lower bound for C
203203
}
204204
```
205205

‎ru/docs/bridge/revolt.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public function resolvesAfterDelay(): void
4040
$suspension = EventLoop::getSuspension();
4141
EventLoop::delay(0.1, static fn() => $suspension->resume('ready'));
4242

43-
Assert::same('ready', $suspension->suspend()); // цикл крутится, пока мы ждём
43+
Assert::same($suspension->suspend(), 'ready'); // цикл крутится, пока мы ждём
4444
}
4545
```
4646

0 commit comments

Comments
 (0)