Thank you for both reviews, Marcels! Merged via SUnit-ct.135. :-)<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-11T12:51:41+01:00, marcel.taeumel@hpi.de wrote:<br>
<br>
> Hi Christoph --<br>
> <br>
> That's correct. All current overrides of Object >> #error are ugly. I checked smalltalkCI. You proposed change should not break it. Maybe stay alert for a while and consult Fabio again after merging this.<br>
> <br>
> +1 Go ahead.<br>
> <br>
> Best,<br>
> Marcel<br>
> Am 10.01.2022 18:17:58 schrieb christoph.thiede at student.hpi.uni-potsdam.de <christoph.thiede at student.hpi.uni-potsdam.de>:<br>
> Hi all,<br>
> <br>
> I am still convinced that overriding #error as an accessor is not a good practice, but I am unsure whether this patch would be too much a breaking change, considering the amount of senders to this method in projects using Squeak or other Smalltalk distributions. Solely on GitHub:<br>
> <br>
> https://sourcegraph.com/search?q=context:global+%5CbTestResult%5Cs%2Berror%5Cb+lang:Smalltalk+-lang:c%23&patternType=regexp&case=yes<br>
> <br>
> What do you think, shall we merge or drop this?<br>
> <br>
> PS: Promise >> #error seems no better to me ...<br>
> <br>
> Best,<br>
> Christoph<br>
> <br>
> ---<br>
> Sent from Squeak Inbox Talk [https://github.com/hpi-swa-lab/squeak-inbox-talk]<br>
> <br>
> On 2020-09-24T09:28:33+00:00, commits at source.squeak.org wrote:<br>
> <br>
> > Christoph Thiede uploaded a new version of SUnit to project The Inbox:<br>
> > http://source.squeak.org/inbox/SUnit-ct.130.mcz<br>
> ><br>
> > ==================== Summary ====================<br>
> ><br>
> > Name: SUnit-ct.130<br>
> > Author: ct<br>
> > Time: 24 September 2020, 11:28:31.626426 am<br>
> > UUID: 43df5072-3a69-634c-946a-ec59903c2519<br>
> > Ancestors: SUnit-pre.122<br>
> ><br>
> > Deprecates TestResult class >> #error and replaces sends to it with #exError. It is a bad and erroneous practice to override #error in a way that does not signal an error but returns a class object.<br>
> ><br>
> > =============== Diff against SUnit-pre.122 ===============<br>
> ><br>
> > Item was changed:<br>
> > ----- Method: SUnitTest>>testDialectLocalizedException (in category 'tests') -----<br>
> > testDialectLocalizedException<br>
> ><br>
> >     self<br>
> >         should: [TestResult signalFailureWith: 'Foo']<br>
> >         raise: TestResult failure.<br>
> >     self<br>
> >         should: [TestResult signalErrorWith: 'Foo']<br>
> > +         raise: TestResult exError.!<br>
> > -         raise: TestResult error.<br>
> > -<br>
> > -             !<br>
> ><br>
> > Item was changed:<br>
> > ----- Method: SUnitTest>>testException (in category 'tests') -----<br>
> > testException<br>
> ><br>
> >     self<br>
> >         should: [self error: 'foo']<br>
> > +         raise: TestResult exError!<br>
> > -         raise: TestResult error<br>
> > -             !<br>
> ><br>
> > Item was changed:<br>
> > ----- Method: SUnitTest>>testWithExceptionDo (in category 'tests') -----<br>
> > testWithExceptionDo<br>
> ><br>
> >     self<br>
> >         should: [self error: 'foo']<br>
> > +         raise: TestResult exError<br>
> > -         raise: TestResult error<br>
> >         withExceptionDo: [:exception |<br>
> >             self assert: (exception description includesSubstring: 'foo')<br>
> > +         ]!<br>
> > -         ]<br>
> > -             !<br>
> ><br>
> > Item was changed:<br>
> > ----- Method: TestResult class>>error (in category 'exceptions') -----<br>
> > error<br>
> > +<br>
> > +     self deprecated: 'ct: Send #exError to retrieve an exception class or #error: to signal an error, depending on what you need.'.<br>
> > +     ^ self exError!<br>
> > -     ^self exError<br>
> > -             !<br>
> ><br>
> > Item was changed:<br>
> > ----- Method: TestResult class>>signalErrorWith: (in category 'exceptions') -----<br>
> > signalErrorWith: aString<br>
> > +     ^ self exError signal: aString!<br>
> > -     self error signal: aString<br>
> > -             !<br>
> ><br>
> ><br>
> -------------- next part --------------<br>
> An HTML attachment was scrubbed...<br>
> URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220111/33bf2579/attachment.html><br>
> <br>
>