3. Add tests
4. Handle resources properly (try-with-resources)
5. Consider thread safety
6. Update JavaDoc

### When Fixing Bugs

1. Write failing test first
2. Understand root cause
3. Minimal changes
4. Verify no regressions

### When Refactoring

1. Preserve behavior
2. Keep tests green
3. Small incremental steps
4. Don't mix with new features

### Code Quality Checklist

     *
     * Expected outputs are generated by the code itself, but they've been
     * checked against the inputs under Bash in order to confirm that the two
     * forms are equivalent (though not necessarily minimal, though we hope this
     * to be the case). Thus, this test is more of a regression test.
     *
     * Rough instructions to regenerate the test outputs and verify correctness:
     * - Temporarily change this test:

            assertNotNull(fileFsSizeInfo.getFree());
        }

        @ParameterizedTest
        @DisplayName("Should decode various sector and allocation configurations")
        @CsvSource({ "1000, 500, 1, 512", // Minimal sectors per alloc
                "1000, 500, 8, 512", // Typical configuration
                "1000, 500, 64, 512", // Large allocation units
                "1000, 500, 8, 4096", // 4K sectors

    @BeforeEach
    void setUp() {
        // Setup will be done per test as needed
    }

    @Test
    void testConstructorWithMinimalParameters() throws UnknownHostException {
        // Test the constructor with minimal parameters
        mockName = new Name(mockConfig, "TESTHOST", 0x20, null);
        NbtAddress nbtAddress = new NbtAddress(mockName, testAddressInt, false, NbtAddress.H_NODE);

        assertNotNull(nbtAddress);

    @DisplayName("Mockito presence: no collaborator interactions to verify")
    void noDependenciesToMock() {
        // This class has no external collaborators; nothing to verify.
        // Keep Mockito import usage minimal to satisfy build-time linkage.
        Runnable r = mock(Runnable.class);
        r.run();
        verify(r, times(1)).run();
    }

 * outside the ASCII character range. Unlike HTML escaping the XML escapers will not escape
 * non-ASCII characters to their numeric entity replacements. These XML escapers provide the minimal
 * level of escaping to ensure that the output can be safely included in a Unicode XML document.
 *
 * <p>For details on the behavior of the escapers in this class, see sections <a

        // Set buffer offset
        SMBUtil.writeInt2(20, buffer, bufferIndex + 2);
        // Set buffer length (minimal SecurityDescriptor)
        SMBUtil.writeInt4(20, buffer, bufferIndex + 4);

        response = spy(response);
        when(response.getHeaderStart()).thenReturn(0);

        // Write minimal SecurityDescriptor header (20 bytes)
        buffer[20] = 1; // Revision
        buffer[21] = 0; // Sbz1

        void dispose_nullContext_noop() throws Exception {
            setPrivateField(ctx, "gssContext", null);
            assertDoesNotThrow(() -> ctx.dispose());
        }
    }

    // Helper to build a minimal exported name token understood by MIEName(byte[])
    // Format: TOK_ID(2 bytes) | OID_LEN(2 bytes) | OID_DER | NAME_LEN(4 bytes) | NAME(bytes)
    private static byte[] buildExportName(Oid mech, String name) throws GSSException {

    }

    @Test
    @DisplayName("getSmbSession creates and then reuses matching session")
    void getSmbSession_createAndReuse() {
        // Arrange: minimal credentials chain so SmbSessionImpl constructor succeeds
        CredentialsInternal creds = mock(CredentialsInternal.class);
        when(ctx.getCredentials()).thenReturn(creds);

        }

        @ParameterizedTest
        @DisplayName("Should decode various sector and allocation configurations")
        @CsvSource({ "1000, 500, 600, 1, 512", // Minimal sectors per alloc
                "1000, 500, 600, 8, 512", // Typical configuration
                "1000, 500, 600, 64, 512", // Large allocation units
                "1000, 500, 600, 8, 4096", // 4K sectors