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.
159 lines
5.1 KiB
PHP
159 lines
5.1 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
use App\Enums\User\Setup;
|
|
use App\Enums\UserWorkspace\Role as WorkspaceRole;
|
|
use App\Mail\WorkspaceInvite as WorkspaceInviteMail;
|
|
use App\Models\User;
|
|
use App\Models\Workspace;
|
|
use App\Models\WorkspaceInvite;
|
|
use Illuminate\Support\Facades\Mail;
|
|
|
|
beforeEach(function () {
|
|
Mail::fake();
|
|
$this->user = User::factory()->create(['setup' => Setup::Completed]);
|
|
$this->workspace = Workspace::factory()->create(['user_id' => $this->user->id]);
|
|
$this->user->update(['current_workspace_id' => $this->workspace->id]);
|
|
});
|
|
|
|
// Index tests
|
|
test('members index requires authentication', function () {
|
|
$response = $this->get(route('app.members'));
|
|
|
|
$response->assertRedirect(route('login'));
|
|
});
|
|
|
|
test('members index redirects to workspace settings', function () {
|
|
$response = $this->actingAs($this->user)->get(route('app.members'));
|
|
|
|
$response->assertRedirect(route('app.workspace.settings'));
|
|
});
|
|
|
|
test('workspace settings shows members and invites', function () {
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
]);
|
|
|
|
$response = $this->actingAs($this->user)->get(route('app.workspace.settings'));
|
|
|
|
$response->assertOk();
|
|
$response->assertInertia(fn ($page) => $page
|
|
->component('settings/Workspace', false)
|
|
->has('workspace')
|
|
->has('members')
|
|
->has('invitations')
|
|
->has('timezones')
|
|
);
|
|
});
|
|
|
|
// Store invite tests
|
|
test('store invite requires authentication', function () {
|
|
$response = $this->post(route('app.invites.store'), [
|
|
'email' => 'test@example.com',
|
|
'role' => WorkspaceRole::Member->value,
|
|
]);
|
|
|
|
$response->assertRedirect(route('login'));
|
|
});
|
|
|
|
test('store invite creates invite and sends email', function () {
|
|
$response = $this->actingAs($this->user)->post(route('app.invites.store'), [
|
|
'email' => 'newmember@example.com',
|
|
'role' => WorkspaceRole::Member->value,
|
|
]);
|
|
|
|
$response->assertRedirect();
|
|
|
|
$this->assertDatabaseHas('workspace_invites', [
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'newmember@example.com',
|
|
]);
|
|
|
|
Mail::assertQueued(WorkspaceInviteMail::class);
|
|
});
|
|
|
|
test('store invite fails if invite already exists', function () {
|
|
WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'email' => 'existing@example.com',
|
|
]);
|
|
|
|
$response = $this->actingAs($this->user)->post(route('app.invites.store'), [
|
|
'email' => 'existing@example.com',
|
|
'role' => WorkspaceRole::Member->value,
|
|
]);
|
|
|
|
$response->assertSessionHasErrors('email');
|
|
});
|
|
|
|
test('store invite fails if user is already member', function () {
|
|
$member = User::factory()->create(['setup' => Setup::Completed]);
|
|
$this->workspace->members()->attach($member->id, ['role' => WorkspaceRole::Member->value]);
|
|
|
|
$response = $this->actingAs($this->user)->post(route('app.invites.store'), [
|
|
'email' => $member->email,
|
|
'role' => WorkspaceRole::Member->value,
|
|
]);
|
|
|
|
$response->assertSessionHasErrors('email');
|
|
});
|
|
|
|
// Destroy invite tests
|
|
test('destroy invite requires authentication', function () {
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
]);
|
|
|
|
$response = $this->delete(route('app.invites.destroy', $invite));
|
|
|
|
$response->assertRedirect(route('login'));
|
|
});
|
|
|
|
test('destroy invite deletes invite', function () {
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
]);
|
|
|
|
$response = $this->actingAs($this->user)->delete(route('app.invites.destroy', $invite));
|
|
|
|
$response->assertRedirect();
|
|
expect(WorkspaceInvite::find($invite->id))->toBeNull();
|
|
});
|
|
|
|
test('destroy invite returns 404 for other workspace invite', function () {
|
|
$otherWorkspace = Workspace::factory()->create();
|
|
$invite = WorkspaceInvite::factory()->create([
|
|
'workspace_id' => $otherWorkspace->id,
|
|
]);
|
|
|
|
$response = $this->actingAs($this->user)->delete(route('app.invites.destroy', $invite));
|
|
|
|
$response->assertNotFound();
|
|
});
|
|
|
|
// Remove member tests
|
|
test('remove member requires authentication', function () {
|
|
$member = User::factory()->create(['setup' => Setup::Completed]);
|
|
$this->workspace->members()->attach($member->id, ['role' => WorkspaceRole::Member->value]);
|
|
|
|
$response = $this->delete(route('app.members.remove', $member));
|
|
|
|
$response->assertRedirect(route('login'));
|
|
});
|
|
|
|
test('remove member removes user from workspace', function () {
|
|
$member = User::factory()->create(['setup' => Setup::Completed]);
|
|
$this->workspace->members()->attach($member->id, ['role' => WorkspaceRole::Member->value]);
|
|
|
|
$response = $this->actingAs($this->user)->delete(route('app.members.remove', $member));
|
|
|
|
$response->assertRedirect();
|
|
expect($this->workspace->hasMember($member))->toBeFalse();
|
|
});
|
|
|
|
test('remove member fails for owner', function () {
|
|
$response = $this->actingAs($this->user)->delete(route('app.members.remove', $this->user));
|
|
|
|
$response->assertSessionHasErrors('member');
|
|
});
|