Skip to content

Commit a84d28e

Browse files
zigzagdevGromNaN
authored andcommitted
feat: refactoring-about-model-test
1 parent 61a3b77 commit a84d28e

File tree

1 file changed

+31
-31
lines changed

1 file changed

+31
-31
lines changed

tests/ModelTest.php

Lines changed: 31 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,13 @@
4949

5050
class ModelTest extends TestCase
5151
{
52-
public function tearDown(): void
52+
protected function setUp(): void
53+
{
54+
parent::setUp();
55+
Carbon::setTestNow();
56+
}
57+
58+
protected function tearDown(): void
5359
{
5460
Carbon::setTestNow();
5561
DB::connection('mongodb')->getCollection('users')->drop();
@@ -83,12 +89,24 @@ public function testQualifyColumn(): void
8389
$this->assertEquals('users.name', $sqlUser->qualifyColumn('name'));
8490
}
8591

92+
private function makeUser(array $overrides = []): User
93+
{
94+
$defaults = [
95+
'name' => 'John Doe',
96+
'title' => 'admin',
97+
'age' => 35,
98+
];
99+
100+
$user = new User();
101+
foreach (array_merge($defaults, $overrides) as $key => $value) {
102+
$user->$key = $value;
103+
}
104+
return $user;
105+
}
106+
86107
public function testInsert(): void
87108
{
88-
$user = new User();
89-
$user->name = 'John Doe';
90-
$user->title = 'admin';
91-
$user->age = 35;
109+
$user = $this->makeUser();
92110

93111
$user->save();
94112

@@ -130,10 +148,7 @@ public function testInsertNonIncrementable(): void
130148

131149
public function testUpdate(): void
132150
{
133-
$user = new User();
134-
$user->name = 'John Doe';
135-
$user->title = 'admin';
136-
$user->age = 35;
151+
$user = $this->makeUser();
137152
$user->save();
138153

139154
$raw = $user->getAttributes();
@@ -264,10 +279,7 @@ public function testManualIntId(): void
264279

265280
public function testDelete(): void
266281
{
267-
$user = new User();
268-
$user->name = 'John Doe';
269-
$user->title = 'admin';
270-
$user->age = 35;
282+
$user = $this->makeUser();
271283
$user->save();
272284

273285
$this->assertTrue($user->exists);
@@ -280,10 +292,7 @@ public function testDelete(): void
280292

281293
public function testAll(): void
282294
{
283-
$user = new User();
284-
$user->name = 'John Doe';
285-
$user->title = 'admin';
286-
$user->age = 35;
295+
$user = $this->makeUser();
287296
$user->save();
288297

289298
$user = new User();
@@ -301,10 +310,7 @@ public function testAll(): void
301310

302311
public function testFind(): void
303312
{
304-
$user = new User();
305-
$user->name = 'John Doe';
306-
$user->title = 'admin';
307-
$user->age = 35;
313+
$user = $this->makeUser();
308314
$user->save();
309315

310316
$check = User::find($user->id);
@@ -384,10 +390,7 @@ public function testCreate(): void
384390

385391
public function testDestroy(): void
386392
{
387-
$user = new User();
388-
$user->name = 'John Doe';
389-
$user->title = 'admin';
390-
$user->age = 35;
393+
$user = $this->makeUser();
391394
$user->save();
392395

393396
User::destroy((string) $user->id);
@@ -397,10 +400,7 @@ public function testDestroy(): void
397400

398401
public function testTouch(): void
399402
{
400-
$user = new User();
401-
$user->name = 'John Doe';
402-
$user->title = 'admin';
403-
$user->age = 35;
403+
$user = $this->makeUser();
404404
$user->save();
405405

406406
$old = $user->updated_at;
@@ -1089,7 +1089,7 @@ public function testChunkById(): void
10891089
$this->assertEquals(['fork', 'spork', 'spoon'], $names);
10901090
}
10911091

1092-
public function testTruncateModel()
1092+
public function testTruncateModel(): void
10931093
{
10941094
User::create(['name' => 'John Doe']);
10951095

@@ -1098,7 +1098,7 @@ public function testTruncateModel()
10981098
$this->assertEquals(0, User::count());
10991099
}
11001100

1101-
public function testGuardedModel()
1101+
public function testGuardedModel(): void
11021102
{
11031103
$model = new Guarded();
11041104

0 commit comments

Comments
 (0)