Skip to content

Commit b78b49a

Browse files
committed
Updated tests
1 parent d6581f3 commit b78b49a

File tree

12 files changed

+46
-38
lines changed

12 files changed

+46
-38
lines changed

app/Filament/Resources/DealResource/RelationManagers/ProductsRelationManager.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ public function form(Schema $schema): Schema
4848
->default(1)
4949
->reactive()
5050
->afterStateUpdated(fn($state, callable $set, callable $get) =>
51-
$set('total_amount', (string)((float)$get('price_per_unit') * (int)$state))),
51+
$set('total_amount', (string)($get('price_per_unit') * $state))),
5252
TextInput::make('price_per_unit')
5353
->mask(RawJs::make('$money($input)'))
5454
->stripCharacters(',')

database/factories/DealFactory.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,8 @@ public function definition()
1919
return [
2020
'title' => fake()->sentence(fake()->numberBetween(2, 4)),
2121
'customer_id' => fake()->numberBetween(1, 25),
22-
'estimated_revenue' => fake()->numberBetween(10000, 100000),
23-
'actual_revenue' => fake()->numberBetween(10000, 100000),
22+
'estimated_revenue' => fake()->randomFloat(1, 10000, 100000),
23+
'actual_revenue' => fake()->randomFloat(1, 10000, 100000),
2424
'status' => fake()->numberBetween(1, 3)
2525
];
2626
}

database/factories/LeadFactory.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ public function definition()
2020
'title' => fake()->sentence(fake()->numberBetween(2, 4)),
2121
'customer_id' => fake()->numberBetween(1, 25),
2222
'source' => fake()->numberBetween(1, 5),
23-
'estimated_revenue' => fake()->numberBetween(10000, 100000),
24-
'description' => fake()->realText(200),
23+
'estimated_revenue' => fake()->randomFloat(1, 10000, 100000),
24+
'description' => fake()->sentence(10),
2525
'status' => fake()->numberBetween(1, 4),
2626
'created_at' => fake()->dateTimeBetween('-1 years')
2727
];

database/factories/ProductFactory.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ public function definition()
2121
'name' => fake()->sentence(rand(2, 3)),
2222
'product_id' => 'PRO-' . strtoupper(Str::random(8)),
2323
'type' => rand(1, 2),
24-
'price' => fake()->numberBetween(500, 5000),
24+
'price' => fake()->randomFloat(1, 500, 5000),
2525
'is_available' => fake()->boolean(90)
2626
];
2727
}

phpunit.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
<env name="APP_ENV" value="testing"/>
1414
<env name="BCRYPT_ROUNDS" value="4"/>
1515
<env name="CACHE_DRIVER" value="array"/>
16-
<!-- <env name="DB_CONNECTION" value="sqlite"/>
17-
<env name="DB_DATABASE" value=":memory:"/> -->
16+
<env name="DB_CONNECTION" value="sqlite"/>
17+
<env name="DB_DATABASE" value=":memory:"/>
1818
<env name="MAIL_MAILER" value="array"/>
1919
<env name="QUEUE_CONNECTION" value="sync"/>
2020
<env name="SESSION_DRIVER" value="array"/>

tests/Feature/AccountTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
});
2626

2727
it('can list', function () {
28-
$accounts = Account::factory(10)->create();
28+
$accounts = Account::take(10)->get();
2929

3030
livewire(AccountResource\Pages\ListAccounts::class)
3131
->assertCanSeeTableRecords($accounts);

tests/Feature/ContactTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
});
2323

