Skip to content

Commit d23ecc6

Browse files
TreggatsTreggats
committed
add Exists validation rule with comprehensive tests
Signed-off-by: Tonko Mulder <[email protected]>
1 parent cf504ad commit d23ecc6

File tree

4 files changed

+275
-0
lines changed

4 files changed

+275
-0
lines changed

‎packages/validation/src/Rules/Exists.php‎

Lines changed: 43 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,43 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace Tempest\Validation\Rules;
6+
7+
use Attribute;
8+
use InvalidArgumentException;
9+
use Tempest\Database\Id;
10+
use Tempest\Validation\Rule;
11+
12+
use function Tempest\Database\query;
13+
14+
#[Attribute(Attribute::TARGET_PROPERTY)]
15+
final readonly class Exists implements Rule
16+
{
17+
public function __construct(
18+
private string $model,
19+
) {
20+
if (! class_exists($this->model)) {
21+
throw new InvalidArgumentException("Model {$this->model} does not exist");
22+
}
23+
}
24+
25+
public function isValid(mixed $value): bool
26+
{
27+
if ((! is_numeric($value) || is_float($value)) && ! is_object($value)) {
28+
return false;
29+
}
30+
$id = is_object($value) ? $value : new Id($value);
31+
32+
$model = query($this->model)
33+
->select()
34+
->get(id: $id);
35+
36+
return $model !== null;
37+
}
38+
39+
public function message(): string
40+
{
41+
return sprintf('Record for model %1$s does not exist', $this->model);
42+
}
43+
}

‎packages/validation/tests/Rules/ExistsTest.php‎

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,56 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace Tempest\Validation\Tests\Rules;
6+
7+
use InvalidArgumentException;
8+
use PHPUnit\Framework\Attributes\Test;
9+
use PHPUnit\Framework\TestCase;
10+
use Tempest\Validation\Rules\Exists;
11+
use Tests\Tempest\Fixtures\Rules\ValidateExistsModel;
12+
13+
/**
14+
* @internal
15+
*/
16+
final class ExistsTest extends TestCase
17+
{
18+
#[Test]
19+
public function throws_exception_for_invalid_model_class(): void
20+
{
21+
$this->expectException(InvalidArgumentException::class);
22+
$this->expectExceptionMessage('Model NonExistentModel does not exist');
23+
24+
new Exists('NonExistentModel');
25+
}
26+
27+
#[Test]
28+
public function returns_false_for_null_or_non_integer_values(): void
29+
{
30+
$rule = new Exists(ValidateExistsModel::class);
31+
32+
$this->assertFalse($rule->isValid('string'));
33+
$this->assertFalse($rule->isValid(1.5));
34+
$this->assertFalse($rule->isValid([]));
35+
$this->assertFalse($rule->isValid(true));
36+
$this->assertFalse($rule->isValid(null));
37+
}
38+
39+
#[Test]
40+
public function returns_correct_error_message(): void
41+
{
42+
$rule = new Exists(ValidateExistsModel::class);
43+
44+
$expectedMessage = sprintf('Record for model %s does not exist', ValidateExistsModel::class);
45+
$this->assertSame($expectedMessage, $rule->message());
46+
}
47+
48+
#[Test]
49+
public function can_be_constructed_with_valid_model_class(): void
50+
{
51+
$rule = new Exists(ValidateExistsModel::class);
52+
53+
$this->assertInstanceOf(Exists::class, $rule);
54+
$this->assertStringContainsString(ValidateExistsModel::class, $rule->message());
55+
}
56+
}

‎tests/Fixtures/Rules/ValidateExistsModel.php‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace Tests\Tempest\Fixtures\Rules;
6+
7+
use Tempest\Database\Id;
8+
9+
/** @internal */
10+
final class ValidateExistsModel
11+
{
12+
public function __construct(
13+
public Id $id,
14+
public string $name,
15+
) {}
16+
}

‎tests/Integration/Http/Rules/ExistsRuleTest.php‎

