From Jiri Vlasak to ~martijnbraam/public-inbox
18. 12. 2024 23:41:29 builds.sr.ht <builds@sr.ht>: > bare-py/patches/.build.yml: FAILED in 1m13s > > [Improve tests][0] v2 from [Jiri Vlasak][1] > > [0]: https://lists.sr.ht/~martijnbraam/public-inbox/patches/56528 > [1]: jiri.hubacek@gmail.com > > ✗ #1392462 FAILED bare-py/patches/.build.yml > https://builds.sr.ht/~martijnbraam/job/1392462 It looks there were some difficulties with srht bulds when the patchset was submitted. Re-triggering the build succeeded:
From Jiri Vlasak to ~martijnbraam/public-inbox
--- bare/test_primitives.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/bare/test_primitives.py b/bare/test_primitives.py index b5f477e..badec6f 100644 --- a/bare/test_primitives.py @@ -8,7 +8,7 @@ from bare import _pack_primitive, _unpack_primitive, BarePrimitive, TypeKind class Test(TestCase): @given(integers(min_value=0, max_value=255)) @given(integers(min_value=0, max_value=2**8 - 1)) [message trimmed]
From Jiri Vlasak to ~martijnbraam/public-inbox
--- bare/test_varint.py | 42 ++++++++++++++++++++++++++++++++++-------- 1 file changed, 34 insertions(+), 8 deletions(-) diff --git a/bare/test_varint.py b/bare/test_varint.py index 61e71f9..c20d065 100644 --- a/bare/test_varint.py @@ -8,25 +8,51 @@ from bare import _pack_varint, _unpack_varint class Test(TestCase): def test__pack_varint(self): self.assertEqual(_pack_varint(0), b'\0', 'uint zero') self.assertEqual(_pack_varint(0x7F), b'\x7f', 'uint 0x7f') [message trimmed]
From Jiri Vlasak to ~martijnbraam/public-inbox
--- bare/test_varint.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bare/test_varint.py b/bare/test_varint.py index d3cd034..61e71f9 100644 --- a/bare/test_varint.py @@ -27,7 +27,7 @@ class Test(TestCase): self.assertEqual(_unpack_varint(b'\0', 0, signed=True)[0], 0, 'int 0') self.assertEqual(_unpack_varint(b'\x02', 0, signed=True)[0], 1, 'int 1') self.assertEqual(_unpack_varint(b'\x01', 0, signed=True)[0], -1, 'int -1') self.assertEqual(_unpack_varint(b'\xf1\x14', 0, signed=True)[0], -1337, 'int -1') self.assertEqual(_unpack_varint(b'\xf1\x14', 0, signed=True)[0], -1337, 'int -1337') [message trimmed]
From Jiri Vlasak to ~martijnbraam/public-inbox
--- bare/test_primitives.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bare/test_primitives.py b/bare/test_primitives.py index 620b108..b5f477e 100644 --- a/bare/test_primitives.py @@ -155,4 +155,4 @@ class Test(TestCase): ast = BarePrimitive(TypeKind.Void) packed = _pack_primitive(ast, None) result = _unpack_primitive(ast, packed, 0)[0] assert result == None assert result is None [message trimmed]
From Jiri Vlasak to ~martijnbraam/public-inbox
--- bare/test_primitives.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bare/test_primitives.py b/bare/test_primitives.py index e18314e..620b108 100644 --- a/bare/test_primitives.py @@ -1,7 +1,7 @@ from math import isnan from unittest import TestCase from hypothesis import given, note, seed from hypothesis import given, note [message trimmed]
From Jiri Vlasak to ~martijnbraam/public-inbox
--- bare/test_primitives.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bare/test_primitives.py b/bare/test_primitives.py index 5315629..e18314e 100644 --- a/bare/test_primitives.py @@ -81,7 +81,7 @@ class Test(TestCase): assert result == number @given(integers()) def test_i64(self, number): def test_int(self, number): [message trimmed]
From Jiri Vlasak to ~martijnbraam/public-inbox
Hello Martijn, this is the second version after I found that the tests did not pass [1]. The original message: > I found out some changes I prepared last year to improve tests in the > bare-py project. It would be nice if you have a bit of time for the > review. Thank you, jiri
From Jiri Vlasak to ~martijnbraam/public-inbox
--- bare/test_varint.py | 42 ++++++++++++++++++++++++++++++++++-------- 1 file changed, 34 insertions(+), 8 deletions(-) diff --git a/bare/test_varint.py b/bare/test_varint.py index 61e71f9..c20d065 100644 --- a/bare/test_varint.py @@ -8,25 +8,51 @@ from bare import _pack_varint, _unpack_varint class Test(TestCase): def test__pack_varint(self): self.assertEqual(_pack_varint(0), b'\0', 'uint zero') self.assertEqual(_pack_varint(0x7F), b'\x7f', 'uint 0x7f') [message trimmed]
From Jiri Vlasak to ~martijnbraam/public-inbox
--- bare/test_varint.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bare/test_varint.py b/bare/test_varint.py index d3cd034..61e71f9 100644 --- a/bare/test_varint.py @@ -27,7 +27,7 @@ class Test(TestCase): self.assertEqual(_unpack_varint(b'\0', 0, signed=True)[0], 0, 'int 0') self.assertEqual(_unpack_varint(b'\x02', 0, signed=True)[0], 1, 'int 1') self.assertEqual(_unpack_varint(b'\x01', 0, signed=True)[0], -1, 'int -1') self.assertEqual(_unpack_varint(b'\xf1\x14', 0, signed=True)[0], -1337, 'int -1') self.assertEqual(_unpack_varint(b'\xf1\x14', 0, signed=True)[0], -1337, 'int -1337') [message trimmed]