Merge pull request #3 from laravelcm/subscribe-test

mckenziearts · web-flow · commit eb938e3954cf · 2023-10-02T11:00:38.000+02:00
Add Pest and setup subscribe test
diff --git a/composer.json b/composer.json
@@ -54,7 +54,8 @@
   },
   "autoload-dev": {
     "psr-4": {
-      "Tests\\": "tests"
+      "Tests\\": "tests/src",
+      "Tests\\Database\\Factories\\": "tests/database/factories"
     }
   },
   "scripts": {
diff --git a/config/laravel-subscriptions.php b/config/laravel-subscriptions.php
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -5,7 +5,7 @@
          colors="true"
          backupStaticProperties="true">
     <testsuites>
-        <testsuite name="Test Suite">
+        <testsuite name="Subscription Test Suite">
             <directory suffix="Test.php">./tests/Feature</directory>
         </testsuite>
     </testsuites>
@@ -15,11 +15,11 @@
         </exclude>
     </source>
     <php>
-        <server name="APP_ENV" value="testing"/>
-        <server name="CACHE_DRIVER" value="array"/>
-        <server name="SESSION_DRIVER" value="array"/>
-        <server name="QUEUE_CONNECTION" value="sync"/>
-        <server name="DB_CONNECTION" value="sqlite"/>
-        <server name="DB_DATABASE" value=":memory:"/>
+        <env name="APP_ENV" value="testing"/>
+        <env name="CACHE_DRIVER" value="array"/>
+        <env name="SESSION_DRIVER" value="array"/>
+        <env name="QUEUE_CONNECTION" value="sync"/>
+        <env name="DB_CONNECTION" value="sqlite"/>
+        <env name="DB_DATABASE" value=":memory:"/>
     </php>
 </phpunit>
diff --git a/src/SubscriptionServiceProvider.php b/src/SubscriptionServiceProvider.php
@@ -12,7 +12,7 @@ final class SubscriptionServiceProvider extends PackageServiceProvider
     public function configurePackage(Package $package): void
     {
         $package->name('laravel-subscriptions')
-            ->hasConfigFile();
+            ->hasConfigFile('laravel-subscriptions');
     }
 
     public function bootingPackage(): void
diff --git a/src/Traits/HasPlanSubscriptions.php b/src/Traits/HasPlanSubscriptions.php
@@ -13,8 +13,6 @@
 
 trait HasPlanSubscriptions
 {
-    abstract public function morphMany(string $related, string $name, ?string $type, ?string $id, ?string $localKey = null): MorphMany;
-
     protected static function bootHasSubscriptions(): void
     {
         static::deleted(function ($plan): void {
diff --git a/tests/Feature/ExampleTest.php b/tests/Feature/ExampleTest.php
diff --git a/tests/Feature/SubscribeTest.php b/tests/Feature/SubscribeTest.php
@@ -0,0 +1,17 @@
+<?php
+
+declare(strict_types=1);
+
+use Tests\Models\User;
+
+it('User model has plan subscription trait or implement subscription methods', function (): void {
+    expect(User::factory()->create())
+        ->toHaveMethods([
+            'activePlanSubscriptions',
+            'planSubscription',
+            'planSubscriptions',
+            'newPlanSubscription',
+            'subscribedPlans',
+            'subscribedTo',
+        ]);
+});
diff --git a/tests/Pest.php b/tests/Pest.php
@@ -2,9 +2,8 @@
 
 declare(strict_types=1);
 
-use Illuminate\Foundation\Testing\RefreshDatabase;
 use Tests\TestCase;
 
-uses(TestCase::class, RefreshDatabase::class)->in('Feature');
+uses(TestCase::class)->in('Feature');
 
 expect()->extend('toBeOne', fn () => $this->toBe(1));
diff --git a/tests/TestCase.php b/tests/TestCase.php
diff --git a/tests/database/factories/UserFactory.php b/tests/database/factories/UserFactory.php
@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Database\Factories;
+
+use Tests\Models\User;
+
+final class UserFactory extends \Orchestra\Testbench\Factories\UserFactory
+{
+    public function modelName(): string
+    {
+        return User::class;
+    }
+}
diff --git a/tests/src/Models/User.php b/tests/src/Models/User.php
@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Models;
+
+use Illuminate\Database\Eloquent\Factories\HasFactory;
+use Illuminate\Foundation\Auth\User as Authenticatable;
+use Laravelcm\Subscriptions\Traits\HasPlanSubscriptions;
+use Tests\Database\Factories\UserFactory;
+
+final class User extends Authenticatable
+{
+    use HasFactory;
+    use HasPlanSubscriptions;
+
+    /**
+     * The attributes that are mass assignable.
+     *
+     * @var array<int, string>
+     */
+    protected $fillable = [
+        'name',
+        'email',
+        'password',
+    ];
+
+    /**
+     * The attributes that should be hidden for serialization.
+     *
+     * @var array<int, string>
+     */
+    protected $hidden = [
+        'password',
+        'remember_token',
+    ];
+
+    /**
+     * The attributes that should be cast.
+     *
+     * @var array<string, string>
+     */
+    protected $casts = [
+        'email_verified_at' => 'datetime',
+    ];
+
+    protected static function newFactory(): UserFactory
+    {
+        return UserFactory::new();
+    }
+}
diff --git a/tests/src/TestCase.php b/tests/src/TestCase.php
@@ -0,0 +1,32 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests;
+
+use Illuminate\Foundation\Testing\LazilyRefreshDatabase;
+use Laravelcm\Subscriptions\SubscriptionServiceProvider;
+use Orchestra\Testbench\TestCase as BaseTestCase;
+use Tests\Models\User;
+
+abstract class TestCase extends BaseTestCase
+{
+    use LazilyRefreshDatabase;
+
+    protected function getPackageProviders($app): array
+    {
+        return [
+            SubscriptionServiceProvider::class,
+        ];
+    }
+
+    protected function defineDatabaseMigrations(): void
+    {
+        $this->loadLaravelMigrations();
+    }
+
+    protected function getEnvironmentSetUp($app): void
+    {
+        $app['config']->set('auth.providers.users.model', User::class);
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ final class SubscriptionServiceProvider extends PackageServiceProvider`
`12`	`12`	`public function configurePackage(Package $package): void`
`13`	`13`	`{`
`14`	`14`	`$package->name('laravel-subscriptions')`
`15`		`- ->hasConfigFile();`
	`15`	`+ ->hasConfigFile('laravel-subscriptions');`
`16`	`16`	`}`
`17`	`17`
`18`	`18`	`public function bootingPackage(): void`
Original file line number	Diff line number	Diff line change
`@@ -13,8 +13,6 @@`
`13`	`13`
`14`	`14`	`trait HasPlanSubscriptions`
`15`	`15`	`{`
`16`		`- abstract public function morphMany(string $related, string $name, ?string $type, ?string $id, ?string $localKey = null): MorphMany;`
`17`		`-`
`18`	`16`	`protected static function bootHasSubscriptions(): void`
`19`	`17`	`{`
`20`	`18`	`static::deleted(function ($plan): void {`