Unit tests and bug fixes (#11)

* Add event and resolver unit tests

Add a suite of unit tests and small test controllers to improve coverage around HTTP events and error handling. New tests include additional-details, endpoint-exists flag, base event helpers, HTTP method enum, HTTP response event helpers, non-existent endpoint request body preservation, response event request data, and runtime-exception-resolver behavior (default and include-message=ALWAYS). Also add ResponseEventDataTestController and RuntimeExceptionResolverTestController. Update ErrorTestControllerTest to include assertions for server/client error response helpers.

* Rename endpointExists to endpointCalled

Replace the endpointExists property with endpointCalled throughout the codebase and update related logic and tests.

- Rename HttpRequestEvent/BaseHttpEvent field and builder usage to endpointCalled.
- Update CachedBodyHttpServletRequest, captor (RequestEndpointCalledCaptor), registry and publishers to set/check endpointCalled.
- Change property name to web-captor.event-details.include-endpoint-called in examples and test resources and update tests accordingly (renamed test packages/files where applicable).
- Update frontend types and UI (label and value) to reflect endpointCalled.
- Fix request wrapper unwrapping in HttpServletUtils by using WebUtils.getNativeRequest to handle decorated requests (e.g. multipart wrappers).
- Ensure HttpResponseEventPublisher publishes request/response wrapper objects and simplify RuntimeExceptionResolver to always include the error reason phrase.
- Add comprehensive EndpointCalledFlagTest to cover various HTTP scenarios (including multipart behavior) and add IDE run configurations for demo/frontend.

* Eagerly serialize multipart files & add previews

Eagerly capture multipart file bytes on the backend and add download/preview support in the frontend.

- Backend: add serializedFiles field and eager serialization in BodyPayload to capture base64 content before multipart temp-file cleanup, plus a safe getter that returns an empty map when absent.
- Frontend: enhance CapturedResult to show human-friendly sizes, provide a Download link for files, enlarge image previews, and add text/JSON preview (with pretty-printing). Also import new icons used for download/preview.
- Demo: add demo-landscape.jpg and update demoData to fetch and send an image file in the multipart demo.
- Tests: add MultipartFileContentTest to verify base64 content is captured and matches original file bytes.

These changes ensure multipart file contents are reliably recorded and usable in the UI (download/preview) and covered by tests.

Author: david-randoll

.run/DemoApplication.run.xml

@@ -0,0 +1,16 @@
+<component name="ProjectRunConfigurationManager">
+  <configuration default="false" name="DemoApplication" type="SpringBootApplicationConfigurationType" factoryName="Spring Boot" nameIsGenerated="true">
+    <option name="FRAME_DEACTIVATION_UPDATE_POLICY" value="UnknownPolicyInFreeMode" />
+    <module name="spring-web-captor-demo" />
+    <option name="SPRING_BOOT_MAIN_CLASS" value="com.davidrandoll.webcaptor.demo.DemoApplication" />
+    <extension name="coverage">
+      <pattern>
+        <option name="PATTERN" value="com.davidrandoll.webcaptor.demo.*" />
+        <option name="ENABLED" value="true" />
+      </pattern>
+    </extension>
+    <method v="2">
+      <option name="Make" enabled="true" />
+    </method>
+  </configuration>
+</component>

.run/Run Example.run.xml

@@ -0,0 +1,7 @@
+<component name="ProjectRunConfigurationManager">
+  <configuration default="false" name="Run Example" type="Multirun" separateTabs="false" reuseTabsWithFailures="false" startOneByOne="true" markFailedProcess="true" hideSuccessProcess="false" delayTime="0.0">
+    <runConfiguration name="DemoApplication" type="Spring Boot" />
+    <runConfiguration name="Run Frontend" type="Shell Script" />
+    <method v="2" />
+  </configuration>
+</component>

.run/Run Frontend.run.xml

@@ -0,0 +1,17 @@
+<component name="ProjectRunConfigurationManager">
+  <configuration default="false" name="Run Frontend" type="ShConfigurationType">
+    <option name="SCRIPT_TEXT" value="npm run dev" />
+    <option name="INDEPENDENT_SCRIPT_PATH" value="true" />
+    <option name="SCRIPT_PATH" value="" />
+    <option name="SCRIPT_OPTIONS" value="" />
+    <option name="INDEPENDENT_SCRIPT_WORKING_DIRECTORY" value="false" />
+    <option name="SCRIPT_WORKING_DIRECTORY" value="$PROJECT_DIR$/examples/frontend" />
+    <option name="INDEPENDENT_INTERPRETER_PATH" value="true" />
+    <option name="INTERPRETER_PATH" value="powershell.exe" />
+    <option name="INTERPRETER_OPTIONS" value="" />
+    <option name="EXECUTE_IN_TERMINAL" value="true" />
+    <option name="EXECUTE_SCRIPT_FILE" value="false" />
+    <envs />
+    <method v="2" />
+  </configuration>
+</component>

examples/backend/src/main/resources/application.yml

@@ -13,7 +13,7 @@ spring:
 web-captor:
   enabled: true
   event-details:
-    include-endpoint-exists: true
+    include-endpoint-called: true
     include-full-url: true
     include-path: true
     include-method: true

examples/frontend/public/demo-landscape.jpg

@@ -7,7 +7,7 @@ import JsonViewer from './JsonViewer';
 import {
   Clock, Globe, Monitor, MapPin, FileText, AlertTriangle,
   Server, ArrowDown, Layers, Search, GitBranch, Upload,
-  RotateCcw, Sparkles, Puzzle, Info,
+  RotateCcw, Sparkles, Puzzle, Info, Download, Eye,
 } from 'lucide-react';
 
 function parseDuration(d: unknown): string {
@@ -224,10 +224,10 @@ export default function CapturedResult({ event, onTryAnother }: Props) {
               </div>
               <div className="bg-slate-900 rounded-xl p-4 border border-slate-800">
                 <div className="flex items-center gap-1 text-[10px] text-slate-500 uppercase tracking-wider mb-2">
-                  <MapPin className="w-3 h-3" /> Endpoint Exists
+                  <MapPin className="w-3 h-3" /> Endpoint Called
                 </div>
-                <span className={`font-mono text-sm font-medium ${event.endpointExists ? 'text-emerald-400' : 'text-red-400'}`}>
-                  {String(event.endpointExists)}
+                <span className={`font-mono text-sm font-medium ${event.endpointCalled ? 'text-emerald-400' : 'text-red-400'}`}>
+                  {String(event.endpointCalled)}
                 </span>
               </div>
             </div>
@@ -354,18 +354,49 @@ export default function CapturedResult({ event, onTryAnother }: Props) {
                   <div key={`${field}-${i}`} className="bg-slate-900 border border-slate-800 rounded-xl p-3 sm:p-4 mb-2">
                     <div className="flex items-center justify-between mb-2">
                       <span className="font-mono text-sm text-sky-400 font-medium">{f.filename}</span>
-                      <span className="text-xs text-slate-500">{(f.size / 1024).toFixed(1)} KB</span>
+                      <div className="flex items-center gap-2">
+                        <span className="text-xs text-slate-500">{f.size < 1024 ? `${f.size} B` : `${(f.size / 1024).toFixed(1)} KB`}</span>
+                        {f.base64Content && (
+                          <a
+                            href={`data:${f.contentType || 'application/octet-stream'};base64,${f.base64Content}`}
+                            download={f.filename}
+                            className="flex items-center gap-1 text-xs text-sky-400 hover:text-sky-300 transition-colors"
+                          >
+                            <Download className="w-3 h-3" /> Download
+                          </a>
+                        )}
+                      </div>
                     </div>
-                    <div className="text-xs text-slate-500">
+                    <div className="text-xs text-slate-500 mb-2">
                       Field: <span className="text-slate-400">{field}</span> | Type: <span className="text-slate-400">{f.contentType}</span>
                     </div>
                     {f.contentType?.startsWith('image/') && f.base64Content && (
                       <img
                         src={`data:${f.contentType};base64,${f.base64Content}`}
                         alt={f.filename}
-                        className="mt-2 max-h-32 rounded border border-slate-700"
+                        className="mt-2 max-h-48 rounded border border-slate-700"
                       />
                     )}
+                    {f.contentType?.startsWith('text/') && f.base64Content && (
+                      <div className="mt-2">
+                        <div className="flex items-center gap-1 text-[10px] text-slate-500 uppercase tracking-wider mb-1">
+                          <Eye className="w-3 h-3" /> Preview
+                        </div>
+                        <pre className="bg-slate-950 border border-slate-800 rounded-lg p-3 text-xs text-slate-300 font-mono overflow-x-auto max-h-40 whitespace-pre-wrap break-all">
+                          {atob(f.base64Content)}
+                        </pre>
+                      </div>
+                    )}
+                    {f.contentType === 'application/json' && f.base64Content && (
+                      <div className="mt-2">
+                        <div className="flex items-center gap-1 text-[10px] text-slate-500 uppercase tracking-wider mb-1">
+                          <Eye className="w-3 h-3" /> Preview
+                        </div>
+                        <pre className="bg-slate-950 border border-slate-800 rounded-lg p-3 text-xs text-slate-300 font-mono overflow-x-auto max-h-40 whitespace-pre-wrap break-all">
+                          {(() => { try { return JSON.stringify(JSON.parse(atob(f.base64Content)), null, 2); } catch { return atob(f.base64Content); } })()}
+                        </pre>
+                      </div>
+                    )}
                   </div>
                 ))
               )}

