2015-10-27 16:45 GMT+01:00 Marcus Denker <marcus.den...@inria.fr>: > > On 15 Oct 2015, at 09:36, Marcus Denker <marcus.den...@inria.fr> wrote: > > Yes, > > -> there is a bug somewhere that flattens traits in some cases > -> we need to write a script to delete all the wrong methods. > > > I inspected the failing lint rule and executed on the dictionary: > > self keys do: [:classSymbol | > (self at: classSymbol) do: [:selector | > classSymbol asClass removeSelector: selector. > ]] > > resulting slice is committed: > https://pharo.fogbugz.com/f/cases/16254/ > > Good!
The strange thing is, I said I ve found the reason why this trait methods were moved to the class. But I can not reproduce it anymore. I was sure "RPackageSet methods " would collect all trait and non-traitmethods of a class and this gets written into thet package. But now, I can not reproduce this anymore. > On 14 Oct 2015, at 17:16, Nicolai Hess <nicolaih...@web.de> wrote: > > Please, someone can look at the Collections-Tests > The ReleaseTest complains about > > testLocalMethodsOfTheClassShouldNotBeRepeatedInItsTraits > (These are all Test-subclasses that use traits, but all(!) trait method are > compiled in the class instead, this happened after someone moved the Test > packages (for bootstrap?) > > OrderedCollectionTest > FloatArrayTest > ArrayTest > DateTest > HeapTest > MorphicTextAdapter > LinkedListTest > DatePrintFormatTester > StackTest > CheckboxButtonMorph > IntervalTest > DateAndTimeTest > SimpleButtonMorph > MethodDictionaryTest > SetTest > DictionaryTest > BagTest > SymbolTest > StringTest > CollectionRootTest > SortedCollectionTest > > > >
