m105andClaude Sonnet 5
42abc658f2
Posts get a nullable image_url. WordPressPublisher now downloads it and uploads it to WordPress's media endpoint before creating the post, setting it as featured_media — and rejects publishing (without touching status) if a blog post has no image set, mirroring the existing title requirement. Every other network treats it as optional. Not enforced at save time, same as title — you can still draft an incomplete WordPress post, it just can't be published until an image is set. Exposed through the UI form (with a thumbnail/""missing"" indicator in the table), the JSON API, and the Python CLI wrapper. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
101 lines
3.9 KiB
PHP
101 lines
3.9 KiB
PHP
<?php
|
|
|
|
use App\Jobs\PublishPostJob;
|
|
use App\Models\Post;
|
|
use App\Models\Site;
|
|
use App\Models\SocialAccount;
|
|
use App\Support\Publishing\PublisherManager;
|
|
use Illuminate\Support\Facades\Http;
|
|
use Illuminate\Support\Facades\Process;
|
|
|
|
test('it publishes and updates the post on success', function () {
|
|
Http::fake([
|
|
'https://images.example.com/*' => Http::response('fake-image-bytes', 200, ['Content-Type' => 'image/jpeg']),
|
|
'*/wp-json/wp/v2/media' => Http::response(['id' => 99], 201),
|
|
'*/wp-json/wp/v2/posts' => Http::response(['link' => 'https://wp.example.com/post'], 201),
|
|
]);
|
|
|
|
$site = Site::factory()->wordpress()->create();
|
|
$post = Post::factory()->create(['site_id' => $site->id, 'type' => 'blog', 'title' => 'Hi', 'image_url' => 'https://images.example.com/photo.jpg', 'status' => 'publishing']);
|
|
|
|
(new PublishPostJob($post->id))->handle(app(PublisherManager::class));
|
|
|
|
expect($post->fresh())
|
|
->status->toBe('published')
|
|
->published_url->toBe('https://wp.example.com/post')
|
|
->publish_error->toBeNull();
|
|
});
|
|
|
|
test('it marks the post failed when the publish attempt fails', function () {
|
|
Http::fake([
|
|
'https://images.example.com/*' => Http::response('fake-image-bytes', 200, ['Content-Type' => 'image/jpeg']),
|
|
'*/wp-json/wp/v2/media' => Http::response(['id' => 99], 201),
|
|
'*/wp-json/wp/v2/posts' => Http::response(['message' => 'nope'], 401),
|
|
]);
|
|
|
|
$site = Site::factory()->wordpress()->create();
|
|
$post = Post::factory()->create(['site_id' => $site->id, 'type' => 'blog', 'title' => 'Hi', 'image_url' => 'https://images.example.com/photo.jpg', 'status' => 'publishing']);
|
|
|
|
(new PublishPostJob($post->id))->handle(app(PublisherManager::class));
|
|
|
|
expect($post->fresh())
|
|
->status->toBe('failed')
|
|
->publish_error->toBe('nope');
|
|
});
|
|
|
|
test('it reverts a rejected post to draft instead of failed', function () {
|
|
$site = Site::factory()->wordpress()->create();
|
|
$post = Post::factory()->create(['site_id' => $site->id, 'type' => 'blog', 'title' => null, 'status' => 'publishing']);
|
|
|
|
(new PublishPostJob($post->id))->handle(app(PublisherManager::class));
|
|
|
|
expect($post->fresh())
|
|
->status->toBe('draft')
|
|
->publish_error->toContain('no title');
|
|
});
|
|
|
|
test('it reverts to draft when no publisher supports the post type', function () {
|
|
$post = Post::factory()->create(['type' => 'twitter', 'status' => 'publishing']);
|
|
|
|
(new PublishPostJob($post->id))->handle(app(PublisherManager::class));
|
|
|
|
expect($post->fresh())
|
|
->status->toBe('draft')
|
|
->publish_error->toContain('No publisher available');
|
|
});
|
|
|
|
test('it skips a duplicate job for a post no longer in publishing state', function () {
|
|
Http::fake([
|
|
'*/wp-json/wp/v2/posts' => Http::response(['link' => 'https://wp.example.com/post'], 201),
|
|
]);
|
|
|
|
$site = Site::factory()->wordpress()->create();
|
|
$post = Post::factory()->create(['site_id' => $site->id, 'type' => 'blog', 'title' => 'Hi', 'status' => 'published', 'published_url' => 'https://wp.example.com/already-there']);
|
|
|
|
(new PublishPostJob($post->id))->handle(app(PublisherManager::class));
|
|
|
|
Http::assertNothingSent();
|
|
expect($post->fresh()->published_url)->toBe('https://wp.example.com/already-there');
|
|
});
|
|
|
|
test('it publishes a reddit post via the sidecar', function () {
|
|
Process::fake([
|
|
'*' => Process::result(output: json_encode(['success' => true, 'url' => 'https://reddit.com/r/test/comments/abc'])),
|
|
]);
|
|
|
|
$account = SocialAccount::factory()->create(['network' => 'reddit']);
|
|
$post = Post::factory()->create([
|
|
'type' => 'reddit',
|
|
'title' => 'Hi',
|
|
'target' => 'test',
|
|
'social_account_id' => $account->id,
|
|
'status' => 'publishing',
|
|
]);
|
|
|
|
(new PublishPostJob($post->id))->handle(app(PublisherManager::class));
|
|
|
|
expect($post->fresh())
|
|
->status->toBe('published')
|
|
->published_url->toBe('https://reddit.com/r/test/comments/abc');
|
|
});
|