Settings pages: - Redesign layout to match Sendkit (max-w-4xl, space-y-12, Separator sections) - Merge Members page into Workspace settings with Table, invite Dialog, ConfirmDeleteModal - Add workspace logo upload/delete routes and controller methods - Translate all hardcoded strings in Workspace.vue modals Language system: - Drop languages table, replace language_id FK with locale string column on users - Create config/languages.php for available languages and default locale - Add Spanish (es) translations (13 files) - Simplify HandleInertiaRequests, ProfileController, RegisteredUserController Code quality: - Add declare(strict_types=1) to all PHP files - Fix MastodonPublisher using wrong attribute (filename -> original_filename) - Fix HasMediaTest for new has_photo/photo_url accessors - Fix PublishToSocialPlatformTest type error revealed by strict_types - Remove orphaned Language model from AppServiceProvider morph map - Update User TypeScript interface (has_photo, photo_url, locale) - Eager load media relation on workspaces to prevent N+1 - Add 8 new tests for workspace logo upload/delete - Update workspace settings test to assert members/invitations props All 710 tests passing.
197 lines
6.1 KiB
PHP
197 lines
6.1 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
use App\Enums\User\Setup;
|
|
use App\Enums\UserWorkspace\Role as WorkspaceRole;
|
|
use App\Models\User;
|
|
use App\Models\Workspace;
|
|
use App\Models\WorkspaceInvite;
|
|
|
|
beforeEach(function () {
|
|
$this->owner = User::factory()->create(['setup' => Setup::Completed]);
|
|
$this->workspace = Workspace::factory()->create(['user_id' => $this->owner->id]);
|
|
});
|
|
|
|
test('show invite displays invite details for guest', function () {
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'newuser@example.com',
|
|
'role' => WorkspaceRole::Member,
|
|
]);
|
|
|
|
$response = $this->get(route('app.invites.show', $invite));
|
|
|
|
$response->assertOk();
|
|
$response->assertInertia(fn ($page) => $page
|
|
->component('auth/AcceptInvite', false)
|
|
->has('invite')
|
|
->where('invite.id', $invite->id)
|
|
->where('invite.email', 'newuser@example.com')
|
|
->where('invite.role.value', WorkspaceRole::Member->value)
|
|
->where('invite.workspace.name', $this->workspace->name)
|
|
);
|
|
});
|
|
|
|
test('show invite displays invite details for authenticated user', function () {
|
|
$user = User::factory()->create([
|
|
'email' => 'invitee@example.com',
|
|
'setup' => Setup::Completed,
|
|
]);
|
|
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'invitee@example.com',
|
|
'role' => WorkspaceRole::Member,
|
|
]);
|
|
|
|
$response = $this->actingAs($user)->get(route('app.invites.show', $invite));
|
|
|
|
$response->assertOk();
|
|
$response->assertInertia(fn ($page) => $page
|
|
->component('auth/AcceptInvite', false)
|
|
->has('invite')
|
|
);
|
|
});
|
|
|
|
test('show invite returns 404 for non-existent invite', function () {
|
|
$response = $this->get(route('app.invites.show', 'non-existent-uuid'));
|
|
|
|
$response->assertNotFound();
|
|
});
|
|
|
|
test('accept invite requires authentication', function () {
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
]);
|
|
|
|
$response = $this->post(route('app.invites.accept', $invite));
|
|
|
|
$response->assertRedirect(route('login'));
|
|
});
|
|
|
|
test('accept invite adds user to workspace', function () {
|
|
$user = User::factory()->create([
|
|
'email' => 'invitee@example.com',
|
|
'setup' => Setup::Completed,
|
|
]);
|
|
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'invitee@example.com',
|
|
'role' => WorkspaceRole::Admin,
|
|
]);
|
|
|
|
$response = $this->actingAs($user)->post(route('app.invites.accept', $invite));
|
|
|
|
$response->assertRedirect(route('app.calendar'));
|
|
|
|
// User should be member of workspace
|
|
expect($this->workspace->hasMember($user))->toBeTrue();
|
|
|
|
// Invite should be deleted
|
|
expect(WorkspaceInvite::find($invite->id))->toBeNull();
|
|
|
|
// User's current workspace should be updated
|
|
$user->refresh();
|
|
expect($user->current_workspace_id)->toBe($this->workspace->id);
|
|
});
|
|
|
|
test('accept invite fails for wrong email', function () {
|
|
$user = User::factory()->create([
|
|
'email' => 'different@example.com',
|
|
'setup' => Setup::Completed,
|
|
]);
|
|
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'invitee@example.com',
|
|
'role' => WorkspaceRole::Member,
|
|
]);
|
|
|
|
$response = $this->actingAs($user)->post(route('app.invites.accept', $invite));
|
|
|
|
$response->assertRedirect(route('app.calendar'));
|
|
$response->assertSessionHas('flash.bannerStyle', 'danger');
|
|
|
|
// Invite should NOT be deleted
|
|
expect(WorkspaceInvite::find($invite->id))->not->toBeNull();
|
|
});
|
|
|
|
test('accept invite handles already member', function () {
|
|
$user = User::factory()->create([
|
|
'email' => 'invitee@example.com',
|
|
'setup' => Setup::Completed,
|
|
]);
|
|
|
|
$this->workspace->members()->attach($user->id, ['role' => WorkspaceRole::Member->value]);
|
|
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'invitee@example.com',
|
|
'role' => WorkspaceRole::Admin,
|
|
]);
|
|
|
|
$response = $this->actingAs($user)->post(route('app.invites.accept', $invite));
|
|
|
|
$response->assertRedirect(route('app.calendar'));
|
|
$response->assertSessionHas('flash.bannerStyle', 'info');
|
|
|
|
// Invite should be deleted
|
|
expect(WorkspaceInvite::find($invite->id))->toBeNull();
|
|
});
|
|
|
|
test('decline invite requires authentication', function () {
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
]);
|
|
|
|
$response = $this->post(route('app.invites.decline', $invite));
|
|
|
|
$response->assertRedirect(route('login'));
|
|
});
|
|
|
|
test('decline invite deletes the invite', function () {
|
|
$user = User::factory()->create([
|
|
'email' => 'invitee@example.com',
|
|
'setup' => Setup::Completed,
|
|
]);
|
|
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'invitee@example.com',
|
|
'role' => WorkspaceRole::Member,
|
|
]);
|
|
|
|
$response = $this->actingAs($user)->post(route('app.invites.decline', $invite));
|
|
|
|
$response->assertRedirect(route('app.calendar'));
|
|
$response->assertSessionHas('flash.bannerStyle', 'info');
|
|
|
|
// Invite should be deleted
|
|
expect(WorkspaceInvite::find($invite->id))->toBeNull();
|
|
|
|
// User should NOT be member of workspace
|
|
expect($this->workspace->hasMember($user))->toBeFalse();
|
|
});
|
|
|
|
test('decline invite fails for wrong email', function () {
|
|
$user = User::factory()->create([
|
|
'email' => 'different@example.com',
|
|
'setup' => Setup::Completed,
|
|
]);
|
|
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'invitee@example.com',
|
|
'role' => WorkspaceRole::Member,
|
|
]);
|
|
|
|
$response = $this->actingAs($user)->post(route('app.invites.decline', $invite));
|
|
|
|
$response->assertRedirect(route('app.calendar'));
|
|
$response->assertSessionHas('flash.bannerStyle', 'danger');
|
|
|
|
// Invite should NOT be deleted
|
|
expect(WorkspaceInvite::find($invite->id))->not->toBeNull();
|
|
});
|