d84666360a
Collapse LinkedIn to one content type per account kind (linkedin_post, linkedin_page_post). Publishers infer the publish format from the attached media — text, single image/video, multi-image carousel, or PDF document — matching how facebook_post/x_post already work; PDF is exclusive of any other attachment. Removes the editor variant picker, keeping only the PDF document title field. Includes a data migration collapsing the retired carousel/document content types. Replace the two LinkedIn account cards with a single Connect LinkedIn button: one unified OAuth grant (linkedin-openid driver, union of scopes) then a post-callback identity picker to post as the personal profile (linkedin) or a company page the member administers (linkedin-page). The chosen organization is validated against the admin-verified list from the OAuth grant. Per-capability gating via LINKEDIN_ENABLED / LINKEDIN_PAGE_ENABLED supports profile-only or org-only self-hosting. Removes LinkedInPageController, LinkedInTokenSynchronizer, the standalone linkedin-page connect routes, and the unused redirect_page config.
289 lines
10 KiB
PHP
289 lines
10 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
use App\Enums\Post\Status as PostStatus;
|
|
use App\Enums\PostPlatform\ContentType;
|
|
use App\Enums\SocialAccount\Platform;
|
|
use App\Enums\UserWorkspace\Role;
|
|
use App\Jobs\PublishPost;
|
|
use App\Mcp\Servers\TryPostServer;
|
|
use App\Mcp\Tools\Post\AttachMediaFromUploadTool;
|
|
use App\Mcp\Tools\Post\CreatePostTool;
|
|
use App\Mcp\Tools\Post\PublishPostTool;
|
|
use App\Mcp\Tools\Post\UpdatePostTool;
|
|
use App\Models\Post;
|
|
use App\Models\PostPlatform;
|
|
use App\Models\SocialAccount;
|
|
use App\Models\User;
|
|
use App\Models\Workspace;
|
|
use Illuminate\Support\Facades\Queue;
|
|
use Illuminate\Support\Str;
|
|
|
|
beforeEach(function () {
|
|
$this->user = User::factory()->create();
|
|
$this->workspace = Workspace::factory()->create(['user_id' => $this->user->id]);
|
|
$this->workspace->members()->attach($this->user->id, ['role' => Role::Member->value]);
|
|
$this->user->update(['current_workspace_id' => $this->workspace->id]);
|
|
|
|
$this->discordAccount = SocialAccount::factory()->discord()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'platform_user_id' => '111222333',
|
|
]);
|
|
});
|
|
|
|
test('create post persists Discord channel + embeds meta', function () {
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(CreatePostTool::class, [
|
|
'content' => 'Hello Discord',
|
|
'platforms' => [[
|
|
'social_account_id' => $this->discordAccount->id,
|
|
'content_type' => ContentType::DiscordMessage->value,
|
|
'meta' => [
|
|
'channel_id' => '444555666',
|
|
'embeds' => [['title' => 'Release']],
|
|
],
|
|
]],
|
|
]);
|
|
|
|
$response->assertOk();
|
|
|
|
$meta = PostPlatform::where('social_account_id', $this->discordAccount->id)->sole()->meta;
|
|
|
|
expect($meta['channel_id'])->toBe('444555666')
|
|
->and(data_get($meta, 'embeds.0.title'))->toBe('Release');
|
|
});
|
|
|
|
test('create post persists LinkedIn document_title meta', function () {
|
|
$linkedin = SocialAccount::factory()->create(['workspace_id' => $this->workspace->id, 'platform' => Platform::LinkedIn]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(CreatePostTool::class, [
|
|
'content' => 'Check our latest deck',
|
|
'platforms' => [[
|
|
'social_account_id' => $linkedin->id,
|
|
'content_type' => ContentType::LinkedInPost->value,
|
|
'meta' => ['document_title' => 'Q2 Report'],
|
|
]],
|
|
]);
|
|
|
|
$response->assertOk();
|
|
|
|
expect(PostPlatform::where('social_account_id', $linkedin->id)->sole()->meta['document_title'])->toBe('Q2 Report');
|
|
});
|
|
|
|
test('update post merges per-platform meta', function () {
|
|
$post = Post::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'user_id' => $this->user->id,
|
|
'status' => PostStatus::Draft,
|
|
]);
|
|
$platform = PostPlatform::factory()->discord()->create([
|
|
'post_id' => $post->id,
|
|
'social_account_id' => $this->discordAccount->id,
|
|
'enabled' => true,
|
|
'meta' => ['channel_name' => 'general'],
|
|
]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(UpdatePostTool::class, [
|
|
'post_id' => $post->id,
|
|
'platforms' => [[
|
|
'id' => $platform->id,
|
|
'meta' => ['channel_id' => '444555666'],
|
|
]],
|
|
]);
|
|
|
|
$response->assertOk();
|
|
|
|
$meta = $platform->fresh()->meta;
|
|
expect($meta['channel_id'])->toBe('444555666')
|
|
->and($meta['channel_name'])->toBe('general'); // merged, not overwritten
|
|
});
|
|
|
|
test('publish guard ignores disabled platforms missing meta', function () {
|
|
Queue::fake();
|
|
|
|
$linkedin = SocialAccount::factory()->create(['workspace_id' => $this->workspace->id, 'platform' => Platform::LinkedIn]);
|
|
|
|
$post = Post::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'user_id' => $this->user->id,
|
|
'status' => PostStatus::Draft,
|
|
]);
|
|
PostPlatform::factory()->linkedin()->create([
|
|
'post_id' => $post->id,
|
|
'social_account_id' => $linkedin->id,
|
|
'enabled' => true,
|
|
]);
|
|
// Disabled Discord with no channel must not block the publish.
|
|
PostPlatform::factory()->discord()->create([
|
|
'post_id' => $post->id,
|
|
'social_account_id' => $this->discordAccount->id,
|
|
'enabled' => false,
|
|
'meta' => [],
|
|
]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(PublishPostTool::class, ['post_id' => $post->id]);
|
|
|
|
$response->assertOk();
|
|
Queue::assertPushed(PublishPost::class);
|
|
});
|
|
|
|
test('publish post rejects a Discord platform without a channel', function () {
|
|
$post = Post::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'user_id' => $this->user->id,
|
|
'status' => PostStatus::Draft,
|
|
]);
|
|
PostPlatform::factory()->discord()->create([
|
|
'post_id' => $post->id,
|
|
'social_account_id' => $this->discordAccount->id,
|
|
'enabled' => true,
|
|
'meta' => [],
|
|
]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(PublishPostTool::class, ['post_id' => $post->id]);
|
|
|
|
$response->assertHasErrors([__('posts.form.discord.channel_required')]);
|
|
});
|
|
|
|
test('publish guard enforces required meta for TikTok and Pinterest', function (string $factoryState, string $field, string $messageKey) {
|
|
$account = SocialAccount::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'platform' => $factoryState === 'tiktok' ? Platform::TikTok : Platform::Pinterest,
|
|
]);
|
|
|
|
$post = Post::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'user_id' => $this->user->id,
|
|
'status' => PostStatus::Draft,
|
|
]);
|
|
PostPlatform::factory()->{$factoryState}()->create([
|
|
'post_id' => $post->id,
|
|
'social_account_id' => $account->id,
|
|
'enabled' => true,
|
|
'meta' => [],
|
|
]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(PublishPostTool::class, ['post_id' => $post->id]);
|
|
|
|
$response->assertHasErrors([__($messageKey)]);
|
|
})->with([
|
|
'tiktok' => ['tiktok', 'privacy_level', 'posts.form.tiktok.privacy_required'],
|
|
'pinterest' => ['pinterest', 'board_id', 'posts.form.pinterest.board_required'],
|
|
]);
|
|
|
|
test('attach media from upload accepts a PDF for a LinkedIn post', function () {
|
|
$linkedin = SocialAccount::factory()->create(['workspace_id' => $this->workspace->id, 'platform' => Platform::LinkedIn]);
|
|
|
|
$post = Post::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'user_id' => $this->user->id,
|
|
'status' => PostStatus::Draft,
|
|
]);
|
|
PostPlatform::factory()->create([
|
|
'post_id' => $post->id, 'social_account_id' => $linkedin->id,
|
|
'platform' => Platform::LinkedIn, 'content_type' => ContentType::LinkedInPost, 'enabled' => true,
|
|
]);
|
|
|
|
$uploadToken = (string) Str::uuid();
|
|
$this->workspace->media()->create([
|
|
'group_id' => (string) Str::uuid(),
|
|
'collection' => 'assets',
|
|
'type' => 'document',
|
|
'path' => 'medias/deck.pdf',
|
|
'original_filename' => 'deck.pdf',
|
|
'mime_type' => 'application/pdf',
|
|
'size' => 1000,
|
|
'order' => 0,
|
|
'upload_token' => $uploadToken,
|
|
]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(AttachMediaFromUploadTool::class, [
|
|
'post_id' => $post->id,
|
|
'upload_token' => $uploadToken,
|
|
]);
|
|
|
|
$response->assertOk();
|
|
|
|
$media = $post->fresh()->media;
|
|
expect($media)->toHaveCount(1)
|
|
->and($media[0]['type'])->toBe('document')
|
|
->and($media[0]['mime_type'])->toBe('application/pdf');
|
|
});
|
|
|
|
test('publish post succeeds for a LinkedIn document that has a PDF', function () {
|
|
Queue::fake();
|
|
|
|
$linkedin = SocialAccount::factory()->create(['workspace_id' => $this->workspace->id, 'platform' => Platform::LinkedIn]);
|
|
|
|
$post = Post::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'user_id' => $this->user->id,
|
|
'status' => PostStatus::Draft,
|
|
'media' => [[
|
|
'id' => 'doc-1', 'path' => 'medias/deck.pdf', 'url' => 'https://example.com/deck.pdf',
|
|
'type' => 'document', 'mime_type' => 'application/pdf', 'original_filename' => 'deck.pdf',
|
|
]],
|
|
]);
|
|
PostPlatform::factory()->create([
|
|
'post_id' => $post->id, 'social_account_id' => $linkedin->id,
|
|
'platform' => Platform::LinkedIn, 'content_type' => ContentType::LinkedInPost, 'enabled' => true,
|
|
]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(PublishPostTool::class, ['post_id' => $post->id]);
|
|
|
|
$response->assertOk();
|
|
Queue::assertPushed(PublishPost::class);
|
|
});
|
|
|
|
test('publish post rejects a LinkedIn post that mixes a PDF with an image', function () {
|
|
$linkedin = SocialAccount::factory()->create(['workspace_id' => $this->workspace->id, 'platform' => Platform::LinkedIn]);
|
|
|
|
$post = Post::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'user_id' => $this->user->id,
|
|
'status' => PostStatus::Draft,
|
|
'media' => [
|
|
['id' => 'doc-1', 'path' => 'medias/deck.pdf', 'url' => 'https://example.com/deck.pdf', 'type' => 'document', 'mime_type' => 'application/pdf', 'original_filename' => 'deck.pdf'],
|
|
['id' => 'img-1', 'path' => 'medias/slide.jpg', 'url' => 'https://example.com/slide.jpg', 'type' => 'image', 'mime_type' => 'image/jpeg', 'original_filename' => 'slide.jpg'],
|
|
],
|
|
]);
|
|
PostPlatform::factory()->create([
|
|
'post_id' => $post->id, 'social_account_id' => $linkedin->id,
|
|
'platform' => Platform::LinkedIn, 'content_type' => ContentType::LinkedInPost, 'enabled' => true,
|
|
]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(PublishPostTool::class, ['post_id' => $post->id]);
|
|
|
|
$response->assertHasErrors(['A PDF document must be the only attachment.']);
|
|
});
|
|
|
|
test('publish post succeeds for a Discord platform with a channel', function () {
|
|
Queue::fake();
|
|
|
|
$post = Post::factory()->create([
|
|
'workspace_id' => $this->workspace->id,
|
|
'user_id' => $this->user->id,
|
|
'status' => PostStatus::Draft,
|
|
]);
|
|
PostPlatform::factory()->discord()->create([
|
|
'post_id' => $post->id,
|
|
'social_account_id' => $this->discordAccount->id,
|
|
'enabled' => true,
|
|
'meta' => ['channel_id' => '444555666'],
|
|
]);
|
|
|
|
$response = TryPostServer::actingAs($this->user)
|
|
->tool(PublishPostTool::class, ['post_id' => $post->id]);
|
|
|
|
$response->assertOk();
|
|
Queue::assertPushed(PublishPost::class);
|
|
});
|