increase test coverage

Author: stevethomas

src/Casts/FeatureStateCast.php

@@ -8,12 +8,12 @@
 
 class FeatureStateCast implements CastsAttributes
 {
-    public function get($model, string $key, $value, array $attributes)
+    public function get($model, string $key, $value, array $attributes): FeatureState
     {
         return FeatureState::from($attributes['state']);
     }
 
-    public function set($model, string $key, $value, array $attributes)
+    public function set($model, string $key, $value, array $attributes): array
     {
         if (! $value instanceof FeatureState) {
             throw new InvalidArgumentException('The given value is not an instance of FeatureState.');

src/Models/Feature.php

@@ -2,10 +2,11 @@
 
 namespace Codinglabs\FeatureFlags\Models;
 
+use Illuminate\Database\Eloquent\Model;
 use Codinglabs\FeatureFlags\Casts\FeatureStateCast;
 use Illuminate\Database\Eloquent\Factories\HasFactory;
 
-class Feature extends \Illuminate\Database\Eloquent\Model
+class Feature extends Model
 {
     use HasFactory;
 

tests/FeatureFlagTest.php

@@ -19,6 +19,15 @@
     FeatureFlag::reset();
 });
 
+it('throws an exception if casting to a feature state that does not exist', function () {
+    $this->expectException(\InvalidArgumentException::class);
+
+    Feature::factory()->create([
+        'name' => 'some-feature',
+        'state' => 'foo',
+    ]);
+});
+
 it('throws an exception if calling isOn on a feature that does not exist', function () {
     $this->expectException(MissingFeatureException::class);
 
@@ -141,6 +150,60 @@
         ->and(FeatureFlag::getState('some-on-feature'))->toBe(FeatureState::on());
 });
 
+it('can turn on a feature', function () {
+    Event::fake();
+
+    Feature::factory()->create([
+        'name' => 'some-feature',
+        'state' => FeatureState::off()
+    ]);
+
+    cache()->store('array')->set('testing.some-feature', 'off');
+
+    FeatureFlag::turnOn('some-feature');
+
+    Event::assertDispatched(FeatureUpdatedEvent::class);
+    expect(FeatureFlag::isOn('some-feature'))->toBeTrue()
+        ->and(FeatureFlag::isOff('some-feature'))->toBeFalse()
+        ->and(cache()->store('array')->get('testing.some-feature'))->toBe(FeatureState::on()->value);
+});
+
+it('can turn off a feature', function () {
+    Event::fake();
+
+    Feature::factory()->create([
+        'name' => 'some-feature',
+        'state' => FeatureState::on()
+    ]);
+
+    cache()->store('array')->set('testing.some-feature', 'on');
+
+    FeatureFlag::turnOff('some-feature');
+
+    Event::assertDispatched(FeatureUpdatedEvent::class);
+    expect(FeatureFlag::isOn('some-feature'))->toBeFalse()
+        ->and(FeatureFlag::isOff('some-feature'))->toBeTrue()
+        ->and(cache()->store('array')->get('testing.some-feature'))->toBe(FeatureState::off()->value);
+});
+
+it('can make a feature dynamic', function () {
+    Event::fake();
+
+    Feature::factory()->create([
+        'name' => 'some-feature',
+        'state' => FeatureState::on()
+    ]);
+
+    cache()->store('array')->set('testing.some-feature', 'on');
+
+    FeatureFlag::makeDynamic('some-feature');
+
+    Event::assertDispatched(FeatureUpdatedEvent::class);
+    expect(FeatureFlag::isOn('some-feature'))->toBeFalse()
+        ->and(FeatureFlag::isOff('some-feature'))->toBeTrue()
+        ->and(cache()->store('array')->get('testing.some-feature'))->toBe(FeatureState::dynamic()->value);
+});
+
 it('can update a features state', function () {
     Event::fake();