Settings
diff --git a/tests/Feature/Livewire/Plugins/ConfigModalTest.php b/tests/Feature/Livewire/Plugins/ConfigModalTest.php
new file mode 100644
index 0000000..4372991
--- /dev/null
+++ b/tests/Feature/Livewire/Plugins/ConfigModalTest.php
@@ -0,0 +1,124 @@
+create();
+ $this->actingAs($user);
+
+ $plugin = Plugin::create([
+ 'uuid' => Str::uuid(),
+ 'user_id' => $user->id,
+ 'name' => 'Test Plugin',
+ 'data_strategy' => 'static',
+ 'configuration_template' => [
+ 'custom_fields' => [[
+ 'keyname' => 'tags',
+ 'field_type' => 'multi_string',
+ 'name' => 'Reading Days',
+ 'default' => 'alpha,beta',
+ ]]
+ ],
+ 'configuration' => ['tags' => 'alpha,beta']
+ ]);
+
+ Volt::test('plugins.config-modal', ['plugin' => $plugin])
+ ->assertSet('multiValues.tags', ['alpha', 'beta']);
+});
+
+test('config modal validates against commas in multi_string boxes', function (): void {
+ $user = User::factory()->create();
+ $this->actingAs($user);
+
+ $plugin = Plugin::create([
+ 'uuid' => Str::uuid(),
+ 'user_id' => $user->id,
+ 'name' => 'Test Plugin',
+ 'data_strategy' => 'static',
+ 'configuration_template' => [
+ 'custom_fields' => [[
+ 'keyname' => 'tags',
+ 'field_type' => 'multi_string',
+ 'name' => 'Reading Days',
+ ]]
+ ]
+ ]);
+
+ Volt::test('plugins.config-modal', ['plugin' => $plugin])
+ ->set('multiValues.tags.0', 'no,commas,allowed')
+ ->call('saveConfiguration')
+ ->assertHasErrors(['multiValues.tags.0' => 'regex']);
+
+ // Assert DB remains unchanged
+ expect($plugin->fresh()->configuration['tags'] ?? '')->not->toBe('no,commas,allowed');
+});
+
+test('config modal merges multi_string boxes into a single CSV string on save', function (): void {
+ $user = User::factory()->create();
+ $this->actingAs($user);
+
+ $plugin = Plugin::create([
+ 'uuid' => Str::uuid(),
+ 'user_id' => $user->id,
+ 'name' => 'Test Plugin',
+ 'data_strategy' => 'static',
+ 'configuration_template' => [
+ 'custom_fields' => [[
+ 'keyname' => 'items',
+ 'field_type' => 'multi_string',
+ 'name' => 'Reading Days',
+ ]]
+ ],
+ 'configuration' => []
+ ]);
+
+ Volt::test('plugins.config-modal', ['plugin' => $plugin])
+ ->set('multiValues.items.0', 'First')
+ ->call('addMultiItem', 'items')
+ ->set('multiValues.items.1', 'Second')
+ ->call('saveConfiguration')
+ ->assertHasNoErrors();
+
+ expect($plugin->fresh()->configuration['items'])->toBe('First,Second');
+});
+
+test('config modal resetForm clears dirty state and increments resetIndex', function (): void {
+ $user = User::factory()->create();
+ $this->actingAs($user);
+
+ $plugin = Plugin::create([
+ 'uuid' => Str::uuid(),
+ 'user_id' => $user->id,
+ 'name' => 'Test Plugin',
+ 'data_strategy' => 'static',
+ 'configuration' => ['simple_key' => 'original_value']
+ ]);
+
+ Volt::test('plugins.config-modal', ['plugin' => $plugin])
+ ->set('configuration.simple_key', 'dirty_value')
+ ->call('resetForm')
+ ->assertSet('configuration.simple_key', 'original_value')
+ ->assertSet('resetIndex', 1);
+});
+
+test('config modal dispatches update event for parent warning refresh', function (): void {
+ $user = User::factory()->create();
+ $this->actingAs($user);
+
+ $plugin = Plugin::create([
+ 'uuid' => Str::uuid(),
+ 'user_id' => $user->id,
+ 'name' => 'Test Plugin',
+ 'data_strategy' => 'static'
+ ]);
+
+ Volt::test('plugins.config-modal', ['plugin' => $plugin])
+ ->call('saveConfiguration')
+ ->assertDispatched('config-updated');
+});
diff --git a/tests/Feature/PluginImportTest.php b/tests/Feature/PluginImportTest.php
index 1b20f93..fae28a8 100644
--- a/tests/Feature/PluginImportTest.php
+++ b/tests/Feature/PluginImportTest.php
@@ -427,6 +427,65 @@ YAML;
->and($displayIncidentField['default'])->toBe('true');
});
+it('throws exception when multi_string default value contains a comma', function (): void {
+ $user = User::factory()->create();
+
+ // YAML with a comma in the 'default' field of a multi_string
+ $invalidYaml = <<
$invalidYaml,
+ 'src/full.liquid' => getValidFullLiquid(),
+ ]);
+
+ $zipFile = UploadedFile::fake()->createWithContent('invalid-default.zip', $zipContent);
+ $pluginImportService = new PluginImportService();
+
+ expect(fn () => $pluginImportService->importFromZip($zipFile, $user))
+ ->toThrow(Exception::class, "Validation Error: The default value for multistring fields like `api_key` cannot contain commas.");
+});
+
+it('throws exception when multi_string placeholder contains a comma', function (): void {
+ $user = User::factory()->create();
+
+ // YAML with a comma in the 'placeholder' field
+ $invalidYaml = << $invalidYaml,
+ 'src/full.liquid' => getValidFullLiquid(),
+ ]);
+
+ $zipFile = UploadedFile::fake()->createWithContent('invalid-placeholder.zip', $zipContent);
+ $pluginImportService = new PluginImportService();
+
+ expect(fn () => $pluginImportService->importFromZip($zipFile, $user))
+ ->toThrow(Exception::class, "Validation Error: The placeholder value for multistring fields like `api_key` cannot contain commas.");
+});
+
// Helper methods
function createMockZipFile(array $files): string
{
diff --git a/tests/Unit/Models/PluginTest.php b/tests/Unit/Models/PluginTest.php
index 0847e36..2771f81 100644
--- a/tests/Unit/Models/PluginTest.php
+++ b/tests/Unit/Models/PluginTest.php
@@ -737,3 +737,175 @@ test('plugin model preserves multi_string csv format', function (): void {
expect($plugin->fresh()->configuration['tags'])->toBe('laravel,pest,security');
});
+
+test('plugin duplicate copies all attributes except id and uuid', function (): void {
+ $user = User::factory()->create();
+
+ $original = Plugin::factory()->create([
+ 'user_id' => $user->id,
+ 'name' => 'Original Plugin',
+ 'data_stale_minutes' => 30,
+ 'data_strategy' => 'polling',
+ 'polling_url' => 'https://api.example.com/data',
+ 'polling_verb' => 'get',
+ 'polling_header' => 'Authorization: Bearer token123',
+ 'polling_body' => '{"query": "test"}',
+ 'render_markup' => 'Test markup
',
+ 'markup_language' => 'blade',
+ 'configuration' => ['api_key' => 'secret123'],
+ 'configuration_template' => [
+ 'custom_fields' => [
+ [
+ 'keyname' => 'api_key',
+ 'field_type' => 'string',
+ ],
+ ],
+ ],
+ 'no_bleed' => true,
+ 'dark_mode' => true,
+ 'data_payload' => ['test' => 'data'],
+ ]);
+
+ $duplicate = $original->duplicate();
+
+ // Refresh to ensure casts are applied
+ $original->refresh();
+ $duplicate->refresh();
+
+ expect($duplicate->id)->not->toBe($original->id)
+ ->and($duplicate->uuid)->not->toBe($original->uuid)
+ ->and($duplicate->name)->toBe('Original Plugin (Copy)')
+ ->and($duplicate->user_id)->toBe($original->user_id)
+ ->and($duplicate->data_stale_minutes)->toBe($original->data_stale_minutes)
+ ->and($duplicate->data_strategy)->toBe($original->data_strategy)
+ ->and($duplicate->polling_url)->toBe($original->polling_url)
+ ->and($duplicate->polling_verb)->toBe($original->polling_verb)
+ ->and($duplicate->polling_header)->toBe($original->polling_header)
+ ->and($duplicate->polling_body)->toBe($original->polling_body)
+ ->and($duplicate->render_markup)->toBe($original->render_markup)
+ ->and($duplicate->markup_language)->toBe($original->markup_language)
+ ->and($duplicate->configuration)->toBe($original->configuration)
+ ->and($duplicate->configuration_template)->toBe($original->configuration_template)
+ ->and($duplicate->no_bleed)->toBe($original->no_bleed)
+ ->and($duplicate->dark_mode)->toBe($original->dark_mode)
+ ->and($duplicate->data_payload)->toBe($original->data_payload)
+ ->and($duplicate->render_markup_view)->toBeNull();
+});
+
+test('plugin duplicate copies render_markup_view file content to render_markup', function (): void {
+ $user = User::factory()->create();
+
+ // Create a test blade file
+ $testViewPath = resource_path('views/recipes/test-duplicate.blade.php');
+ $testContent = 'Test Content
';
+
+ // Ensure directory exists
+ if (! is_dir(dirname($testViewPath))) {
+ mkdir(dirname($testViewPath), 0755, true);
+ }
+
+ file_put_contents($testViewPath, $testContent);
+
+ try {
+ $original = Plugin::factory()->create([
+ 'user_id' => $user->id,
+ 'name' => 'View Plugin',
+ 'render_markup' => null,
+ 'render_markup_view' => 'recipes.test-duplicate',
+ 'markup_language' => null,
+ ]);
+
+ $duplicate = $original->duplicate();
+
+ expect($duplicate->render_markup)->toBe($testContent)
+ ->and($duplicate->markup_language)->toBe('blade')
+ ->and($duplicate->render_markup_view)->toBeNull()
+ ->and($duplicate->name)->toBe('View Plugin (Copy)');
+ } finally {
+ // Clean up test file
+ if (file_exists($testViewPath)) {
+ unlink($testViewPath);
+ }
+ }
+});
+
+test('plugin duplicate handles liquid file extension', function (): void {
+ $user = User::factory()->create();
+
+ // Create a test liquid file
+ $testViewPath = resource_path('views/recipes/test-duplicate-liquid.liquid');
+ $testContent = '{{ data.message }}
';
+
+ // Ensure directory exists
+ if (! is_dir(dirname($testViewPath))) {
+ mkdir(dirname($testViewPath), 0755, true);
+ }
+
+ file_put_contents($testViewPath, $testContent);
+
+ try {
+ $original = Plugin::factory()->create([
+ 'user_id' => $user->id,
+ 'name' => 'Liquid Plugin',
+ 'render_markup' => null,
+ 'render_markup_view' => 'recipes.test-duplicate-liquid',
+ 'markup_language' => null,
+ ]);
+
+ $duplicate = $original->duplicate();
+
+ expect($duplicate->render_markup)->toBe($testContent)
+ ->and($duplicate->markup_language)->toBe('liquid')
+ ->and($duplicate->render_markup_view)->toBeNull();
+ } finally {
+ // Clean up test file
+ if (file_exists($testViewPath)) {
+ unlink($testViewPath);
+ }
+ }
+});
+
+test('plugin duplicate handles missing view file gracefully', function (): void {
+ $user = User::factory()->create();
+
+ $original = Plugin::factory()->create([
+ 'user_id' => $user->id,
+ 'name' => 'Missing View Plugin',
+ 'render_markup' => null,
+ 'render_markup_view' => 'recipes.nonexistent-view',
+ 'markup_language' => null,
+ ]);
+
+ $duplicate = $original->duplicate();
+
+ expect($duplicate->render_markup_view)->toBeNull()
+ ->and($duplicate->name)->toBe('Missing View Plugin (Copy)');
+});
+
+test('plugin duplicate uses provided user_id', function (): void {
+ $user1 = User::factory()->create();
+ $user2 = User::factory()->create();
+
+ $original = Plugin::factory()->create([
+ 'user_id' => $user1->id,
+ 'name' => 'Original Plugin',
+ ]);
+
+ $duplicate = $original->duplicate($user2->id);
+
+ expect($duplicate->user_id)->toBe($user2->id)
+ ->and($duplicate->user_id)->not->toBe($original->user_id);
+});
+
+test('plugin duplicate falls back to original user_id when no user_id provided', function (): void {
+ $user = User::factory()->create();
+
+ $original = Plugin::factory()->create([
+ 'user_id' => $user->id,
+ 'name' => 'Original Plugin',
+ ]);
+
+ $duplicate = $original->duplicate();
+
+ expect($duplicate->user_id)->toBe($original->user_id);
+});