# ADR-0004 - Use a platform-oriented architecture for Gradle

## Date

2024-02-07

## Context

The Gradle code base is essentially a large monolith, without strong internal boundaries.
This has a number of negative effects on productivity, including:

- Unclear ownership of code.
- Difficult to focus on one particular area.
- Unintended coupling between areas of the code, including tests.

## Decision

        SMBUtil.writeInt2(getSubCommand() & 0xFF, dst, dstIndex);
        dstIndex += 2;
        System.arraycopy(descr, 0, dst, dstIndex, descr.length);
        dstIndex += descr.length;
        SMBUtil.writeInt2(0x0001, dst, dstIndex);
        dstIndex += 2;
        SMBUtil.writeInt2(this.maxDataCount, dst, dstIndex);
        dstIndex += 2;
        SMBUtil.writeInt4(this.serverTypes, dst, dstIndex);
        dstIndex += 4;

        assertEquals((byte) 0x10, SmbComTransaction.TRANS2_GET_DFS_REFERRAL);
        assertEquals((byte) 0x08, SmbComTransaction.TRANS2_SET_FILE_INFORMATION);

        // Test NET subcommands
        assertEquals(0x0000, SmbComTransaction.NET_SHARE_ENUM);
        assertEquals(0x0068, SmbComTransaction.NET_SERVER_ENUM2);
        assertEquals(0x00D7, SmbComTransaction.NET_SERVER_ENUM3);

        // Test TRANS subcommands

    void createOptionsAddedWhenLow() {
        // createOptions less than 0x0001 (i.e., 0) should be padded with 0x0040
        SmbComNTCreateAndX req = createRequest(0, 0);
        int actual = (int) getPrivateField(req, "createOptions");
        assertEquals(0x0040, actual, "createOptions should be padded with 0x0040");
    }

    @Test
    @DisplayName("addFlags0 combines flags correctly")
    void addFlags0() {

    @DisplayName("Should support AES-256 cipher constants")
    void testAES256CipherConstants() {
        // Verify AES-256 constants are defined
        assertEquals(0x0003, Smb2EncryptionContext.CIPHER_AES_256_CCM, "AES-256-CCM constant should be defined");
        assertEquals(0x0004, Smb2EncryptionContext.CIPHER_AES_256_GCM, "AES-256-GCM constant should be defined");

        // Test creating context with AES-256 cipher IDs

                createDisposition = FILE_OPEN_IF;
            }
        } else {
            createDisposition = FILE_OPEN;
        }

        if ((createOptions & 0x0001) == 0) {
            this.createOptions = createOptions | 0x0040;
        } else {
            this.createOptions = createOptions;
        }
        impersonationLevel = 0x02; // As seen on NT :~)

            buffer[16 + i] = expectedData[i];
        }

        // Set header values
        SMBUtil.writeInt4(0x100, buffer, 0);
        SMBUtil.writeInt4(0x200, buffer, 4);
        SMBUtil.writeInt4(0x300, buffer, 8);
        SMBUtil.writeInt4(0x400, buffer, 12);

        // Decode
        response.decode(buffer, 0, 32);

        // Verify data integrity

            dstIndex++;
            dst[dstIndex++] = (byte) 0x00; // Reserved1
            writeInt2(flags, dst, dstIndex);
            dstIndex += 2;
            writeInt4(timeout, dst, dstIndex);
            dstIndex += 4;
            dst[dstIndex++] = (byte) 0x00; // Reserved2
            dst[dstIndex++] = (byte) 0x00;
        }
        writeInt2(parameterCount, dst, dstIndex);
        dstIndex += 2;

    }

    @Test
    void testWriteHeaderReadHeader() {
        byte[] buffer = new byte[32];
        smb.command = ServerMessageBlock.SMB_COM_NEGOTIATE;
        smb.flags = (byte) 0x18;
        smb.flags2 = 0x0001;
        smb.tid = 1;
        smb.pid = 2;
        smb.uid = 3;
        smb.mid = 4;

        smb.writeHeaderWireFormat(buffer, 0);

        TestServerMessageBlock readSmb = new TestServerMessageBlock();

/**
 * Test class for ValidateNegotiateInfoRequest
 */
class ValidateNegotiateInfoRequestTest {

    private static final int DEFAULT_CAPABILITIES = 0x12345678;
    private static final int DEFAULT_SECURITY_MODE = 0x0003;
    private byte[] defaultClientGuid;
    private int[] defaultDialects;

    @BeforeEach
    void setUp() {
        // Initialize default GUID (16 bytes)
        defaultClientGuid = new byte[16];