[squeak-dev] The Inbox: SUnit-ct.121.mcz

David T. Lewis lewis at mail.msen.com
Sat Dec 7 21:06:02 UTC 2019


Yes, it should be 'tests'.
   
But don't feel badly about picking the "wrong" category, just have
a look at what all the rest of us have done over the years:
   
   Dictionary withAll: ( { 'testing' . 'Testing' . 'tests' . 'test' }
      collect: [ :cat |
         cat -> (TestCase withAllSubclasses
            select: [ :cls | cls organization categories includes: cat ])]).

Dave


On Sat, Dec 07, 2019 at 08:40:39PM +0000, Thiede, Christoph wrote:
> 'tests'?
> 
> ________________________________
> Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Chris Muller <asqueaker at gmail.com>
> Gesendet: Samstag, 7. Dezember 2019 21:07:59
> An: squeak dev
> Betreff: Re: [squeak-dev] The Inbox: SUnit-ct.121.mcz
> 
> "testing" is for messages that answer booleans.  For example, TestCase>>#shouldPass.
> 
> The #test... methods should be categorized differently.
> 
>  - Chris
> 
> 
> On Sat, Dec 7, 2019 at 8:31 AM <commits at source.squeak.org<mailto:commits at source.squeak.org>> wrote:
> A new version of SUnit was added to project The Inbox:
> http://source.squeak.org/inbox/SUnit-ct.121.mcz
> 
> ==================== Summary ====================
> 
> Name: SUnit-ct.121
> Author: ct
> Time: 7 December 2019, 3:31:28.823146 pm
> UUID: 56da0773-e6ce-8d44-9496-57eea4253f3f
> Ancestors: SUnit-mt.120
> 
> Specify default message category for test selectors. Depends on Kernel-mt.1284.
> 
> =============== Diff against SUnit-mt.120 ===============
> 
> Item was added:
> + ----- Method: TestCase class>>defaultCategoryForSelector: (in category 'organization') -----
> + defaultCategoryForSelector: aSelector
> +
> +       aSelector isTestSelector
> +               ifTrue: [^ #testing].
> +       ^ super defaultCategoryForSelector: aSelector!
> 
> 

> 



More information about the Squeak-dev mailing list