[15.x] Stops using loadMigrationsFrom (#1613)

* Stops using `loadMigrationsFrom`

* Adjusts upgrade guide

* Bumps testbench

UPGRADE.md

@@ -1,5 +1,15 @@
 # Upgrade Guide
 
+## Upgrading To 15.0 From 14.x
+
+### Migration Changes
+
+Cashier 15.0 no longer automatically loads migrations from its own migrations directory. Instead, you should run the following command to publish Cashier's migrations to your application:
+
+```bash
+php artisan vendor:publish --tag=cashier-migrations
+```
+
 ## Upgrading To 14.12.2 From 14.12
 
 ### Webhook Added

composer.json

@@ -39,7 +39,7 @@
     "require-dev": {
         "dompdf/dompdf": "^2.0",
         "mockery/mockery": "^1.0",
-        "orchestra/testbench": "^7.0|^8.0",
+        "orchestra/testbench": "^7.14|^8.14",
         "phpstan/phpstan": "^1.10",
         "phpunit/phpunit": "^9.0"
     },

src/Cashier.php

@@ -42,13 +42,6 @@ class Cashier
      */
     protected static $formatCurrencyUsing;
 
-    /**
-     * Indicates if Cashier migrations will be run.
-     *
-     * @var bool
-     */
-    public static $runsMigrations = true;
-
     /**
      * Indicates if Cashier routes will be registered.
      *
@@ -173,18 +166,6 @@ public static function formatAmount($amount, $currency = null, $locale = null, a
         return $moneyFormatter->format($money);
     }
 
-    /**
-     * Configure Cashier to not register its migrations.
-     *
-     * @return static
-     */
-    public static function ignoreMigrations()
-    {
-        static::$runsMigrations = false;
-
-        return new static;
-    }
-
     /**
      * Configure Cashier to not register its routes.
      *

src/CashierServiceProvider.php

@@ -22,7 +22,6 @@ public function boot()
         $this->registerLogger();
         $this->registerRoutes();
         $this->registerResources();
-        $this->registerMigrations();
         $this->registerPublishing();
         $this->registerCommands();
 
@@ -123,18 +122,6 @@ protected function registerResources()
         $this->loadViewsFrom(__DIR__.'/../resources/views', 'cashier');
     }
 
-    /**
-     * Register the package migrations.
-     *
-     * @return void
-     */
-    protected function registerMigrations()
-    {
-        if (Cashier::$runsMigrations && $this->app->runningInConsole()) {
-            $this->loadMigrationsFrom(__DIR__.'/../database/migrations');
-        }
-    }
-
     /**
      * Register the package's publishable resources.
      *

testbench.yaml

@@ -0,0 +1,5 @@
+providers:
+  - Laravel\Cashier\CashierServiceProvider
+
+migrations:
+  - database/migrations

tests/Feature/FeatureTestCase.php

@@ -6,13 +6,14 @@
 use Laravel\Cashier\Cashier;
 use Laravel\Cashier\Tests\Fixtures\User;
 use Laravel\Cashier\Tests\TestCase;
+use Orchestra\Testbench\Concerns\WithLaravelMigrations;
 use Stripe\ApiRequestor as StripeApiRequestor;
 use Stripe\HttpClient\CurlClient as StripeCurlClient;
 use Stripe\StripeClient;
 
 abstract class FeatureTestCase extends TestCase
 {
-    use RefreshDatabase;
+    use RefreshDatabase, WithLaravelMigrations;
 
     protected function setUp(): void
     {
@@ -27,11 +28,6 @@ protected function setUp(): void
         StripeApiRequestor::setHttpClient($curl);
     }
 
-    protected function defineDatabaseMigrations()
-    {
-        $this->loadLaravelMigrations();
-    }
-
     protected static function stripe(array $options = []): StripeClient
     {
         return Cashier::stripe(array_merge(['api_key' => getenv('STRIPE_SECRET')], $options));

tests/TestCase.php

@@ -5,12 +5,14 @@
 use Illuminate\Support\Str;
 use InvalidArgumentException;
 use Laravel\Cashier\Cashier;
-use Laravel\Cashier\CashierServiceProvider;
 use Laravel\Cashier\Tests\Fixtures\User;
+use Orchestra\Testbench\Concerns\WithWorkbench;
 use Orchestra\Testbench\TestCase as OrchestraTestCase;
 
 abstract class TestCase extends OrchestraTestCase
 {
+    use WithWorkbench;
+
     protected function getEnvironmentSetUp($app)
     {
         $apiKey = config('cashier.secret');
@@ -21,9 +23,4 @@ protected function getEnvironmentSetUp($app)
 
         Cashier::useCustomerModel(User::class);
     }
-
-    protected function getPackageProviders($app)
-    {
-        return [CashierServiceProvider::class];
-    }
 }