Inbox because I would not be surprised if there were many (external) senders of this assertion that have aligned their assumptions about this selector to the previous reality. But I don't like this idea at all.<br>
<br>
Here is a different idea which I even tend to prefer: Deprecate meaningless #shouldFix: and introduce new #shouldFail: which scans for TestResult exError.<br>
<br>
As it turns out now, "TestResult error" was indeed used in more places than I was assuming which I now need to rewrite. If you agree to introduce #shouldFail:, then I can rewrite ~40 senders in the Trunk image from "self should: [...] raise: TestResult error" to "self shouldFail: [...]", which I would consider more elegant anyway. :-) Please give feedback!<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><font color="#808080"><i>Sent from </i></font><font color="#808080"><i><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><u><font color="#808080">Squeak Inbox Talk</font></u></a></i></font><br>
<br>
On 2022-01-19T22:54:56+00:00, commits@source.squeak.org wrote:<br>
<br>
> A new version of SUnit was added to project The Inbox:<br>
> http://source.squeak.org/inbox/SUnit-ct.137.mcz<br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: SUnit-ct.137<br>
> Author: ct<br>
> Time: 19 January 2022, 11:54:53.214828 pm<br>
> UUID: dd617d36-ff31-f143-932d-09d1c332f5dc<br>
> Ancestors: SUnit-ct.136<br>
> <br>
> Revises exception choice in TestCase >> #shouldFix:. It is a bad practice to handle, or even to search for arbitary Exceptions. Many mechanisms signal possibly unhandled "harmless" exceptions and rely on their #defaultAction. This includes accesses to the source files, debugging operations, and many others. The assertion in its current form makes little sense.<br>
> <br>
> =============== Diff against SUnit-ct.136 ===============<br>
> <br>
> Item was changed:<br>
>   ----- Method: TestCase>>shouldFix: (in category 'asserting - extensions') -----<br>
>   shouldFix: aBlock <br>
>   <br>
> +     ^self should: aBlock raise: TestResult exError!<br>
> -     ^self should: aBlock raise: Exception!<br>
> <br>
>