// nothing really to assert?
    assertSame(CharStreams.nullWriter(), CharStreams.nullWriter());
  }

  /**
   * Returns a reader wrapping the given reader that only reads half of the maximum number of
   * characters that it could read in read(char[], int, int).
   */
  private static Reader newNonBufferFillingReader(Reader reader) {
    return new FilterReader(reader) {
      @Override

    }

    /**
     * Reads a null-terminated string from the buffer with specified encoding
     * @param src source buffer
     * @param srcIndex starting index in the buffer
     * @param maxLen maximum length to read
     * @param unicode whether to use Unicode encoding
     * @return read string
     */
    public String readString(final byte[] src, int srcIndex, final int maxLen, final boolean unicode) {

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

        assertEquals(0, bytesRead);
    }

    @Test
    @DisplayName("Test size calculation with different buffer sizes")
    void testSizeCalculationWithDifferentBufferSizes() {
        // Test with smaller maximum buffer size
        when(mockConfig.getMaximumBufferSize()).thenReturn(8192);
        when(mockConfig.getListSize()).thenReturn(16384);

        request = new Smb2QueryDirectoryRequest(mockConfig);

        assertThrows(SMBProtocolDecodingException.class, () -> {
            notifyInfo.decode(buffer, 0, buffer.length);
        });
    }

    @Test
    @DisplayName("Test decode with maximum file name length")
    void testDecodeMaxFileNameLength() throws SMBProtocolDecodingException {
        // Create a very long filename (but within reasonable bounds)
        StringBuilder sb = new StringBuilder();

        }

        @Test
        @DisplayName("parse should handle truncated message missing context")
        void testParse_InputLengthTooSmallForContext() {
            // Given
            // Create a minimal Type2 message manually
            byte[] truncatedMessage = new byte[32];
            System.arraycopy(Type2Message.NTLMSSP_SIGNATURE, 0, truncatedMessage, 0, 8);

     */
    public long getMinResponseTime() {
        long min = minResponseTime.get();
        return min == Long.MAX_VALUE ? 0 : min;
    }

    /**
     * Get maximum response time
     *
     * @return maximum response time in milliseconds
     */
    public long getMaxResponseTime() {
        return maxResponseTime.get();
    }

    /**
     * Close circuit breaker and cleanup resources

    @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);

        }
    }

    @Nested
    @DisplayName("Edge Cases and Error Conditions")
    class EdgeCasesTests {

        @Test
        @DisplayName("Should handle maximum size arrays")
        void testMaximumSizeArrays() {
            // Test with reasonably large arrays
            int[] largeCiphers = new int[100];
            for (int i = 0; i < 100; i++) {

        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 {