Skip to content

Commit cef3a1d

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

File tree

4 files changed

+252
-0
lines changed

4 files changed

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

0 commit comments

Comments
 (0)