* more bytes.
   *
   * <p>Note: Direct testing of the {@link IOException} scenario during write/flush is challenging
   * without mocking. This test verifies that normal operation with threshold crossing still works
   * correctly with the fix in place.
   */
  public void testThresholdCrossing_resourceManagement() throws Exception {
    FileBackedOutputStream out = new FileBackedOutputStream(/* fileThreshold= */ 10);

            // Clean up
            tempFile.delete();
        }
    }

    public void test_createResponseBodyFile_filePermissions() throws Exception {
        // Test that file permissions are set correctly
        ResponseData responseData = new ResponseData() {
            @Override
            public InputStream getResponseBody() {
                return new ByteArrayInputStream("Test permissions".getBytes());

        }
    }

    @Nested
    @DisplayName("Version Comparison")
    class VersionComparisonTests {

        @Test
        @DisplayName("should compare versions correctly")
        void shouldCompareVersionsCorrectly() {
            // Based on the actual implementation, it only handles specific cases
            assertTrue(ModelVersionUtils.isVersionGreaterOrEqual("4.1.0", "4.1.0"));

        int actual = (int) getPrivateField(req, "createOptions");
        assertEquals(0x0040, actual, "createOptions should be padded with 0x0040");
    }

    @Test
    @DisplayName("addFlags0 combines flags correctly")
    void addFlags0() {
        SmbComNTCreateAndX req = createRequest(0, 0);
        req.addFlags0(0x10);
        int flags0 = (int) getPrivateField(req, "flags0");

        assertEquals(dataLength, largeResponse.getDataLength());
        assertEquals(dataRemaining, largeResponse.getDataRemaining());
    }

    @Test
    @DisplayName("Should handle empty data correctly")
    void testReadBytesWireFormatEmptyData() throws Exception {
        // Given
        byte[] buffer = new byte[256];
        int bodyStart = 0;
        int dataLength = 0;
        int dataRemaining = 0;

        // The class should be serializable as it extends RuntimeException
        assertTrue(exception instanceof java.io.Serializable);
    }

    @Test
    @DisplayName("Should behave correctly in catch blocks")
    void testCatchBlockBehavior() {
        // Test that RuntimeCIFSException can be caught as RuntimeException
        assertThrows(RuntimeException.class, () -> {

     * the exponent. This is exactly the behavior we get from just adding signifRounded to bits
     * directly. If the exponent is MAX_DOUBLE_EXPONENT, we round up (correctly) to
     * Double.POSITIVE_INFINITY.
     */
    bits |= x.signum() & SIGN_MASK;
    return longBitsToDouble(bits);
  }

  /** Returns its argument if it is non-negative, zero if it is negative. */

            }
            lifecyclesPrinted = true;
        }

        // If people are going to make their own lifecycles then we need to tell people how to namespace them correctly
        // so that they don't interfere with internally defined lifecycles.

        Map<String, Lifecycle> phaseToLifecycleMap = new HashMap<>();

        for (Lifecycle lifecycle : getLifeCycles()) {

**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**:
- `validateInputStream()` method behavior

 * implement {@link java.util.RandomAccess}.
 *
 * <p>This class is not threadsafe when any concurrent operations update the multimap. Concurrent
 * read operations will work correctly. To allow concurrent update operations, wrap your multimap
 * with a call to {@link Multimaps#synchronizedListMultimap}.
 *
 * <p>See the Guava User Guide article on <a href=