GeezerVison

cc: @kumaraditya303 @erlend-aasland

🤖 New build scheduled with the buildbot fleet by @kumaraditya303 for commit bb4e191 🤖

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

Can you please add tests for basic type qualities:

• immutability
• disallow instantiation

Refleaks looks good:

0:00:00 load avg: 2.73 Run tests sequentially
0:00:00 load avg: 2.73 [1/1] test_decimal
beginning 6 repetitions
123456
......

== Tests result: SUCCESS ==

1 test OK.

Total duration: 26.0 sec
Tests result: SUCCESS

I found another interesting problem while testing. The following code will causes a segmentation fault:

>>> import decimal
>>> tp = type(decimal.Context().flags)  # SignalDict type
>>> tp()  # Segmentation fault

I guess that the problem is caused by accessing the wild pointer when executing the signaldict_repr function. But this problem is not caused by this PR. I tested version Python 3.10 on Linux, and this problem exists as well. Maybe this problem is worth to open a new issue.

There are currently two test classes for the C implementation: CWhitebox and SignatureTest. None of them are suitable for new tests?

Good point; I think the added tests can fit in the CWhitebox test case.

I'll merge this later today after a refleak test.

🤖 New build scheduled with the buildbot fleet by @kumaraditya303 for commit 2f4f538 🤖

If you want to schedule another build, you need to add the 🔨 test-with-refleak-buildbots label again.

I'll merge this later today after a refleak test.

@kumaraditya303, let me amend the tests before merging.

It would be nice to wait until #106209¹ be merged.

Good job, @CharlieZhao95!

Good job, @CharlieZhao95!

Thanks all! Let's move on to the next! :)