>> That strange, Traits-adrian_lienhard.258 removes only two test >> methods >> (testTraitsUsersSanity and testUsersWithClassChanges). These two >> methods do not exist in my image. The comment said this fix the issue >> 443 (http://code.google.com/p/pharo/issues/detail?id=443). I do not >> see how you change is related to this bug. > > Traits-adrian_lienhard.258 should *add* the two tests you mention > above. They document the problem and should run green after loading > the Kernel package and after executing the script.
Sorry, I misread the merger. After having loaded Kernel-adrian_lienhard.252, testTraitsUsersSanity goes red (where testUsersWithClassChanges goes green). It seems that the following traits do not include their users: an IdentitySet(SetTest ArrayTest OrderedCollectionTest BagTest) Did I do something wrong? Cheers, Alexandre >> >> >> Hope it helps, >> Alexandre >> >> >> >> On 18 Jan 2009, at 18:27, Adrian Lienhard wrote: >> >>> I've published a fix to the inbox. Since the bug was related to >>> some >>> implementation details of the ClassBuilder and hence non-trivial, >>> I'd >>> appreciate if somebody could try it out and verify that traits still >>> work as expected. I've added tests that document the bug and they >>> run >>> green now (with the other 80 traits tests). >>> For details see http://code.google.com/p/pharo/issues/detail?id=443 >>> Adrian >>> >>> On Jan 17, 2009, at 15:29 , Adrian Lienhard wrote: >>> >>>> Well, if you just want to make your numbers be right, that's easy: >>>> >>>> Smalltalk allTraits do: [ :each | each instVarNamed: 'users' put: >>>> IdentitySet new ]. >>>> Smalltalk allClassesAndTraits do: [ :each | >>>> each hasTraitComposition ifTrue: [ each setTraitComposition: each >>>> traitComposition ] ]. >>>> >>>> This recreates all users sets. >>>> >>>> Adrian >>>> >>>> On Jan 15, 2009, at 18:20 , Damien Cassou wrote: >>>> >>>>> On Thu, Jan 15, 2009 at 6:18 PM, Adrian Lienhard <[email protected]> >>>>> wrote: >>>>>> BTW, why is this so pressing now? This bug has existed for four >>>>>> years. >>>>> >>>>> We are writing an article and we have lots of metrics >>>>> automatically >>>>> calculated (and the LaTeX tables are also automatically >>>>> generated). >>>>> Since the article and the metrics are about trait users... >>>>> >>>>> -- >>>>> Damien Cassou >>>>> http://damiencassou.seasidehosting.st >>>>> >>>>> _______________________________________________ >>>>> Pharo-project mailing list >>>>> [email protected] >>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo- >>>>> project >>>> >>>> >>>> _______________________________________________ >>>> Pharo-project mailing list >>>> [email protected] >>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >>> >>> _______________________________________________ >>> Pharo-project mailing list >>> [email protected] >>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project >>> >> >> -- >> _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: >> Alexandre Bergel http://www.bergel.eu >> ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. >> >> >> >> >> >> >> _______________________________________________ >> Pharo-project mailing list >> [email protected] >> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > > > _______________________________________________ > Pharo-project mailing list > [email protected] > http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project > -- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;. _______________________________________________ Pharo-project mailing list [email protected] http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
