@Test
        @DisplayName("Should create with configuration only")
        void testConstructorWithConfig() {
            ServerMessageBlock2 msg = new TestServerMessageBlock2(mockConfig);
            assertNotNull(msg);
            assertEquals(mockConfig, msg.getConfig());
            assertEquals(0, msg.getCommand());
        }

        @Test
        @DisplayName("Should create with configuration and command")

   - Multiple reviewers are encouraged for complex PRs.

5. **No Self-Edits**:
   - Don’t modify the PR directly (e.g., fixing bugs). Request changes from the submitter or create a follow-up PR.
   - If you edit, you’re a collaborator, not a reviewer, and cannot merge.

6. **Testing**:
   - Assume the submitter tested the code. If testing is unclear, ask for details (e.g., “How was this tested?”).

    }

    @Test
    @DisplayName("Options map is not mutated by configuration (verify no writes)")
    void optionsMap_notMutated_verifyNoWrites() {
        // Arrange: create a regular map first, then spy on it
        Map<String, Object> originalMap = new HashMap<>();
        originalMap.put("refreshKrb5Config", "true");

        @SuppressWarnings("unchecked")

        notification = new Smb2OplockBreakNotification(mockConfig);
    }

    @Nested
    @DisplayName("Constructor Tests")
    class ConstructorTests {

        @Test
        @DisplayName("Should create notification with configuration")
        void testConstructorWithConfiguration() {
            Smb2OplockBreakNotification testNotification = new Smb2OplockBreakNotification(mockConfig);
            assertNotNull(testNotification);

        // Make many attempts to trigger global rate limit
        for (int i = 1; i <= 20; i++) {
            assertTrue(rateLimiter.checkAttempt("user" + i, "192.168.1." + i), "Attempt " + i + " should be allowed");
        }

        // 21st attempt should be blocked by global rate limit
        assertFalse(rateLimiter.checkAttempt("user21", "192.168.1.21"), "Should block after global rate limit");
    }

    @Test

        this.stopWords =
                Analysis.parseWords(env, settings, "stopwords", JapaneseAnalyzer.getDefaultStopSet(), NAMED_STOP_WORDS, ignoreCase);
    }

    @Override
    public TokenStream create(TokenStream tokenStream) {
        if (removeTrailing) {
            return new StopFilter(tokenStream, stopWords);
        } else {
            return new SuggestStopFilter(tokenStream, stopWords);
        }
    }

    }

    /**
     * Test that the constructor stores the reference to the input byte array.
     * Note: The implementation does not create a defensive copy.
     */
    @Test
    void testConstructorStoresReference() {
        byte[] originalHash = { 0x01, 0x02, 0x03 };
        AvChannelBindings avChannelBindings = new AvChannelBindings(originalHash);

        response = new Smb2WriteResponse(mockConfig);
    }

    @Nested
    @DisplayName("Constructor Tests")
    class ConstructorTests {

        @Test
        @DisplayName("Should create response with configuration")
        void testConstructor() {
            Smb2WriteResponse writeResponse = new Smb2WriteResponse(mockConfig);
            assertNotNull(writeResponse);

        lenient().when(mockContext.getConfig()).thenReturn(mockConfig);

        request = new Smb2ChangeNotifyRequest(mockConfig, testFileId);
    }

    @Test
    @DisplayName("Should create request with configuration and file ID")
    void testConstructor() {
        // Given
        Configuration config = mock(Configuration.class);
        when(config.getNotifyBufferSize()).thenReturn(8192);

    private final AtomicLong totalIpsBlocked = new AtomicLong(0);

    /**
     * Create rate limiter with default settings
     */
    public AuthenticationRateLimiter() {
        this(5, 10, 100, Duration.ofMinutes(30), Duration.ofMinutes(5));
    }

    /**
     * Create rate limiter with custom settings
     *
     * @param maxAttemptsPerAccount max failed attempts per account before lockout