test(gui): fix FindingDetail test assertions for dynamic dialog title

Fix frontend test failures in FindingDetail.test.tsx by updating text
assertions to match the actual dynamic dialog title format. The component
renders "Finding Details: {service} on {host}:{port}" but tests were
expecting just "Finding Details".

Changes:
- Replace exact text match with regex pattern (/Finding Details:/) in two
  test cases to accommodate the dynamic title format
- Ensures tests pass in CI/CD workflows while maintaining coverage

Fixes 2 test failures in GUI CI workflow (121 tests now pass)

Technical Details:
- Tests affected: "renders finding details correctly" and "calls onClose
  when dialog is dismissed"
- Root cause: Component uses more specific title than original test
  expectations
- Solution: Use flexible regex matching instead of exact string matching

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: doublegate

crates/spectre-gui/frontend/src/components/reports/__tests__/FindingDetail.test.tsx

@@ -32,7 +32,7 @@ describe('FindingDetail', () => {
       />
     );
 
-    expect(screen.getByText('Finding Details')).toBeInTheDocument();
+    expect(screen.getByText(/Finding Details:/)).toBeInTheDocument();
     expect(screen.getByText('192.168.1.1')).toBeInTheDocument();
     expect(screen.getByText('OpenSSH 7.4')).toBeInTheDocument();
     expect(screen.getByText('Outdated SSH version with known vulnerabilities')).toBeInTheDocument();
@@ -82,7 +82,7 @@ describe('FindingDetail', () => {
     // Dialog should have a close button (provided by Dialog component)
     // We can't easily test this without mocking the Dialog component
     // But we can verify the dialog is rendered
-    expect(screen.getByText('Finding Details')).toBeInTheDocument();
+    expect(screen.getByText(/Finding Details:/)).toBeInTheDocument();
   });
 
   it('handles export actions', () => {