Lines changed: 160 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,160 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace Tests\Tempest\Integration\Http\Rules;
6+
7+
use PHPUnit\Framework\Attributes\Test;
8+
use Tempest\Database\Migrations\CreateMigrationsTable;
9+
use Tempest\Validation\Rules\Exists;
10+
use Tests\Tempest\Fixtures\Migrations\CreateAuthorTable;
11+
use Tests\Tempest\Fixtures\Migrations\CreateBookTable;
12+
use Tests\Tempest\Fixtures\Modules\Books\Models\Author;
13+
use Tests\Tempest\Fixtures\Modules\Books\Models\Book;
14+
use Tests\Tempest\Integration\FrameworkIntegrationTestCase;
15+
16+
/**
17+
* @internal
18+
*/
19+
final class ExistsRuleTest extends FrameworkIntegrationTestCase
20+
{
21+
#[Test]
22+
public function validates_existing_record_returns_true(): void
23+
{
24+
$this->migrate(
25+
CreateMigrationsTable::class,
26+
CreateBookTable::class,
27+
);
28+
29+
$book = Book::create(title: 'Timeline Taxi');
30+
31+
$rule = new Exists(Book::class);
32+
33+
$this->assertTrue($rule->isValid($book->id));
34+
}
35+
36+
#[Test]
37+
public function validates_non_existent_record_returns_false(): void
38+
{
39+
$this->migrate(
40+
CreateMigrationsTable::class,
41+
CreateBookTable::class,
42+
);
43+
44+
$rule = new Exists(Book::class);
45+
46+
$this->assertFalse($rule->isValid(99999));
47+
$this->assertFalse($rule->isValid(12345));
48+
}
49+
50+
#[Test]
51+
public function validates_multiple_existing_records(): void
52+
{
53+
$this->migrate(
54+
CreateMigrationsTable::class,
55+
CreateBookTable::class,
56+
);
57+
58+
$book1 = Book::create(title: 'The Lord of the Rings');
59+
$book2 = Book::create(title: 'The Silmarillion');
60+
$book3 = Book::create(title: 'Unfinished Tales');
61+
62+
$rule = new Exists(Book::class);
63+
64+
$this->assertTrue($rule->isValid($book1->id->id));
65+
$this->assertTrue($rule->isValid($book2->id->id));
66+
$this->assertTrue($rule->isValid($book3->id->id));
67+
68+
$this->assertFalse($rule->isValid(99999));
69+
}
70+
71+
#[Test]
72+
public function validates_different_model_types(): void
73+
{
74+
$this->migrate(
75+
CreateMigrationsTable::class,
76+
CreateAuthorTable::class,
77+
CreateBookTable::class,
78+
);
79+
80+
$author = Author::create(name: 'B. Roose');
81+
$book = Book::create(title: 'Timeline Taxi');
82+
83+
$authorRule = new Exists(Author::class);
84+
$bookRule = new Exists(Book::class);
85+
86+
$this->assertTrue($authorRule->isValid($author->id->id));
87+
$this->assertTrue($bookRule->isValid($book->id->id));
88+
89+
$this->assertFalse($authorRule->isValid(99999));
90+
$this->assertFalse($bookRule->isValid(99999));
91+
92+
$author2 = Author::create(name: 'B. Roose');
93+
$book2 = Book::create(title: 'Timeline Taxi 2');
94+
95+
$this->assertTrue($authorRule->isValid($author2->id->id));
96+
$this->assertTrue($bookRule->isValid($book2->id->id));
97+
}
98+
99+
#[Test]
100+
public function validates_edge_cases_with_large_id_numbers(): void
101+
{
102+
$this->migrate(
103+
CreateMigrationsTable::class,
104+
CreateBookTable::class,
105+
);
106+
107+
$rule = new Exists(Book::class);
108+
109+
$this->assertFalse($rule->isValid(PHP_INT_MAX));
110+
$this->assertFalse($rule->isValid(999999999));
111+
$this->assertFalse($rule->isValid(2147483647)); // Max 32-bit integer
112+
}
113+
114+
#[Test]
115+
public function validates_after_record_deletion(): void
116+
{
117+
$this->migrate(
118+
CreateMigrationsTable::class,
119+
CreateBookTable::class,
120+
);
121+
122+
$book = Book::create(title: 'Timeline Taxi Draft');
123+
$bookId = $book->id->id;
124+
125+
$rule = new Exists(Book::class);
126+
127+
$this->assertTrue($rule->isValid($bookId));
128+
129+
$book->delete();
130+
131+
$this->assertFalse($rule->isValid($bookId));
132+
}
133+
134+
#[Test]
135+
public function validates_with_sequential_id_creation(): void
136+
{
137+
$this->migrate(
138+
CreateMigrationsTable::class,
139+
CreateBookTable::class,
140+
);
141+
142+
$rule = new Exists(Book::class);
143+
$createdIds = [];
144+
145+
for ($i = 1; $i <= 5; $i++) {
146+
$book = Book::create(title: "Book {$i}");
147+
$createdIds[] = $book->id->id;
148+
149+
$this->assertTrue($rule->isValid($book->id->id));
150+
}
151+
152+
foreach ($createdIds as $id) {
153+
$this->assertTrue($rule->isValid($id));
154+
}
155+
156+
$maxId = max($createdIds);
157+
$this->assertFalse($rule->isValid($maxId + 1));
158+
$this->assertFalse($rule->isValid($maxId + 100));
159+
}
160+
}

0 commit comments

Comments
 (0)