assertNotNull(results);
            // Should return empty or minimal results
            assertTrue(results.size() <= 10);
        }
    }

    /**
     * Test search with negative start position (should be handled gracefully).
     */
    @Test
    public void test_searchWithNegativeStartPosition() throws Exception {

            jcifs.internal.smb1.com.SmbComReadAndX req = cap.getValue();
            assertEquals(1024, req.getMinCount(), "Named pipe minCount should be 1024");
            assertEquals(1024, req.getMaxCount(), "Named pipe maxCount should be 1024");
            assertEquals(1024, req.getRemaining(), "Named pipe remaining should be 1024");
        }
    }

    @Nested
    @DisplayName("Lifecycle and exception mapping")

        List<FileNotifyInformation> result = sut.watch();

        assertNull(result, "Cancelled watch should return null");
        verify(tree, times(1)).close();
    }

    // Error code 0x10C should clear the notify list and return it
    @Test
    @DisplayName("watch() clears notify list on error 0x10C and returns it")

        // Act: relative path should exercise the 'else if' path in parseURL
        URL rel = new URL(base, "foo/bar", handler);

        // Assert
        assertEquals(SmbConstants.DEFAULT_PORT, rel.getPort(), "Default port should be applied");
        String actualPath = rel.getPath();
        assertNotNull(actualPath, "Path should not be null");

If you check the previous app, in the UI you can see that, up to now, it lets the client decide the `id` of the `Hero` to create. 😱

We shouldn't let that happen, they could overwrite an `id` we already have assigned in the DB. Deciding the `id` should be done by the **backend** or the **database**, **not by the client**.

        // With enforcement disabled, this should log a warning but not throw
        PreauthIntegrityService lenientService = new PreauthIntegrityService(secureRandom, PreauthIntegrityService.HASH_ALGO_SHA512, false);

        // Should not throw exception
        lenientService.updatePreauthHash("non-existent", "test".getBytes());

        // With enforcement enabled, should throw exception

        for (final String cmd : processedCommands) {
            assertFalse("Command should not contain ${url}", cmd.contains("${url}"));
            assertFalse("Command should not contain ${outputFile}", cmd.contains("${outputFile}"));
            assertTrue("Command should contain temp path", cmd.contains(tempPath));
            assertTrue("Command should contain output path", cmd.contains(outputPath));
        }
    }

        assertNotNull(res.getClasspath(), "null classpath after runtime transform");
        assertEquals(4, res.getClasspath().size(), "runtime classpath should have 4 entries");

        ArtifactMetadata md = res.getClasspath().get(3);
        assertEquals("1.1", md.getVersion(), "runtime artifact version should be 1.1");
    }

    // ------------------------------------------------------------------------------------------
    @Test

        helper.init();
        // destroy() cancels the task and flushes; should not throw
        helper.destroy();
    }

    @Test
    public void test_destroy_beforeInit() {
        LogNotificationHelper helper = new LogNotificationHelper();
        // destroy() without init() should not throw NPE because of null checks
        helper.destroy();
    }

    @Test

        valve1.setShowReport(true);
        valve1.setShowServerInfo(true);

        // valve2 should still have default values
        assertFalse("valve2 should have independent ShowReport setting", valve2.isShowReport());
        assertFalse("valve2 should have independent ShowServerInfo setting", valve2.isShowServerInfo());
    }

    @Test
    public void test_toggleSettingsMultipleTimes() {