assertFalse((executeMask & ACE.FILE_WRITE_DATA) != 0, "Execute should not include write access");
        }

        @Test
        @DisplayName("Should validate documented bit comment consistency")
        void shouldValidateDocumentedBitComments() {
            // Verify that comments in source match actual constant values (not bit positions)

        assertTrue("Should contain doc_freq", json.contains("100"));
    }

    // ============================================================
    // Tests for getId consistency
    // ============================================================

    @Test
    public void test_getId_sameTextSameId() {
        String[] text1 = { "test" };
        String[][] readings1 = { { "reading1" } };

    public static final int SMB2_LEASE_FULL           = 0x07;  // RWH
}
```

### 3.2 Lease Versions
- **Lease V1**: Basic lease support (SMB 3.0)
- **Lease V2**: Adds epoch support for better consistency (SMB 3.0.2+)

## 4. Data Structures

### 4.1 Lease Key Structure
```java
package jcifs.internal.smb2.lease;

import java.security.SecureRandom;
import java.util.Arrays;

public class Smb2LeaseKey {

                .filter(rap -> !allAvailableProfiles.contains(rap))
                .collect(toSet());

        if (!notFoundRequiredProfiles.isEmpty()) {
            // Use SLF4J formatter for consistency with warnings reported by logger
            final String message = MessageFormatter.format(
                            "The requested profiles {} could not be activated or deactivated because they do not"

We don't really need to inherit from `HeroBase` because we are re-declaring all the fields. I'll leave it inheriting just for consistency, but this is not necessary. It's more a matter of personal taste. 🤷

The fields of `HeroUpdate` are:

* `name`
* `age`
* `secret_name`

    }

    /**
     * @return <code>true</code> if the Mojo is online, <code>false</code> otherwise.
     */
    // blech! this isn't even intelligible as a method name. provided for
    // consistency...
    public boolean isOnlineRequired() {
        return onlineRequired;
    }

    /**
     * @return <code>true</code> if the Mojo is online, <code>false</code> otherwise.
     */

  }

  /**
   * Returns the empty map. This map behaves and performs comparably to {@link
   * Collections#emptyMap}, and is preferable mainly for consistency and maintainability of your
   * code.
   *
   * <p><b>Performance note:</b> the instance returned is a singleton.
   */
  @SuppressWarnings("unchecked")
  public static <K, V> ImmutableMap<K, V> of() {

 * href="https://docs.oracle.com/en/java/javase/25/docs/api/java.base/java/util/Set.html#unmodifiable">{@code
 * Set.of}</a>, we recommend using <i>these</i> classes instead for this reason (as well as for
 * consistency).
 *
 * <h4>Creation</h4>
 *
 * <p>Except for logically "abstract" types like {@code ImmutableCollection} itself, each {@code

    assertEquals(initialFiles.length + 1, updatedFiles.length);
}
```

## 9. Error Handling

### 9.1 Cache Consistency
```java
public class CacheConsistencyManager {
    public void handleInconsistency(String directoryPath, String fileName) {
        log.warn("Cache inconsistency detected for: {}/{}", directoryPath, fileName);
        
        // Invalidate affected cache entries

      throw new UnsupportedOperationException();
    }

    @Override
    public String toString() {
      return getKey() + "=" + getValue();
    }
  }

  // TODO(fry): Separate logic for consistency between emul and nonemul implementation.
  // TODO(fry): Look into Maps.KeySet and Maps.Values, which can ideally be reused here but are
  // currently only package visible.