Got back to using the new Boot phase for the authorization

Author: edgrosvenor

packages/till/src/Listeners/AuthorizesLedgerTransactionsListener.php

@@ -14,7 +14,7 @@
 
 class AuthorizesLedgerTransactionsListener
 {
-    #[On(Phase::Authorize)]
+    #[On(Phase::Boot)]
     public function authorizeLedgerTransaction(Event $event): void
     {
         $reflection = new ReflectionClass($event);

packages/till/src/Providers/TillServiceProvider.php

@@ -30,6 +30,7 @@ public function boot(): void
         app(Dispatcher::class)->register(new AuthorizesLedgerTransactionsListener);
         app(Dispatcher::class)->register(new ProcessesLedgerTransactionsListener);
         app(Dispatcher::class)->register(new ResetsAbilities);
+
         if ($this->app->runningInConsole()) {
             $this->commands([
                 InstallCommand::class,

packages/till/src/SubscriptionPlans/Abilities/Spend.php

@@ -67,6 +67,7 @@ public function __invoke(State $state, Ledgers $ledger, int $attempted_spend = 1
         $wallet_balance = data_get($state->wallet, $ledger->name, 0);
 
         // Don't have enough to do this even with the wallet balance
+
         if ($wallet_balance + $pre_run_balance < $attempted_spend) {
             return false;
         }

packages/till/tests/LedgersTest.php

@@ -20,7 +20,6 @@
 });
 describe('usage ledger on the subscriber state', function (): void {
     it('adds a single transaction', function (): void {
-
         RickRollSent::commit(
             subscriber_id: 1,
         );
@@ -143,8 +142,7 @@
     });
 
     it('resets when expected', function (): void {
-
-        FiveRickRollsSent::commit(
+        FiveRickRollsSent::fire(
             subscriber_id: 1,
         );
 
@@ -155,7 +153,7 @@
 
         Carbon::setTestNow(now()->addMonth()->addMinute());
 
-        FiveRickRollsSent::commit(
+        FiveRickRollsSent::fire(
             subscriber_id: 1,
         );