Skip to content

Commit bc13747

Browse files
rubenvanasschegithub-actions[bot]
authored andcommitted
Fix styling
1 parent b947774 commit bc13747

File tree

3 files changed

+6
-7
lines changed

3 files changed

+6
-7
lines changed

src/SettingsConfig.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ class SettingsConfig
1616
/** @var class-string<\Spatie\LaravelSettings\Settings> */
1717
private string $settingsClass;
1818

19-
/** @var array<string> */
19+
/** @var array<string> */
2020
private array $defaultValueLoadedProperties;
2121

2222
/** @var Collection<string, ?\Spatie\LaravelSettings\SettingsCasts\SettingsCast> */

src/SettingsMapper.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ private function ensureNoMissingSettings(
141141
->getReflectedProperties()
142142
->keys()
143143
->diff($properties->keys())
144-
->when($operation === 'saving', fn(Collection $collection) => $collection->concat($config->getDefaultValueLoadedProperties()))
144+
->when($operation === 'saving', fn (Collection $collection) => $collection->concat($config->getDefaultValueLoadedProperties()))
145145
->toArray();
146146

147147
if (! empty($missingSettings)) {

tests/SettingsTest.php

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
use Illuminate\Support\Facades\DB;
1717
use Illuminate\Support\Facades\Event;
1818
use Illuminate\Support\Str;
19-
use Spatie\LaravelSettings\Tests\TestClasses\DummySettingsWithDefaultValue;
2019
use function Pest\Laravel\assertDatabaseHas;
2120
use function Pest\Laravel\assertDatabaseMissing;
2221
use Spatie\LaravelSettings\Events\LoadingSettings;
@@ -35,6 +34,7 @@
3534
use Spatie\LaravelSettings\Tests\TestClasses\DummyIntEnum;
3635
use Spatie\LaravelSettings\Tests\TestClasses\DummySettings;
3736
use Spatie\LaravelSettings\Tests\TestClasses\DummySettingsWithCast;
37+
use Spatie\LaravelSettings\Tests\TestClasses\DummySettingsWithDefaultValue;
3838
use Spatie\LaravelSettings\Tests\TestClasses\DummySettingsWithRepository;
3939
use Spatie\LaravelSettings\Tests\TestClasses\DummySimpleSettings;
4040
use Spatie\LaravelSettings\Tests\TestClasses\DummyStringEnum;
@@ -101,8 +101,7 @@
101101
});
102102

103103
it('will fail loading settings when a default value and non default value is present', function () {
104-
$settings = new class extends DummySettingsWithDefaultValue
105-
{
104+
$settings = new class extends DummySettingsWithDefaultValue {
106105
public string $name;
107106
};
108107

@@ -200,15 +199,15 @@
200199
$settings->save();
201200
})->throws(MissingSettings::class);
202201

203-
it('cannot save a settings class whose default values are not migrated', function (){
202+
it('cannot save a settings class whose default values are not migrated', function () {
204203
$settings = resolve(DummySettingsWithDefaultValue::class);
205204

206205
$settings->site = 'flareapp.io';
207206

208207
$settings->save();
209208
})->throws(MissingSettings::class);
210209

211-
it('can save settings with a default value when correctly migrated', function (){
210+
it('can save settings with a default value when correctly migrated', function () {
212211
$settings = resolve(DummySettingsWithDefaultValue::class);
213212

214213
expect($settings->site)->toBe('spatie.be');

0 commit comments

Comments
 (0)