feat(contracts): revert on unknown operator

[mempirate]

No description provided.

[merklefruit]

by looking at the implementation of pause() in PauseableEnumerableSet, this doesn't seem necessary as it's covered by the library. However if we want to provide custom errors and be more explicit, I am not too much against a double check...

sauce:

function pause(Uint160Set storage self, uint48 timestamp, uint160 value) internal {
        if (self.positions[value] == 0) revert NotRegistered(); // <--- covered here
        self.array[self.positions[value] - 1].status.disable(timestamp);
    }

[mempirate]

Yeah this was because stuff was reverting in the CLI without a clear error, so it's about being explicit

[merklefruit]

all good, that works!