test(cli): force class init in terminal smoke tests

A class literal (PosixTerminal.class / WindowsTerminal.class) loads but
does not run <clinit>, so the FFM downcall handles in static fields were
never resolved — the tests asserted nothing and the classes showed 0%
coverage. Switch to Class.forName(name, true, loader) to actually trigger
static initialization, so a missing/renamed libc or kernel32 symbol fails
the smoke test as intended and the init path is covered.

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

Author: dfa1

cli/src/test/java/io/github/dfa1/vortex/cli/tui/term/PosixTerminalSmokeTest.java

@@ -14,18 +14,24 @@
 ///
 /// `open()` and `size()` are deliberately not exercised — they need a real
 /// console, and a stray successful `open()` would switch the developer's terminal
-/// into the alternate screen / raw mode. Class load alone is the safe, meaningful
-/// assertion: it forces every `Linker.downcallHandle` in the static initializer
-/// (tcgetattr, tcsetattr, cfmakeraw, ioctl) to resolve its libc symbol, throwing
+/// into the alternate screen / raw mode. Forcing class *initialization* is the
+/// safe, meaningful assertion: it runs every `Linker.downcallHandle` in the
+/// static initializer (tcgetattr, tcsetattr, cfmakeraw, ioctl), throwing
 /// [UnsatisfiedLinkError] during `<clinit>` if an entry point is missing.
 class PosixTerminalSmokeTest {
 
     @Test
     @EnabledOnOs({OS.LINUX, OS.MAC})
-    void classLoad_resolvesEveryLibcSymbol() {
-        // Given / When — touching the class triggers <clinit>, which calls
-        // Linker.downcallHandle for tcgetattr / tcsetattr / cfmakeraw / ioctl.
-        Class<?> sut = PosixTerminal.class;
+    void classLoad_resolvesEveryLibcSymbol() throws ClassNotFoundException {
+        // Given / When — a class *literal* (PosixTerminal.class) only loads the
+        // class; it does NOT run <clinit>, so the downcall handles in static
+        // fields would never resolve and this would assert nothing. Force
+        // initialization explicitly so the Linker.downcallHandle calls for
+        // tcgetattr / tcsetattr / cfmakeraw / ioctl actually fire.
+        Class<?> sut = Class.forName(
+                "io.github.dfa1.vortex.cli.tui.term.PosixTerminal",
+                true,
+                getClass().getClassLoader());
 
         // Then
         assertThat(sut).isNotNull();

cli/src/test/java/io/github/dfa1/vortex/cli/tui/term/WindowsTerminalSmokeTest.java

@@ -21,13 +21,17 @@ class WindowsTerminalSmokeTest {
 
     @Test
     @EnabledOnOs(OS.WINDOWS)
-    void classLoad_resolvesEveryKernel32Symbol() {
-        // Given / When — touching the class triggers <clinit>, which calls
-        // Linker.downcallHandle for every imported kernel32 function.
+    void classLoad_resolvesEveryKernel32Symbol() throws ClassNotFoundException {
+        // Given / When — a class *literal* (WindowsTerminal.class) only loads the
+        // class; it does NOT run <clinit>. Force initialization so the
+        // Linker.downcallHandle calls for every imported kernel32 function fire.
         // Symbols covered: GetStdHandle, GetConsoleMode, SetConsoleMode,
         // GetConsoleScreenBufferInfo, GetFileType, ReadFile,
         // GetNumberOfConsoleInputEvents.
-        Class<?> sut = WindowsTerminal.class;
+        Class<?> sut = Class.forName(
+                "io.github.dfa1.vortex.cli.tui.term.WindowsTerminal",
+                true,
+                getClass().getClassLoader());
 
         // Then
         assertThat(sut).isNotNull();