Allow StateId attribute to set an alias when using a single state (#102)

* failing test

* Fix styling

* Update StateAliasTest.php

* Get to green!

* Fix styling

---------

Co-authored-by: jdiddydave <[email protected]>
Co-authored-by: Chris Morrell <[email protected]>

Author: jdiddydave

src/Attributes/Autodiscovery/AppliesToState.php

@@ -38,7 +38,7 @@ public function discoverState(Event $event, StateManager $manager): array
         // TODO: Check type of data
 
         if (! is_array($id)) {
-            $this->alias = $this->inferAliasFromVariableName($property);
+            $this->alias ??= $this->inferAliasFromVariableName($property);
         }
 
         return collect(Arr::wrap($id))

src/Attributes/Autodiscovery/StateId.php

@@ -34,7 +34,7 @@ public function discoverState(Event $event, StateManager $manager): array
         }
 
         if (! is_array($id)) {
-            $this->alias = $this->inferAliasFromVariableName($this->property->getName());
+            $this->alias ??= $this->inferAliasFromVariableName($this->property->getName());
         }
 
         return collect(Arr::wrap($id))

tests/Feature/StateAliasTest.php

@@ -0,0 +1,35 @@
+<?php
+
+use Thunk\Verbs\Attributes\Autodiscovery\AppliesToState;
+use Thunk\Verbs\Attributes\Autodiscovery\StateId;
+use Thunk\Verbs\Event;
+use Thunk\Verbs\Facades\Verbs;
+use Thunk\Verbs\State;
+
+test('stateid attribute allows setting an alias for a state collection with a single state', function () {
+    $event = StateAliasTestEvent::fire(bar: 1, baz: 2);
+
+    Verbs::commit();
+
+    $state_collection = $event->states();
+
+    expect($state_collection->aliasNames())->not->toContain('bar')
+        ->and($state_collection->aliasNames())->not->toContain('baz')
+        ->and($state_collection->aliasNames())->toContain('foo')
+        ->and($state_collection->aliasNames())->toContain('hello');
+});
+
+class StateAliasTestState extends State
+{
+}
+
+#[AppliesToState(StateAliasTestState::class, id: 'baz', alias: 'hello')]
+class StateAliasTestEvent extends Event
+{
+    public function __construct(
+        #[StateId(StateAliasTestState::class, 'foo')]
+        public int $bar,
+        public int $baz,
+    ) {
+    }
+}