Committing changes from GustavoSantos/Refactoring2 on Smalltalkhub

[gustavojss]

In this commit I introduce:

• new package Refactoring2
• Code Transformations to be distinguished from Refactorings
• precondition checking is optional
• atomic transformations to be distinguished from composite ones
• examples of existing Refactoring, such as "Extract Method" and "Push Down Variable", as composite transformations
• introduction of new composite transformations, such as "Deprecate Class" and "Rename and Deprecate Class"

[Ducasse]

Super cool! I will have a look!

[Ducasse]

Pavel do you know if Refactoring2 should be added to a baseline? I have to check the code.

[Ducasse]

Gustavo do you have tests?

[Ducasse]

We saw the tests.
Now why do you have a new BRClass2?

[Ducasse]

You have tests for the transformations and I would like to know how I can change the refactorings to use your transformation and check the tests.

[gustavojss]

• I created RBNamespace2 and RBClass2 because I wanted to change the original classes without breaking the existing refactorings. It's in my long-term plan to completely refactor these classes because they are too big
• I test the preconditions in my tests; and for that purpose, I use transformations as refactorings. When you instantiate a *Transformation, you can send #asRefactoring then #transform

[Ducasse]

Thanks for the information I will have a look.

[pavel-krivanek]

I think that the tests should be placed in a separate package, they are now part of Refactoring2-Transformations