2 commits: lowLevelError + testExpectEncodeStg

[ZeroEkkusu]

feat(stdError): add lowLevelError

Motivation

Cover INVALID opcode cases, such as trying to directly access an element of an array in another contract that is oob, or when there was an error while decoding the error message (0x).

Showcase with arrays

contract C {
    bool[] public bools;

    constructor() {
        bools.push(true);
    }
}

contract TestC is DSTest {
    function testLowLevelError() public {
        Vm vm = Vm(HEVM_ADDRESS);
        C c = new C();
        // P.S. No, we cannot use `indexOOBError` here
        vm.expectRevert(stdError.lowLevelError);
        c.bools(1);
    }
}

test(stdError): add testExpectEncodeStg

Add a test for encodeStorageError. This was marked as TODO.

/// INSIDE TESTING CONTRACT

function testExpectEncodeStg() public {
    vm.expectRevert(stdError.encodeStorageError);
    test.encodeStgError();
}

/// INSIDE DUMMY CONTRACT

bytes someBytes;

function encodeStgError() public {
    assembly {
        sstore(someBytes.slot, 1)
    }
    bytes memory b = someBytes;
}

[brockelmore]

lmk wen ready for review

[ZeroEkkusu]

if it isn't draft, it is ready :)

[brockelmore]

if it isn't draft, it is ready :)

plz request reviews going forward regardless :)

[ZeroEkkusu]

i can't formally request reviews as a non-collaborator

will @ u in the future, if that's what you meant

Edit: 😳

[brockelmore]

lgtm :)