Issue 7169 in pharo: Failing test: testInstanceSideMethodsWithNilKeyInLastLiteral

Status: FailingTest
Owner: [hidden email]
Labels: Type-FailingTest Milestone-2.0

New issue 7169 by [hidden email]: Failing test:  
testInstanceSideMethodsWithNilKeyInLastLiteral
http://code.google.com/p/pharo/issues/detail?id=7169

ReleaseTest>>#testInstanceSideMethodsWithNilKeyInLastLiteral is failing


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 7169 by [hidden email]: Failing test:  
testInstanceSideMethodsWithNilKeyInLastLiteral
http://code.google.com/p/pharo/issues/detail?id=7169

Execute this:

SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral do:  
[ :thisShouldNotBeAStringButAMethod |
        | method classAndSelector |
        classAndSelector := ' ' split: thisShouldNotBeAStringButAMethod.
        method := (Smalltalk at: classAndSelector first asSymbol) methodNamed:  
classAndSelector second asSymbol.
        method recompile.
].

By the way SystemNavigation needs a serious review. For example if  
allSelect: (which is a bad name btw) returns methods instead of string the  
preceding snippet gets reduced to:
SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral do:  
#recompile
And there is a lot of inconsistent stuff so I'll open a issue for it


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixToInclude

Comment #2 on issue 7169 by [hidden email]: Failing test:  
testInstanceSideMethodsWithNilKeyInLastLiteral
http://code.google.com/p/pharo/issues/detail?id=7169

Ok, works

Another question is who installed the wrong methods...


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #3 on issue 7169 by [hidden email]: Failing test:  
testInstanceSideMethodsWithNilKeyInLastLiteral
http://code.google.com/p/pharo/issues/detail?id=7169

in 2.0 467


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #4 on issue 7169 by [hidden email]: Failing test:  
testInstanceSideMethodsWithNilKeyInLastLiteral
http://code.google.com/p/pharo/issues/detail?id=7169

>> Another question is who installed the wrong methods...

Nobody, it's RB tests that change the shape of a mock class and boom, see  
issue 7240.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #5 on issue 7169 by marianopeck: Failing test:  
testInstanceSideMethodsWithNilKeyInLastLiteral
http://code.google.com/p/pharo/issues/detail?id=7169

so can we make RB tests silent?


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker