[squeak-dev] The Inbox: Tests-cwp.158.mcz

Colin Putney colin at wiresong.com
Tue Aug 14 14:36:46 UTC 2012


On Tue, Aug 14, 2012 at 5:09 AM, Frank Shearar <frank.shearar at gmail.com> wrote:


> Why both testPrefFalse and testTokenishFalse (and both testPrefTrue
> and testTokenishTrue) if they both do the same thing?
> (testPref(True|False) are in the image now, so unless there's good
> reason, I don't see why we can't just remove testTokenish(True|False)
> from the above.
>
> Nit: I'd like testPref(True|False) categorised.
>
> Otherwise, +1 from me.

I renamed them. The new tests are related to selector precedence, so I
wanted the tokenish tests to have that word in their selectors. The
new versions are categorized too!

Colin


More information about the Squeak-dev mailing list