Implement silent handling of same state transitions in StateManager and add corresponding test case

Author: nthndnn

src/StateManager.php

@@ -47,6 +47,10 @@ public function transition(
         $from = $this->getCurrentState($model, $field);
         $toValue = $this->extractStateValue($to);
 
+        if ($from === $toValue) {
+            return true;
+        }
+
         if (! $this->stateMachine->canTransition($from, $toValue)) {
             throw new InvalidStateTransitionException($from, $toValue, $field, get_class($model));
         }
@@ -78,7 +82,7 @@ public function transition(
 
             return true;
         } catch (\Exception $e) {
-            $model->setAttribute($field, $from);
+            $model->$field = $from;
             $model->save();
 
             throw $e;

tests/Feature/HasStateTest.php

@@ -285,6 +285,24 @@ public function it_uses_config_based_model_resolution()
         $relatedModel = $states->getRelated();
         $this->assertInstanceOf(config('state-history.model'), $relatedModel);
     }
+
+    #[Test]
+    public function it_handles_same_state_transitions_silently()
+    {
+        $model = new TestModel;
+        $model->current_state = TestState::Draft->value;
+        $model->save();
+
+        $initialCount = $model->states('state')->count();
+
+        $result = $model->transitionTo('state', TestState::Draft);
+
+        $this->assertTrue($result);
+
+        $this->assertEquals($initialCount, $model->states('state')->count());
+
+        $this->assertEquals(TestState::Draft->value, $model->fresh()->current_state);
+    }
 }
 
 enum TestState: string