2424
it('can list', function () {
25-
$contacts = Contact::factory(10)->create();
25+
$contacts = Contact::take(10)->get();
2626

2727
livewire(ContactResource\Pages\ListContacts::class)
2828
->assertCanSeeTableRecords($contacts);

tests/Feature/DealTest.php

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,6 @@
1414

1515
beforeEach(function() {
1616
login();
17-
18-
Account::factory(25)->create();
1917
});
2018

2119
it('can see', function() {
@@ -39,7 +37,7 @@
3937
});
4038

4139
it('can list', function () {
42-
$deals = Deal::factory(10)->create();
40+
$deals = Deal::take(10)->get();
4341

4442
livewire(DealResource\Pages\ListDeals::class)
4543
->assertCanSeeTableRecords($deals);
@@ -136,8 +134,8 @@
136134
->title->toBe($newData->title)
137135
->customer_id->toBe($newData->customer_id)
138136
->source->toBe($newData->source)
139-
->actual_revenue->toBe((string)$newData->actual_revenue)
140-
->estimated_revenue->toBe((string)$newData->estimated_revenue)
137+
->actual_revenue->toBe($newData->actual_revenue)
138+
->estimated_revenue->toBe($newData->estimated_revenue)
141139
->status->value->toBe(DealStatus::Open->value);
142140
});
143141

tests/Feature/LeadTest.php

Lines changed: 27 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@
1111

1212
beforeEach(function() {
1313
login();
14-
15-
Account::factory(25)->create();
1614
});
1715

1816
it('can see', function() {
@@ -40,7 +38,7 @@
4038
});
4139

4240
it('can list', function () {
43-
$lead = Lead::factory(10)->create();
41+
$lead = Lead::take(10)->get();
4442

4543
livewire(LeadResource\Pages\ListLeads::class)
4644
->assertCanSeeTableRecords($lead);
@@ -61,7 +59,17 @@
6159
'customer_id' => $newData->customer_id,
6260
'source' => $newData->source,
6361
'estimated_revenue' => $newData->estimated_revenue,
64-
'description' => $newData->description,
62+
'description' => [
63+
'type' => 'doc',
64+
'content' => [
65+
[
66+
'type' => 'paragraph',
67+
'content' => [
68+
['type' => 'text', 'text' => $newData->description],
69+
],
70+
],
71+
],
72+
],
6573
'status' => $newData->status,
6674
])
6775
->call('create')
@@ -72,7 +80,7 @@
7280
'customer_id' => $newData->customer_id,
7381
'source' => $newData->source,
7482
'estimated_revenue' => (string)$newData->estimated_revenue,
75-
'description' => $newData->description,
83+
'description' => '<p>' . $newData->description . '</p>',
7684
'status' => $newData->status,
7785
]);
7886
});
@@ -111,7 +119,7 @@
111119
'customer_id' => $lead->customer_id,
112120
'status' => $lead->status->value,
113121
'source' => $lead->source,
114-
'description' => $lead->description,
122+
'description' => '<p>' . $lead->description . '</p>',
115123
'estimated_revenue' => $lead->estimated_revenue,
116124
]);
117125
});
@@ -129,7 +137,17 @@
129137
'title' => $newData->title,
130138
'customer_id' => $newData->customer_id,
131139
'source' => $newData->source,
132-
'description' => $newData->description,
140+
'description' => [
141+
'type' => 'doc',
142+
'content' => [
143+
[
144+
'type' => 'paragraph',
145+
'content' => [
146+
['type' => 'text', 'text' => $newData->description],
147+
],
148+
],
149+
],
150+
],
133151
'estimated_revenue' => $newData->estimated_revenue,
134152
'status' => LeadStatus::Open->value
135153
])
@@ -140,8 +158,8 @@
140158
->title->toBe($newData->title)
141159
->customer_id->toBe($newData->customer_id)
142160
->source->toBe($newData->source)
143-
->description->toBe($newData->description)
144-
->estimated_revenue->toBe((string)$newData->estimated_revenue)
161+
->description->toBe('<p>' . $newData->description . '</p>')
162+
->estimated_revenue->toBe($newData->estimated_revenue)
145163
->status->value->toBe(LeadStatus::Open->value);
146164
});
147165

tests/Feature/ProductTest.php

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
});
2222

2323
it('can list', function () {
24-
$product = Product::factory(10)->create();
24+
$product = Product::take(10)->get();
2525

2626
livewire(ProductResource\Pages\ListProducts::class)
2727
->assertCanSeeTableRecords($product);
@@ -113,13 +113,7 @@
113113
expect($product->refresh())
114114
->name->toBe($newData->name)
115115
->product_id->toBe($newData->product_id)
116-
/**
117-
* Casting the retruned value to string.
118-
*
119-
* See - https://github.com/laravel/framework/issues/3548
120-
*
121-
*/
122-
->price->toBe((string)$newData->price)
116+
->price->toBe($newData->price)
123117
->type->toBe($newData->type);
124118
});
125119

0 commit comments

Comments
 (0)