}
    }

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

        @Test
        @DisplayName("Should handle malformed buffer gracefully")
        void testReadDataWireFormatMalformedBuffer() {
            byte[] buffer = new byte[3]; // Too small for valid DFS referral

            assertThrows(ArrayIndexOutOfBoundsException.class, () -> {

        Mockito.doThrow(new RuntimeException("Test exception")).when(mockTransport).release();

        session.acquire();

        // Should handle exceptions gracefully
        assertDoesNotThrow(() -> session.release());
    }

    /**
     * Test AutoCloseable pattern
     */
    @Test
    @Timeout(5)
    public void testAutoCloseablePattern() throws Exception {

# Test file for malformed robots.txt parsing
# This file contains various malformed directives that should be handled gracefully

# Case 1: Directives before any User-agent (should be ignored)
Disallow: /orphaned1/
Allow: /orphaned2/

# Case 2: Valid user-agent with various malformed directives
User-agent: GoodBot
Disallow: /admin/
InvalidDirective: some-value
unknown-field: test
Disallow /missing-colon

        assertEquals(intermediateCause, finalException.getCause());
        assertEquals(rootCause, finalException.getCause().getCause());
    }

    @Test
    @DisplayName("Should handle null message gracefully")
    void testNullMessage() {
        // When/Then
        assertDoesNotThrow(() -> {
            SmbException exception = new SmbException((String) null);
            assertNotNull(exception);
        });

        verify(mockDelegate).isIpcSigningEnforced();
        verify(mockDelegate, times(2)).isForceExtendedSecurity();
    }

    @Test
    @DisplayName("Delegation should handle null returns gracefully")
    void testNullReturnHandling() {
        // Given - configure delegate to return nulls
        when(mockDelegate.getRandom()).thenReturn(null);
        when(mockDelegate.getLocalAddr()).thenReturn(null);

        void testNullDataNonZeroLength() {
            request.setData(null, 0, 100);

            byte[] buffer = new byte[256];

            // Should handle gracefully or throw appropriate exception
            assertThrows(NullPointerException.class, () -> request.writeBytesWireFormat(buffer, 0));
        }

        @Test

- ZipExtractor
- TarExtractor

**Test Count**: 12 tests

**Key Scenarios**:
- ✅ Descriptive error messages for invalid archives
- ✅ Continues processing when some entries fail
- ✅ Handles empty archives gracefully
- ✅ Mixed valid/invalid entries processed correctly
- ✅ Null input stream validation

---

### 4. AbstractExtractorTest.java
**Purpose**: Test the base AbstractExtractor functionality.

**Key Test Areas**:

        assertEquals(0, buffer[1], "Second byte must be null terminator");
    }

    @Test
    @DisplayName("writeBytesWireFormat handles empty string gracefully")
    void testWriteEmptyString() {
        when(mockConfig.getOemEncoding()).thenReturn("windows-1252");
        SmbComQueryInformation emptyCmd = new SmbComQueryInformation(mockConfig, "");

        // but we can verify the context behaves correctly after wiping)
        // After wiping, operations that require keys should fail

        // Attempting to use the context after wiping should handle gracefully
        // Note: The actual implementation might throw an exception or handle differently
    }

    @Test
    @DisplayName("Should securely wipe keys when closing context")

                final String msg = ex.getMessage();
                final boolean timeout = msg != null && msg.equals("Read timed out");
                /* If just a timeout, try to disconnect gracefully
                 */
                final boolean hard = !timeout;

                if (!timeout && LogStream.level >= 3) {
                    ex.printStackTrace(log);
                }