WIP

Author: stevebauman

tests/BartenderManagerTest.php

@@ -3,23 +3,34 @@
 use DirectoryTree\Bartender\BartenderManager;
 use DirectoryTree\Bartender\Facades\Bartender;
 use DirectoryTree\Bartender\Tests\User;
+use Illuminate\Support\Facades\Route;
 
-test('it can register handlers', function () {
+it('is bound to facade', function () {
+    expect(Bartender::getFacadeRoot())->toBeInstanceOf(BartenderManager::class);
+});
+
+it('can register handlers', function () {
     $manager = new BartenderManager();
 
     $manager->serve('foo', stdClass::class);
 
     expect($manager->handlers())->toBe(['foo' => stdClass::class]);
 });
 
-test('it returns new user model', function () {
+it('returns new user model', function () {
     $manager = new BartenderManager();
 
     $manager->setUserModel(User::class);
 
     expect($manager->getUserModel())->toBe(User::class);
 });
 
-test('it is bound to facade', function () {
-    expect(Bartender::getFacadeRoot())->toBeInstanceOf(BartenderManager::class);
+it('registers routes', function () {
+    $manager = new BartenderManager();
+
+    $manager->routes();
+
+    expect(Route::getRoutes())->toHaveCount(2);
+    expect(Route::has('auth.driver.callback'))->toBeTrue();
+    expect(Route::has('auth.driver.redirect'))->toBeTrue();
 });

tests/UserProviderQueryTest.php tests/UserProviderRepositoryTest.php

@@ -9,11 +9,6 @@
 beforeEach(fn () => Bartender::setUserModel(User::class));
 
 it('determines if user already exists with a different provider', function () {
-    $socialite = tap(new SocialiteUser(), function ($user) {
-        $user->id = '1';
-        $user->email = '[email protected]';
-    });
-
     User::create([
         'provider_id' => '1',
         'provider_name' => 'foo',
@@ -22,15 +17,15 @@
         'password' => bcrypt('password'),
     ]);
 
-    expect((new UserProviderRepository)->exists('bar', $socialite))->toBeTrue();
-});
-
-it('determines if user already exists with no provider', function () {
     $socialite = tap(new SocialiteUser(), function ($user) {
         $user->id = '1';
         $user->email = '[email protected]';
     });
 
+    expect((new UserProviderRepository)->exists('bar', $socialite))->toBeTrue();
+});
+
+it('determines if user already exists with no provider', function () {
     User::create([
         'provider_id' => null,
         'provider_name' => null,
@@ -39,6 +34,11 @@
         'password' => bcrypt('password'),
     ]);
 
+    $socialite = tap(new SocialiteUser(), function ($user) {
+        $user->id = '1';
+        $user->email = '[email protected]';
+    });
+
     expect((new UserProviderRepository)->exists('bar', $socialite))->toBeTrue();
 });
 
@@ -58,12 +58,6 @@
 });
 
 it('updates user not associated to provider', function () {
-    $socialite = tap(new SocialiteUser(), function (SocialiteUser $user) {
-        $user->id = '1';
-        $user->name = 'foo';
-        $user->email = '[email protected]';
-    });
-
     User::create([
         'provider_id' => '1',
         'provider_name' => 'foo',
@@ -72,6 +66,12 @@
         'password' => 'password',
     ]);
 
+    $socialite = tap(new SocialiteUser(), function (SocialiteUser $user) {
+        $user->id = '1';
+        $user->name = 'foo';
+        $user->email = '[email protected]';
+    });
+
     $user = (new UserProviderRepository)->updateOrCreate('foo', $socialite);
 
     expect($user->wasRecentlyCreated)->toBeFalse();
@@ -82,12 +82,6 @@
 });
 
 it('throws exception when attempting to create existing user with null provider', function () {
-    $socialite = tap(new SocialiteUser(), function (SocialiteUser $user) {
-        $user->id = '1';
-        $user->name = 'foo';
-        $user->email = '[email protected]';
-    });
-
     User::create([
         'name' => 'bar',
         'email' => '[email protected]',
@@ -96,16 +90,16 @@
 
     $this->expectException(QueryException::class);
 
-    (new UserProviderRepository)->updateOrCreate('foo', $socialite);
-});
-
-it('throws exception when attempting to create existing user with another provider', function () {
     $socialite = tap(new SocialiteUser(), function (SocialiteUser $user) {
-        $user->id = '123';
+        $user->id = '1';
         $user->name = 'foo';
         $user->email = '[email protected]';
     });
 
+    (new UserProviderRepository)->updateOrCreate('foo', $socialite);
+});
+
+it('throws exception when attempting to create existing user with another provider', function () {
     User::create([
         'name' => 'bar',
         'provider_id' => '456',
@@ -114,6 +108,12 @@
         'password' => 'password',
     ]);
 
+    $socialite = tap(new SocialiteUser(), function (SocialiteUser $user) {
+        $user->id = '123';
+        $user->name = 'bar';
+        $user->email = '[email protected]';
+    });
+
     $this->expectException(QueryException::class);
 
     (new UserProviderRepository)->updateOrCreate('bar', $socialite);