Le 29/3/15 21:30, Thierry Goubier a écrit :
Le 29/03/2015 21:18, stepharo a écrit :
In fact this is even worse (BTW I used the latest Pillar image)
I renamed AA into CC and then CC into DD
and now
self name = CC
We broke something and I find surprising that we do not have test
covering refactorings after all these years.
Do you mean that RBRenameClassTest covers nothing?
I do not know. But what is clear is that
- Refactoring should not be difficult to test well
- we break them regularly
Thierry
We broke already two times refactorings and this is a bad signal.
Stef
Le 29/3/15 21:14, stepharo a écrit :
In fact I tried on another class that I did not renamed
And I get the same behavior.
Now what is really strange is that I did
AA renamed into CC
and when I do a push down on XX which has nothing to do with AA or CC
the
^baseClass realClass
-> self name = CC
so I do not get how this is possible.
Stef
Le 29/3/15 21:09, stepharo a écrit :
Hi
When I select an instance method and use push down in a subclass
without obvious conflicts.
I get two DNUs.
realClass
^baseClass realClass theMetaClass
dnu: theMetaClass
while baseClass is a class
realClass -> nil
the environement does not contain the class.
Now I did a class rename before and I wonder if it is not the one
that is broken :(
Stef