Er is geen geschikt artikel in de kennisbank gevonden voor deze vraag.
@php($suggestion = $ticket->result_payload['draft_article_suggestion'] ?? null)
- @if(is_array($suggestion))
+ @if (is_array($suggestion))
@endif
- @if($ticket->support_reply && !$ticket->needs_article_draft)
+ @if ($ticket->support_reply && !$ticket->needs_article_draft)
Concept reactie voor klant
- @if(is_array($ticket->result_payload['quick_reply'] ?? null))
+ @if (is_array($ticket->result_payload['quick_reply'] ?? null))
- Snelantwoord gebruikt: #{{ $ticket->result_payload['quick_reply']['id'] ?? '-' }} {{ $ticket->result_payload['quick_reply']['title'] ?? '' }}
+ Snelantwoord gebruikt: #{{ $ticket->result_payload['quick_reply']['id'] ?? '-' }}
+ {{ $ticket->result_payload['quick_reply']['title'] ?? '' }}
@endif
{{ $ticket->support_reply }}
@@ -52,24 +50,25 @@
@endif
Origineel: {{ $ticket->message }}
- @if($ticket->normalized_message)
-
Genormaliseerd: {{ $ticket->normalized_message }}
+ @if ($ticket->normalized_message)
+
Genormaliseerd: {{ $ticket->normalized_message }}
+
@endif
- @if($ticket->redaction_report)
-
{{ json_encode($ticket->redaction_report, JSON_PRETTY_PRINT|JSON_UNESCAPED_UNICODE) }}
+ @if ($ticket->redaction_report)
+
{{ json_encode($ticket->redaction_report, JSON_PRETTY_PRINT | JSON_UNESCAPED_UNICODE) }}
@endif
- @if(is_array($ticket->api_credentials) && !empty($ticket->api_credentials['apiuser']))
+ @if (is_array($ticket->api_credentials) && !empty($ticket->api_credentials['apiuser']))
API credentials aanwezig voor deze ticket-run. Waarden worden niet getoond.
@endif
- @if($ticket->error_message)
+ @if ($ticket->error_message)
{{ $ticket->error_message }}
@endif
- @if($ticket->bestArticle)
+ @if ($ticket->bestArticle)
Beste artikel: #{{ $ticket->bestArticle->id }} - {{ $ticket->bestArticle->title }}
- @if($ticket->confidence !== null)
+ @if ($ticket->confidence !== null)
(confidence {{ number_format($ticket->confidence, 2) }})
@endif
@@ -78,34 +77,35 @@
Toolcalls
- @if($ticket->toolCalls->isEmpty())
+ @if ($ticket->toolCalls->isEmpty())
Geen toolcalls uitgevoerd of voorgesteld.
@else
- @foreach($ticket->toolCalls as $toolCall)
+ @foreach ($ticket->toolCalls as $toolCall)
{{ $toolCall->action }} ({{ $toolCall->status }})
- @if($toolCall->article)
+ @if ($toolCall->article)
via artikel #{{ $toolCall->article->id }}
@endif
-
{{ $toolCall->executed_at ?? $toolCall->created_at }}
+
{{ $toolCall->executed_at ?? $toolCall->created_at }}
- @if($toolCall->parameters)
+ @if ($toolCall->parameters)
Parameters
-
{{ json_encode($toolCall->parameters, JSON_PRETTY_PRINT|JSON_UNESCAPED_UNICODE) }}
+
{{ json_encode($toolCall->parameters, JSON_PRETTY_PRINT | JSON_UNESCAPED_UNICODE) }}
@endif
- @if($toolCall->response)
+ @if ($toolCall->response)
Response
-
{{ json_encode($toolCall->response, JSON_PRETTY_PRINT|JSON_UNESCAPED_UNICODE|JSON_UNESCAPED_SLASHES) }}
+
{{ json_encode($toolCall->response, JSON_PRETTY_PRINT | JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES) }}
@endif
- @if($toolCall->error)
+ @if ($toolCall->error)
{{ $toolCall->error }}
@endif
@@ -118,8 +118,9 @@
Verwerkingsstappen
@php($orderedLogs = $ticket->logs->sortBy([['created_at', 'desc'], ['id', 'desc']]))
@php($latestLog = $orderedLogs->first())
- @if($ticket->status === 'processing' && $latestLog)
-
+ @if ($ticket->status === 'processing' && $latestLog)
+
Huidige stap: {{ $latestLog->step }}
@endif
- @foreach($orderedLogs as $log)
+ @foreach ($orderedLogs as $log)
{{ $log->step }} ({{ $log->status }})
{{ $log->created_at }}
- @if($log->message)
+ @if ($log->message)
{{ $log->message }}
@endif
- @if($log->context)
-
{{ json_encode($log->context, JSON_PRETTY_PRINT|JSON_UNESCAPED_UNICODE) }}
+ @if ($log->context)
+
{{ json_encode($log->context, JSON_PRETTY_PRINT | JSON_UNESCAPED_UNICODE) }}
@endif
@endforeach
diff --git a/resources/views/welcome.blade.php b/resources/views/welcome.blade.php
index 2c2e7c2..cf28968 100644
--- a/resources/views/welcome.blade.php
+++ b/resources/views/welcome.blade.php
@@ -1,225 +1,2406 @@
-
-
-
-
{{ config('app.name', 'Laravel') }}
+
+
+
-
-
-
+
{{ config('app.name', 'Laravel') }}
-
- @if (file_exists(public_path('build/manifest.json')) || file_exists(public_path('hot')))
- @vite(['resources/css/app.css', 'resources/js/app.js'])
- @else
-
- @endif
-
-
-
- @if (Route::has('login'))
-
- @auth
-
- Dashboard
-
- @else
-
- Log in
-
+
+
- Register
-
- @endif
- @endauth
-
- @endif
-
-
-
-
-
Let's get started
-
With so many options available to you,
-
-
+
+ @if (file_exists(public_path('build/manifest.json')) || file_exists(public_path('hot')))
+ @vite(['resources/css/app.css', 'resources/js/app.js'])
+ @else
+
+ @endif
+
+
+
+
@if (Route::has('login'))
-
+
+ @auth
+
+ Dashboard
+
+ @else
+
+ Log in
+
+
+ @if (Route::has('register'))
+
+ Register
+
+ @endif
+ @endauth
+
@endif
-
+
+
+
+
+
Let's get started
+
With so many options available to you,
+
+
+
+
+ v{{ app()->version() }}
+
+ View changelog
+
+
+
+
+
+ {{-- Laravel Logo --}}
+
+
+
+ {{-- 13 --}}
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+ @if (Route::has('login'))
+
+ @endif
+
+
diff --git a/tests/Fakes/FakeArticleRepository.php b/tests/Fakes/FakeArticleRepository.php
new file mode 100644
index 0000000..75cd53a
--- /dev/null
+++ b/tests/Fakes/FakeArticleRepository.php
@@ -0,0 +1,17 @@
+ */
+ public array $candidates = [];
+
+ public function findSimilarByEmbedding(array $embedding, int $limit = 5, array $embeddingContext = []): array
+ {
+ return array_slice($this->candidates, 0, $limit);
+ }
+}
diff --git a/tests/Fakes/FakeDomainInfoTool.php b/tests/Fakes/FakeDomainInfoTool.php
new file mode 100644
index 0000000..339b995
--- /dev/null
+++ b/tests/Fakes/FakeDomainInfoTool.php
@@ -0,0 +1,29 @@
+ */
+ public array $calls = [];
+
+ public array $response = ['ok' => true, 'data' => ['domain' => 'example.nl']];
+
+ public function __construct()
+ {
+ parent::__construct(new OxxaClient);
+ }
+
+ public function execute(array $parameters, array $credentials): array
+ {
+ $this->calls[] = [
+ 'parameters' => $parameters,
+ 'credentials' => $credentials,
+ ];
+
+ return $this->response;
+ }
+}
diff --git a/tests/Fakes/FakeLlmClient.php b/tests/Fakes/FakeLlmClient.php
new file mode 100644
index 0000000..6ea2ca5
--- /dev/null
+++ b/tests/Fakes/FakeLlmClient.php
@@ -0,0 +1,32 @@
+> */
+ public array $embeddings = [];
+
+ /** @var array
*/
+ public array $responses = [];
+
+ /** @var array */
+ public array $generatedPrompts = [];
+
+ public function embed(string $text): array
+ {
+ return $this->embeddings[$text] ?? [0.1, 0.2, 0.3];
+ }
+
+ public function generate(string $prompt, array $options = []): string
+ {
+ $this->generatedPrompts[] = [
+ 'prompt' => $prompt,
+ 'options' => $options,
+ ];
+
+ return array_shift($this->responses) ?? '{}';
+ }
+}
diff --git a/tests/Fakes/FakeTicketProcessingLoggerService.php b/tests/Fakes/FakeTicketProcessingLoggerService.php
new file mode 100644
index 0000000..b5148d1
--- /dev/null
+++ b/tests/Fakes/FakeTicketProcessingLoggerService.php
@@ -0,0 +1,22 @@
+ */
+ public array $logs = [];
+
+ public function log(Ticket $ticket, string $step, string $status = 'info', ?string $message = null, array $context = []): void
+ {
+ $this->logs[] = [
+ 'step' => $step,
+ 'status' => $status,
+ 'message' => $message,
+ 'context' => $context,
+ ];
+ }
+}
diff --git a/tests/Feature/ArticleApiTest.php b/tests/Feature/ArticleApiTest.php
new file mode 100644
index 0000000..14feb19
--- /dev/null
+++ b/tests/Feature/ArticleApiTest.php
@@ -0,0 +1,47 @@
+postJson('/api/articles', [
+ 'title' => 'DNS instellen',
+ 'content' => 'Stap 1...',
+ 'note' => 'Alleen voor DNS vragen',
+ 'allowed_actions' => ['domain_inf'],
+ ]);
+
+ $response->assertStatus(201);
+
+ $article = Article::query()->first();
+ $this->assertNotNull($article);
+ $this->assertSame('Alleen voor DNS vragen', $article->note);
+ $this->assertSame(['domain_inf'], $article->allowed_actions);
+ }
+
+ public function test_it_rejects_invalid_allowed_action(): void
+ {
+ $response = $this->postJson('/api/articles', [
+ 'title' => 'DNS instellen',
+ 'content' => 'Stap 1...',
+ 'allowed_actions' => ['invalid_action'],
+ ]);
+
+ $response->assertStatus(422);
+ }
+
+ public function test_it_lists_articles(): void
+ {
+ Article::query()->create(['title' => 'A', 'content' => 'B']);
+ $response = $this->getJson('/api/articles');
+ $response->assertOk();
+ }
+}
diff --git a/tests/Feature/ProcessTicketJobFlowTest.php b/tests/Feature/ProcessTicketJobFlowTest.php
new file mode 100644
index 0000000..2719ba1
--- /dev/null
+++ b/tests/Feature/ProcessTicketJobFlowTest.php
@@ -0,0 +1,216 @@
+create(['title' => 'DNS', 'content' => 'stappen']);
+ $quickReply = QuickReply::query()->create(['title' => 'DNS Quick', 'content' => 'Gebruik DNS quick antwoord', 'is_active' => true]);
+ $article->quickReplies()->sync([$quickReply->id]);
+
+ $ticket = Ticket::query()->create(['message' => 'DNS vraag', 'status' => 'queued']);
+
+ $embedding = $this->createMock(EmbeddingService::class);
+ $embedding->method('embed')->willReturn(array_fill(0, 768, 0.1));
+ $embedding->method('context')->willReturn(['provider_instance_id' => 'p1', 'embedding_model' => 'm1']);
+
+ $semantic = $this->createMock(SemanticSearchService::class);
+ $semantic->method('findBestArticle')->willReturn([
+ 'best_article' => $article,
+ 'confidence' => 0.9,
+ 'explanation' => 'match',
+ 'top_3_candidates' => [],
+ 'top_5_candidates' => [],
+ 'retrieval_meta' => [],
+ 'requested_tool_call' => ['action' => 'domain_inf', 'parameters' => ['sld' => 'example', 'tld' => 'nl']],
+ 'classifier_raw_response' => ['mode' => 'llm'],
+ ]);
+
+ $normalizer = $this->createMock(TicketNormalizationService::class);
+ $normalizer->method('normalize')->willReturn([
+ 'normalized_message' => 'dns vraag',
+ 'redaction_report' => ['language' => 'nl'],
+ ]);
+
+ $logger = app(TicketProcessingLoggerService::class);
+
+ $knowledgeGap = $this->createMock(KnowledgeGapService::class);
+ $knowledgeGap->method('shouldCreateDraft')->willReturn(false);
+
+ $toolCallService = $this->createMock(TicketToolCallService::class);
+ $toolCallService->expects($this->never())->method('executeRequestedTool');
+
+ $quickReplyResolver = $this->createMock(QuickReplyResolver::class);
+ $quickReplyResolver->method('resolveForArticle')->willReturn($quickReply);
+
+ $supportReply = $this->createMock(SupportReplyService::class);
+ $supportReply->expects($this->never())->method('build');
+
+ $job = new ProcessTicketJob($ticket->id);
+ $job->handle(
+ $embedding,
+ $semantic,
+ $normalizer,
+ $logger,
+ $knowledgeGap,
+ $toolCallService,
+ $quickReplyResolver,
+ $supportReply,
+ new TicketResultPayloadBuilder
+ );
+
+ $ticket->refresh();
+ $this->assertSame('completed', $ticket->status);
+ $this->assertSame('Gebruik DNS quick antwoord', $ticket->support_reply);
+ $this->assertSame($quickReply->id, $ticket->result_payload['quick_reply']['id']);
+ }
+
+ public function test_it_executes_tool_call_when_no_quick_reply_exists(): void
+ {
+ $article = Article::query()->create(['title' => 'DNS', 'content' => 'stappen', 'allowed_actions' => ['domain_inf']]);
+ $ticket = Ticket::query()->create([
+ 'message' => 'DNS vraag',
+ 'status' => 'queued',
+ 'api_credentials' => ['apiuser' => 'u', 'apipassword' => 'p'],
+ ]);
+
+ $embedding = $this->createMock(EmbeddingService::class);
+ $embedding->method('embed')->willReturn(array_fill(0, 768, 0.1));
+ $embedding->method('context')->willReturn(['provider_instance_id' => 'p1', 'embedding_model' => 'm1']);
+
+ $semantic = $this->createMock(SemanticSearchService::class);
+ $semantic->method('findBestArticle')->willReturn([
+ 'best_article' => $article,
+ 'confidence' => 0.9,
+ 'explanation' => 'match',
+ 'top_3_candidates' => [],
+ 'top_5_candidates' => [],
+ 'retrieval_meta' => [],
+ 'requested_tool_call' => ['action' => 'domain_inf', 'parameters' => ['sld' => 'example', 'tld' => 'nl']],
+ 'classifier_raw_response' => ['mode' => 'llm'],
+ ]);
+
+ $normalizer = $this->createMock(TicketNormalizationService::class);
+ $normalizer->method('normalize')->willReturn([
+ 'normalized_message' => 'dns vraag',
+ 'redaction_report' => ['language' => 'nl'],
+ ]);
+
+ $knowledgeGap = $this->createMock(KnowledgeGapService::class);
+ $knowledgeGap->method('shouldCreateDraft')->willReturn(false);
+
+ $quickReplyResolver = $this->createMock(QuickReplyResolver::class);
+ $quickReplyResolver->method('resolveForArticle')->willReturn(null);
+
+ $toolRecord = new \App\Models\TicketToolCall([
+ 'action' => 'domain_inf',
+ 'status' => 'success',
+ 'parameters' => ['sld' => 'example', 'tld' => 'nl'],
+ ]);
+ $toolCallService = $this->createMock(TicketToolCallService::class);
+ $toolCallService->method('executeRequestedTool')->willReturn($toolRecord);
+
+ $supportReply = $this->createMock(SupportReplyService::class);
+ $supportReply->method('build')->willReturn('1. Doe stap 1');
+
+ $job = new ProcessTicketJob($ticket->id);
+ $job->handle(
+ $embedding,
+ $semantic,
+ $normalizer,
+ app(TicketProcessingLoggerService::class),
+ $knowledgeGap,
+ $toolCallService,
+ $quickReplyResolver,
+ $supportReply,
+ new TicketResultPayloadBuilder
+ );
+
+ $ticket->refresh();
+ $this->assertSame('completed', $ticket->status);
+ $this->assertSame('1. Doe stap 1', $ticket->support_reply);
+ $this->assertSame('domain_inf', $ticket->result_payload['requested_tool_call']['action']);
+ }
+
+ public function test_it_marks_knowledge_gap_and_skips_customer_reply(): void
+ {
+ $article = Article::query()->create(['title' => 'DNS', 'content' => 'stappen']);
+ $ticket = Ticket::query()->create(['message' => 'DNS vraag', 'status' => 'queued']);
+
+ $embedding = $this->createMock(EmbeddingService::class);
+ $embedding->method('embed')->willReturn(array_fill(0, 768, 0.1));
+ $embedding->method('context')->willReturn(['provider_instance_id' => 'p1', 'embedding_model' => 'm1']);
+
+ $semantic = $this->createMock(SemanticSearchService::class);
+ $semantic->method('findBestArticle')->willReturn([
+ 'best_article' => $article,
+ 'confidence' => 0.2,
+ 'explanation' => 'low confidence',
+ 'top_3_candidates' => [],
+ 'top_5_candidates' => [],
+ 'retrieval_meta' => [],
+ 'requested_tool_call' => null,
+ 'classifier_raw_response' => ['mode' => 'llm'],
+ ]);
+
+ $normalizer = $this->createMock(TicketNormalizationService::class);
+ $normalizer->method('normalize')->willReturn([
+ 'normalized_message' => 'dns vraag',
+ 'redaction_report' => ['language' => 'nl'],
+ ]);
+
+ $knowledgeGap = $this->createMock(KnowledgeGapService::class);
+ $knowledgeGap->method('shouldCreateDraft')->willReturn(true);
+ $knowledgeGap->method('suggestArticleDraft')->willReturn([
+ 'title' => 'Nieuwe DNS handleiding',
+ 'content' => 'Nog aan te vullen',
+ ]);
+
+ $toolCallService = $this->createMock(TicketToolCallService::class);
+ $toolCallService->expects($this->never())->method('executeRequestedTool');
+
+ $quickReplyResolver = $this->createMock(QuickReplyResolver::class);
+ $quickReplyResolver->expects($this->never())->method('resolveForArticle');
+
+ $supportReply = $this->createMock(SupportReplyService::class);
+ $supportReply->expects($this->never())->method('build');
+
+ $job = new ProcessTicketJob($ticket->id);
+ $job->handle(
+ $embedding,
+ $semantic,
+ $normalizer,
+ app(TicketProcessingLoggerService::class),
+ $knowledgeGap,
+ $toolCallService,
+ $quickReplyResolver,
+ $supportReply,
+ new TicketResultPayloadBuilder
+ );
+
+ $ticket->refresh();
+ $this->assertTrue($ticket->needs_article_draft);
+ $this->assertNull($ticket->support_reply);
+ $this->assertSame('Nieuwe DNS handleiding', $ticket->result_payload['draft_article_suggestion']['title']);
+ }
+}
diff --git a/tests/Feature/QuickReplyAdminTest.php b/tests/Feature/QuickReplyAdminTest.php
new file mode 100644
index 0000000..9ddcc52
--- /dev/null
+++ b/tests/Feature/QuickReplyAdminTest.php
@@ -0,0 +1,36 @@
+set('title', 'DNS Basis')
+ ->set('content', 'Gebruik deze stappen')
+ ->set('isActive', true)
+ ->call('save');
+
+ $reply = QuickReply::query()->first();
+ $this->assertNotNull($reply);
+
+ Livewire::test(QuickReplyManager::class)
+ ->set("editRows.{$reply->id}.title", 'DNS Geupdated')
+ ->set("editRows.{$reply->id}.content", 'Nieuwe content')
+ ->set("editRows.{$reply->id}.is_active", false)
+ ->call('updateQuickReply', $reply->id);
+
+ $reply->refresh();
+ $this->assertSame('DNS Geupdated', $reply->title);
+ $this->assertFalse($reply->is_active);
+ }
+}
diff --git a/tests/Feature/TicketAndArticleModelTest.php b/tests/Feature/TicketAndArticleModelTest.php
new file mode 100644
index 0000000..30ef3eb
--- /dev/null
+++ b/tests/Feature/TicketAndArticleModelTest.php
@@ -0,0 +1,62 @@
+create(['title' => 'DNS', 'content' => 'x']);
+ $reply = QuickReply::query()->create(['title' => 'Quick', 'content' => 'y', 'is_active' => true]);
+
+ $article->quickReplies()->sync([$reply->id]);
+ $this->assertCount(1, $article->quickReplies);
+
+ $article->delete();
+
+ $pivotCount = DB::table('article_quick_reply')->count();
+ $this->assertSame(0, $pivotCount);
+ }
+
+ public function test_ticket_credentials_are_stored_encrypted_and_decrypted_via_cast(): void
+ {
+ $ticket = Ticket::query()->create([
+ 'message' => 'vraag',
+ 'api_credentials' => ['apiuser' => 'demo', 'apipassword' => 'secret'],
+ ]);
+
+ $this->assertSame('demo', $ticket->api_credentials['apiuser']);
+ $raw = DB::table('tickets')->where('id', $ticket->id)->value('api_credentials');
+ $this->assertIsString($raw);
+ $this->assertStringNotContainsString('secret', $raw);
+ }
+
+ public function test_ticket_tool_call_casts_arrays(): void
+ {
+ $ticket = Ticket::query()->create(['message' => 'vraag']);
+ $article = Article::query()->create(['title' => 'DNS', 'content' => 'x']);
+
+ $toolCall = TicketToolCall::query()->create([
+ 'ticket_id' => $ticket->id,
+ 'article_id' => $article->id,
+ 'action' => 'domain_inf',
+ 'status' => 'success',
+ 'parameters' => ['sld' => 'example', 'tld' => 'nl'],
+ 'response' => ['ok' => true],
+ ]);
+
+ $toolCall->refresh();
+ $this->assertSame('example', $toolCall->parameters['sld']);
+ $this->assertTrue($toolCall->response['ok']);
+ }
+}
diff --git a/tests/Feature/TicketIngestionTest.php b/tests/Feature/TicketIngestionTest.php
new file mode 100644
index 0000000..4adf0fc
--- /dev/null
+++ b/tests/Feature/TicketIngestionTest.php
@@ -0,0 +1,48 @@
+postJson('/api/tickets', [
+ 'message' => 'Mijn mail werkt niet',
+ 'api_credentials' => [
+ 'apiuser' => 'demo',
+ 'apipassword' => 'secret',
+ ],
+ ]);
+
+ $response->assertStatus(202)
+ ->assertJsonPath('status', 'queued');
+
+ $ticket = Ticket::query()->latest('id')->first();
+ $this->assertNotNull($ticket);
+ $this->assertSame('queued', $ticket->status);
+ $this->assertSame('demo', $ticket->api_credentials['apiuser']);
+
+ $raw = DB::table('tickets')->where('id', $ticket->id)->value('api_credentials');
+ $this->assertIsString($raw);
+ $this->assertStringNotContainsString('secret', $raw);
+
+ Queue::assertPushed(ProcessTicketJob::class);
+ }
+
+ public function test_it_validates_required_message(): void
+ {
+ $response = $this->postJson('/api/tickets', []);
+ $response->assertStatus(422);
+ }
+}
diff --git a/tests/Feature/TicketShowPageTest.php b/tests/Feature/TicketShowPageTest.php
new file mode 100644
index 0000000..78ef39b
--- /dev/null
+++ b/tests/Feature/TicketShowPageTest.php
@@ -0,0 +1,43 @@
+create(['title' => 'DNS', 'content' => 'x']);
+ $ticket = Ticket::query()->create([
+ 'message' => 'vraag',
+ 'status' => 'completed',
+ 'best_article_id' => $article->id,
+ 'support_reply' => 'Gebruik deze stappen',
+ 'result_payload' => [
+ 'quick_reply' => ['id' => 1, 'title' => 'DNS Quick'],
+ 'draft_article_suggestion' => null,
+ ],
+ ]);
+
+ TicketToolCall::query()->create([
+ 'ticket_id' => $ticket->id,
+ 'article_id' => $article->id,
+ 'action' => 'domain_inf',
+ 'status' => 'success',
+ 'parameters' => ['sld' => 'example', 'tld' => 'nl'],
+ 'response' => ['ok' => true],
+ ]);
+
+ $response = $this->get("/admin/tickets/{$ticket->id}");
+ $response->assertOk();
+ $response->assertSee('Snelantwoord gebruikt', false);
+ $response->assertSee('Toolcalls', false);
+ }
+}
diff --git a/tests/TestCase.php b/tests/TestCase.php
index fe1ffc2..539c7dd 100644
--- a/tests/TestCase.php
+++ b/tests/TestCase.php
@@ -6,5 +6,4 @@ use Illuminate\Foundation\Testing\TestCase as BaseTestCase;
abstract class TestCase extends BaseTestCase
{
- //
}
diff --git a/tests/Unit/AIClassifierServiceTest.php b/tests/Unit/AIClassifierServiceTest.php
index 5a427cc..42cd858 100644
--- a/tests/Unit/AIClassifierServiceTest.php
+++ b/tests/Unit/AIClassifierServiceTest.php
@@ -41,22 +41,9 @@ class AIClassifierServiceTest extends TestCase
}
};
- $settings = new class extends AppSettingsService
- {
- public function getPrompt(string $key, ?string $default = null): ?string
- {
- return 'Select best article.';
- }
-
- public function get(string $key, ?string $default = null): ?string
- {
- return $default;
- }
- };
-
$service = new AIClassifierService(
$client,
- $settings,
+ $this->fakeSettings(),
new ClassifierPromptBuilder,
new LlmJsonDecoder,
new ToolCallRequestValidator
@@ -81,4 +68,122 @@ class AIClassifierServiceTest extends TestCase
$this->assertStringContainsString('Allowed actions: ["domain_inf"]', $client->prompt);
$this->assertStringContainsString('Internal note for support assistant', $client->prompt);
}
+
+ public function test_it_falls_back_when_ranking_disabled(): void
+ {
+ config()->set('services.llm.ranking_enabled', false);
+
+ $client = new class implements LlmClientInterface
+ {
+ public function embed(string $text): array
+ {
+ return [];
+ }
+
+ public function generate(string $prompt, array $options = []): string
+ {
+ return '{}';
+ }
+ };
+
+ $service = new AIClassifierService(
+ $client,
+ $this->fakeSettings(),
+ new ClassifierPromptBuilder,
+ new LlmJsonDecoder,
+ new ToolCallRequestValidator
+ );
+
+ $result = $service->rank('vraag', [$this->candidate(15)]);
+ $this->assertSame(15, $result->articleId);
+ $this->assertSame(0.2, $result->confidence);
+ }
+
+ public function test_it_falls_back_when_llm_json_is_invalid(): void
+ {
+ config()->set('services.llm.ranking_enabled', true);
+
+ $client = new class implements LlmClientInterface
+ {
+ public function embed(string $text): array
+ {
+ return [];
+ }
+
+ public function generate(string $prompt, array $options = []): string
+ {
+ return 'not-json';
+ }
+ };
+
+ $service = new AIClassifierService(
+ $client,
+ $this->fakeSettings(),
+ new ClassifierPromptBuilder,
+ new LlmJsonDecoder,
+ new ToolCallRequestValidator
+ );
+
+ $result = $service->rank('vraag', [$this->candidate(21)]);
+ $this->assertSame(21, $result->articleId);
+ $this->assertStringContainsString('invalid JSON', $result->explanation);
+ }
+
+ public function test_it_falls_back_when_article_id_not_in_candidates(): void
+ {
+ $client = new class implements LlmClientInterface
+ {
+ public function embed(string $text): array
+ {
+ return [];
+ }
+
+ public function generate(string $prompt, array $options = []): string
+ {
+ return json_encode([
+ 'article_id' => 999,
+ 'confidence' => 0.9,
+ 'explanation' => 'x',
+ ]);
+ }
+ };
+
+ $service = new AIClassifierService(
+ $client,
+ $this->fakeSettings(),
+ new ClassifierPromptBuilder,
+ new LlmJsonDecoder,
+ new ToolCallRequestValidator
+ );
+
+ $result = $service->rank('vraag', [$this->candidate(33)]);
+ $this->assertSame(33, $result->articleId);
+ $this->assertStringContainsString('schema invalid', $result->explanation);
+ }
+
+ private function fakeSettings(): AppSettingsService
+ {
+ return new class extends AppSettingsService
+ {
+ public function getPrompt(string $key, ?string $default = null): ?string
+ {
+ return 'Select best article.';
+ }
+
+ public function get(string $key, ?string $default = null): ?string
+ {
+ return $default;
+ }
+ };
+ }
+
+ private function candidate(int $id): ArticleCandidateDTO
+ {
+ return new ArticleCandidateDTO(
+ articleId: $id,
+ title: 'A',
+ content: 'B',
+ distance: 0.2
+ );
+ }
}
diff --git a/tests/Unit/ClassifierPromptBuilderTest.php b/tests/Unit/ClassifierPromptBuilderTest.php
new file mode 100644
index 0000000..c25089d
--- /dev/null
+++ b/tests/Unit/ClassifierPromptBuilderTest.php
@@ -0,0 +1,37 @@
+build(
+ 'Base prompt',
+ 'Hoe stel ik DNS in?',
+ [
+ new ArticleCandidateDTO(
+ articleId: 10,
+ title: 'DNS',
+ content: 'Stappen voor DNS.',
+ distance: 0.12,
+ sourceUrl: 'https://example.test/article',
+ note: 'Alleen gebruiken voor DNS.',
+ allowedActions: ['domain_inf']
+ ),
+ ],
+ 'nl'
+ );
+
+ $this->assertStringContainsString('Base prompt', $prompt);
+ $this->assertStringContainsString('User language: nl', $prompt);
+ $this->assertStringContainsString('Allowed actions: ["domain_inf"]', $prompt);
+ $this->assertStringContainsString('Internal note for support assistant', $prompt);
+ $this->assertStringContainsString('"tool_call"', $prompt);
+ }
+}
diff --git a/tests/Unit/EmbeddingServiceTest.php b/tests/Unit/EmbeddingServiceTest.php
new file mode 100644
index 0000000..079d334
--- /dev/null
+++ b/tests/Unit/EmbeddingServiceTest.php
@@ -0,0 +1,70 @@
+fakeSettings();
+ $llm = new FakeLlmClient;
+ $service = new EmbeddingService($llm, $settings);
+
+ EmbeddingCache::query()->create([
+ 'provider_instance_id' => 'instance-1',
+ 'embedding_model' => 'embed-model',
+ 'text_hash' => hash('sha256', 'abc'),
+ 'text' => 'abc',
+ 'embedding' => [0.9, 0.8],
+ ]);
+
+ $embedding = $service->embed('abc');
+ $this->assertSame([0.9, 0.8], $embedding);
+ $this->assertCount(0, $llm->generatedPrompts);
+ }
+
+ public function test_it_throws_when_embedding_is_empty(): void
+ {
+ $settings = $this->fakeSettings();
+ $llm = new FakeLlmClient;
+ $llm->embeddings['abc'] = [];
+ $service = new EmbeddingService($llm, $settings);
+
+ $this->expectException(OllamaUnavailableException::class);
+ $service->embed('abc');
+ }
+
+ private function fakeSettings(): AppSettingsService
+ {
+ return new class extends AppSettingsService {
+ public function activeProviderInstance(): array
+ {
+ return ['id' => 'instance-1', 'embedding_model' => 'embed-model'];
+ }
+
+ public function activeProviderInstanceId(): string
+ {
+ return 'instance-1';
+ }
+
+ public function get(string $key, ?string $default = null): ?string
+ {
+ if ($key === 'llm.models.embedding') {
+ return 'embed-model';
+ }
+
+ return $default;
+ }
+ };
+ }
+}
diff --git a/tests/Unit/LlmJsonDecoderTest.php b/tests/Unit/LlmJsonDecoderTest.php
new file mode 100644
index 0000000..d6a8a4b
--- /dev/null
+++ b/tests/Unit/LlmJsonDecoderTest.php
@@ -0,0 +1,40 @@
+decode('{"a":1}');
+
+ $this->assertSame(['a' => 1], $decoded);
+ }
+
+ public function test_it_decodes_fenced_json(): void
+ {
+ $decoder = new LlmJsonDecoder;
+ $decoded = $decoder->decode("```json\n{\"a\":2}\n```");
+
+ $this->assertSame(['a' => 2], $decoded);
+ }
+
+ public function test_it_extracts_json_from_mixed_text(): void
+ {
+ $decoder = new LlmJsonDecoder;
+ $decoded = $decoder->decode("noise before {\"a\":3} noise after");
+
+ $this->assertSame(['a' => 3], $decoded);
+ }
+
+ public function test_it_returns_null_on_invalid_json(): void
+ {
+ $decoder = new LlmJsonDecoder;
+
+ $this->assertNull($decoder->decode('no json here'));
+ }
+}
diff --git a/tests/Unit/QuickReplyResolverTest.php b/tests/Unit/QuickReplyResolverTest.php
new file mode 100644
index 0000000..040fbd1
--- /dev/null
+++ b/tests/Unit/QuickReplyResolverTest.php
@@ -0,0 +1,40 @@
+assertNull($resolver->resolveForArticle(null));
+ }
+
+ public function test_it_returns_first_active_quick_reply(): void
+ {
+ $article = Article::query()->create([
+ 'title' => 'DNS',
+ 'content' => 'content',
+ ]);
+
+ $inactive = QuickReply::query()->create(['title' => 'B Reply', 'content' => 'B', 'is_active' => false]);
+ $activeB = QuickReply::query()->create(['title' => 'Z Reply', 'content' => 'Z', 'is_active' => true]);
+ $activeA = QuickReply::query()->create(['title' => 'A Reply', 'content' => 'A', 'is_active' => true]);
+
+ $article->quickReplies()->sync([$inactive->id, $activeB->id, $activeA->id]);
+
+ $resolver = new QuickReplyResolver;
+ $resolved = $resolver->resolveForArticle($article);
+
+ $this->assertInstanceOf(QuickReply::class, $resolved);
+ $this->assertSame($activeA->id, $resolved->id);
+ }
+}
diff --git a/tests/Unit/SupportReplyServiceTest.php b/tests/Unit/SupportReplyServiceTest.php
new file mode 100644
index 0000000..02064ed
--- /dev/null
+++ b/tests/Unit/SupportReplyServiceTest.php
@@ -0,0 +1,68 @@
+responses = ['1. Doe X'];
+
+ $service = new SupportReplyService(
+ $this->fakeSettings(),
+ $llm,
+ new FakeTicketProcessingLoggerService
+ );
+
+ $ticket = Ticket::query()->create(['message' => 'vraag', 'normalized_message' => 'vraag']);
+ $article = Article::query()->create(['title' => 'DNS', 'content' => 'steps']);
+
+ $reply = $service->build($ticket, $article, 'relevant');
+
+ $this->assertSame('1. Doe X', $reply);
+ $this->assertStringContainsString('Gebruikersvraag (genormaliseerd): vraag', $llm->generatedPrompts[0]['prompt']);
+ }
+
+ public function test_it_falls_back_when_llm_returns_empty(): void
+ {
+ $llm = new FakeLlmClient;
+ $llm->responses = [''];
+
+ $service = new SupportReplyService(
+ $this->fakeSettings(),
+ $llm,
+ new FakeTicketProcessingLoggerService
+ );
+
+ $ticket = Ticket::query()->create(['message' => 'vraag']);
+ $article = Article::query()->create(['title' => 'DNS', 'content' => 'steps']);
+
+ $reply = $service->build($ticket, $article, 'relevant');
+
+ $this->assertStringContainsString('Gebruik het kennisbankartikel', $reply);
+ }
+
+ private function fakeSettings(): AppSettingsService
+ {
+ return new class extends AppSettingsService {
+ public function getPrompt(string $key, ?string $default = null): ?string
+ {
+ return 'Prompt';
+ }
+ };
+ }
+}
diff --git a/tests/Unit/TicketResultPayloadBuilderTest.php b/tests/Unit/TicketResultPayloadBuilderTest.php
new file mode 100644
index 0000000..9dbf924
--- /dev/null
+++ b/tests/Unit/TicketResultPayloadBuilderTest.php
@@ -0,0 +1,36 @@
+ 7, 'title' => 'Quick']);
+ $quickReply->id = 7;
+ $toolCall = new TicketToolCall(['action' => 'domain_inf', 'status' => 'success']);
+
+ $payload = $builder->build(
+ [
+ 'top_3_candidates' => [['article_id' => 1]],
+ 'top_5_candidates' => [['article_id' => 1], ['article_id' => 2]],
+ 'classifier_raw_response' => ['mode' => 'llm'],
+ 'requested_tool_call' => ['action' => 'domain_inf'],
+ ],
+ $toolCall,
+ $quickReply,
+ false,
+ null
+ );
+
+ $this->assertSame(7, $payload['quick_reply']['id']);
+ $this->assertSame('domain_inf', $payload['requested_tool_call']['action']);
+ $this->assertFalse($payload['knowledge_gap']);
+ }
+}
diff --git a/tests/Unit/TicketToolCallServiceTest.php b/tests/Unit/TicketToolCallServiceTest.php
new file mode 100644
index 0000000..fa8dacc
--- /dev/null
+++ b/tests/Unit/TicketToolCallServiceTest.php
@@ -0,0 +1,53 @@
+create(['title' => 'A', 'content' => 'B', 'allowed_actions' => []]);
+ $ticket = Ticket::query()->create(['message' => 'vraag']);
+
+ $record = $service->executeRequestedTool($ticket, $article, [
+ 'action' => 'domain_inf',
+ 'parameters' => ['sld' => 'example', 'tld' => 'nl'],
+ ]);
+
+ $this->assertNotNull($record);
+ $this->assertSame('skipped', $record->status);
+ }
+
+ public function test_it_executes_domain_info_when_allowed_and_credentials_present(): void
+ {
+ $fakeTool = new FakeDomainInfoTool;
+ $service = new TicketToolCallService($fakeTool, new FakeTicketProcessingLoggerService);
+
+ $article = Article::query()->create(['title' => 'A', 'content' => 'B', 'allowed_actions' => ['domain_inf']]);
+ $ticket = Ticket::query()->create([
+ 'message' => 'vraag',
+ 'api_credentials' => ['apiuser' => 'u', 'apipassword' => 'p'],
+ ]);
+
+ $record = $service->executeRequestedTool($ticket, $article, [
+ 'action' => 'domain_inf',
+ 'parameters' => ['sld' => 'example', 'tld' => 'nl'],
+ ]);
+
+ $this->assertNotNull($record);
+ $this->assertSame('success', $record->status);
+ $this->assertCount(1, $fakeTool->calls);
+ }
+}
diff --git a/tests/Unit/ToolCallRequestValidatorTest.php b/tests/Unit/ToolCallRequestValidatorTest.php
new file mode 100644
index 0000000..a17a166
--- /dev/null
+++ b/tests/Unit/ToolCallRequestValidatorTest.php
@@ -0,0 +1,45 @@
+validate([
+ 'action' => 'domain_inf',
+ 'parameters' => ['sld' => 'Example', 'tld' => 'NL'],
+ 'reason' => 'Needed',
+ ]);
+
+ $this->assertSame([
+ 'action' => 'domain_inf',
+ 'parameters' => ['sld' => 'example', 'tld' => 'nl'],
+ 'reason' => 'Needed',
+ ], $validated);
+ }
+
+ public function test_it_rejects_missing_parameters(): void
+ {
+ $validator = new ToolCallRequestValidator;
+
+ $this->assertNull($validator->validate([
+ 'action' => 'domain_inf',
+ 'parameters' => ['sld' => 'example'],
+ ]));
+ }
+
+ public function test_it_rejects_unknown_action(): void
+ {
+ $validator = new ToolCallRequestValidator;
+
+ $this->assertNull($validator->validate([
+ 'action' => 'dns_update',
+ 'parameters' => ['sld' => 'example', 'tld' => 'nl'],
+ ]));
+ }
+}