Skip to content

Commit 0d30047

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

File tree

5 files changed

+285
-0
lines changed

5 files changed

+285
-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/Fixtures/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 Tempest\Validation\Tests\Fixtures;
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+
}

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

0 commit comments

Comments
 (0)