Hi,
> I extended the test with duplicated hex variants to better visualize the
> boundaries:
> @Test
> public void checkUnsignedBoundsLongHex() {
>// Hex representation to visualize valid bounds in bytes
>ByteValue.checkUnsignedBounds(0x0_00_00, 2);
>
Hi Justin,
First of all thanks for the Tests (and finding the Bug :). You are absolutely
right about your statements but I don’t think its a issue of focus rather than
time and resources (at least in my case). My intention was to bring the ADS
branch to master a soon as possible to encourage
I take a look at it.
Regarding breaking build: somehow our pipeline implementation handles a test
fail a failure rather than unstable.
So from my POV it would be a +1 on fixing obvious small bugs as soon as you
find them, but when we find a bug with a test that we can’t immediately fix I
think