examples/frontend/src/components/CapturedResult.tsx

@@ -104,12 +104,14 @@ export const DEMOS: DemoScenario[] = [
     method: 'POST',
     url: '/demo/upload',
     headers: {},
-    body: '(multipart: demo.txt + description)',
+    body: '(multipart: image + description)',
     tags: ['POST', 'Multipart', 'Files'],
-    run() {
+    async run() {
+      const res = await fetch('/demo-landscape.jpg');
+      const blob = await res.blob();
       const fd = new FormData();
-      fd.append('file', new Blob(['Hello from Spring Web Captor!'], { type: 'text/plain' }), 'demo.txt');
-      fd.append('description', 'Demo file upload');
+      fd.append('file', new File([blob], 'landscape.jpg', { type: 'image/jpeg' }), 'landscape.jpg');
+      fd.append('description', 'Beautiful landscape photo');
       return sendRequest('POST', '/demo/upload', { body: fd });
     },
   },

examples/frontend/src/components/demoData.ts

@@ -11,7 +11,7 @@ export interface BodyPayload {
 }
 
 export interface HttpRequestEvent {
-  endpointExists: boolean;
+  endpointCalled: boolean;
   fullUrl: string;
   path: string;
   method: string;

examples/frontend/src/types/events.ts

@@ -28,7 +28,7 @@
 @AllArgsConstructor
 @SuperBuilder(toBuilder = true)
 public class BaseHttpEvent {
-    private boolean endpointExists;
+    private boolean endpointCalled;
     private String fullUrl;
     private String path;
     private HttpMethodEnum method;

spring-web-captor/src/main/java/com/davidrandoll/spring_web_captor/event/BaseHttpEvent.java

@@ -21,17 +21,27 @@ public class BodyPayload {
     @JsonIgnore
     private MultiValueMap<String, MultipartFile> files;
 
+    private Map<String, List<SerializedFile>> serializedFiles;
+
     public BodyPayload(JsonNode body) {
         this.body = body;
     }
 
     public BodyPayload(JsonNode body, MultiValueMap<String, MultipartFile> files) {
         this.body = body;
         this.files = files;
+        // Eagerly serialize files so that base64 content is captured before
+        // Spring cleans up multipart temp files
+        this.serializedFiles = serializeFiles(files);
     }
 
     @JsonProperty("files")
     public Map<String, List<SerializedFile>> getSerializedFiles() {
+        if (serializedFiles != null) return serializedFiles;
+        return Collections.emptyMap();
+    }
+
+    private static Map<String, List<SerializedFile>> serializeFiles(MultiValueMap<String, MultipartFile> files) {
         if (files == null || files.isEmpty()) return Collections.emptyMap();
         Map<String, List<SerializedFile>> result = new LinkedHashMap<>();
         files.forEach((key, multipartFiles) ->