Replies: 1 comment 6 replies
-
|
There's a config setting that allows you to change the redirects for specific routes. Something like See here: socialstream/config/socialstream.php Lines 23 to 29 in d1c5cde |
Beta Was this translation helpful? Give feedback.
6 replies
-
|
@FoksVHox you need to update your tests |
Beta Was this translation helpful? Give feedback.
-
@joelbutcher I've updated the tests, and yet it is still giving the following stacktrace. The test file is looking like this: <?php
namespace Tests\Feature;
use Illuminate\Foundation\Testing\RefreshDatabase;
use Illuminate\Support\Facades\Session;
use JoelButcher\Socialstream\Providers;
use Laravel\Socialite\Facades\Socialite;
use Laravel\Socialite\Two\User;
use Mockery;
use function Pest\Laravel\get;
uses(RefreshDatabase::class);
test('users get redirected correctly', function (string $provider) {
if (! Providers::enabled($provider)) {
$this->markTestSkipped("Registration support with the $provider provider is not enabled.");
}
config()->set("services.$provider", [
'client_id' => 'client-id',
'client_secret' => 'client-secret',
'redirect' => "http://localhost/oauth/$provider/callback",
]);
$response = get("/oauth/$provider");
$response->assertRedirectContains($provider);
})->with([
['apple'],
[Providers::facebook()],
]);
test('users can register using socialite providers', function (string $socialiteProvider) {
if (! Providers::enabled($socialiteProvider)) {
$this->markTestSkipped("Registration support with the $socialiteProvider provider is not enabled.");
}
$user = (new User)
->map([
'id' => 'abcdefgh',
'nickname' => 'Jane',
'name' => 'Jane Doe',
'email' => '[email protected]',
'avatar' => null,
'avatar_original' => null,
])
->setToken('user-token')
->setRefreshToken('refresh-token')
->setExpiresIn(3600);
$provider = Mockery::mock('Laravel\\Socialite\\Two\\'.$socialiteProvider.'Provider');
$provider->shouldReceive('user')->once()->andReturn($user);
Socialite::shouldReceive('driver')->once()->with($socialiteProvider)->andReturn($provider);
Session::put('socialstream.previous_url', route('register'));
$response = get("/oauth/$socialiteProvider/callback");
$this->assertAuthenticated();
$response->assertRedirect(route('home', absolute: false));
})->with([
['apple'],
[Providers::facebook()],
]);The Route::get('/', function () {
return Inertia::render('Front');
})->name('home');As a fact I do belive it's an issue with SocialStream as the stacktrace is highlighting the following like of code within the |
Beta Was this translation helpful? Give feedback.
-
|
@joelbutcher Any suggestion? |
Beta Was this translation helpful? Give feedback.
-
|
Looks there is a bug in the route resolution when returning from oAuth. Please can you raise a new issue linking to this thread? Thanks! |
Beta Was this translation helpful? Give feedback.
-
|
@joelbutcher Opened! Sorry for the delay, been at vacation :D |
Beta Was this translation helpful? Give feedback.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hello!
How can I change the default behavior to redirect to
/dashboardwhen you've authenticated?Beta Was this translation helpful? Give feedback.
All reactions