Passed
Pull Request — master (#75)
by Stephen
14:12
created

UserNotificationFactory   A

Complexity

Total Complexity 1

Size/Duplication

Total Lines 23
Duplicated Lines 0 %

Importance

Changes 0
Metric Value
eloc 7
dl 0
loc 23
rs 10
c 0
b 0
f 0
wmc 1

1 Method

Rating   Name   Duplication   Size   Complexity  
A definition() 0 4 1
1
<?php
2
3
namespace Sfneal\Users\Factories;
4
5
use Illuminate\Database\Eloquent\Factories\Factory;
6
use Sfneal\Users\Models\UserNotification;
7
8
class UserNotificationFactory extends Factory
9
{
10
    public const TYPES = [
11
        'Domain\Clients\Notifications\NewInquiryNotification',
12
        'Domain\Plans\Notifications\BucketChangedNotification',
13
    ];
14
15
    /**
16
     * The name of the factory's corresponding model.
17
     *
18
     * @var string
19
     */
20
    protected $model = UserNotification::class;
21
22
    /**
23
     * Define the model's default state.
24
     *
25
     * @return array
26
     */
27
    public function definition(): array
28
    {
29
        return [
30
            'type' => $this->faker->randomElement(self::TYPES),
0 ignored issues
show
Bug introduced by
The call to Faker\Generator::randomElement() has too few arguments starting with 'b'. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

30
            'type' => $this->faker->/** @scrutinizer ignore-call */ randomElement(self::TYPES),

This check compares calls to functions or methods with their respective definitions. If the call has less arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
31
        ];
32
    }
33
}
34