<div dir="ltr">Hi Leandro!<div> great work!! It follows the design of others refactoring and also with tests! Great job!! congratulations!</div><div><br></div><div> It is on github now.</div><div><br></div><div> There are a few things that would be great if you could do them:</div><div>1) When checking if the method references an instance variable, you are checking only in the method class but not in its superclasses. You should change </div><div>assertIsNotAccessingInstanceVariable: aMethodToMove<br> <br>      aMethodToMove methodClass instVarNames ....<br></div><div><br></div><div>with:</div><div>assertIsNotAccessingInstanceVariable: aMethodToMove<br>  <br>      aMethodToMove methodClass allInstVarNames <br></div><div><br></div><div>and write a test for it :-)</div><div>2) When checking for senders:</div><div>a) that should be done by the refactoring and not the applier. In the case there are senders a RefactoringWarning should be signaled. RefactoringWarning is resumable and it is currently handled by the applier to ask you if you want to continue or not.</div><div>b) do not raise the warning if the sender is only itself (and write a test for it :-) )</div><div><br></div><div> Let me know if you can do it!</div><div><br></div><div>Hernan</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 9, 2019 at 2:34 AM Leandro Yampolsky via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Sorry, I am sending now the file with the tests. I assumed it would be on the changeset.<div><br></div><div>Regards,</div><div>Leandro</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El lun., 9 dic. 2019 a las 2:21, Leandro Yampolsky (<<a href="mailto:leandroyampolsky@gmail.com" target="_blank">leandroyampolsky@gmail.com</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi everyone.<div><br><div>This is my first time posting here. I am sending a changeset that includes a new refactoring to move a method to an instance or class method (according to where it is), which I find could be useful at times.</div></div><div><br></div><div>It does some basic checks: if the method references instance variables, or if it has any local variable with the same name as an instance variable in the target class, it can't be done and an error message displays indicating that. Also, if there are any senders of the method in the same class, a confirmation prompt appears warning that there are senders.</div><div><br></div><div>Please, let me know if it needs to be changed in some way.</div><div><br></div><div>Regards,</div><div>Leandro</div></div>
</blockquote></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal"><span style="font-weight:bold">Hernán Wilkinson</span><br>Agile Software Development, Teaching & Coaching</span></font></span></span></span></strong></span></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">Phone: +54-011</span></font></span></span></span></strong></span><font face="tahoma, sans-serif" size="2">-4893-2057</font></div><div><strong style="font-family:tahoma,sans-serif;font-size:xx-small"><span style="font-size:8pt"><span style="font-size:small"><font size="2"><span style="font-weight:normal">Twitter: @HernanWilkinson</span></font></span></span></strong></div><div><span style="font-family:tahoma,sans-serif;font-size:xx-small;border-collapse:collapse"><strong><span style="font-size:8pt"><span><span style="font-size:small"><font size="2"><span style="font-weight:normal">site: <a href="http://www.10pines.com/" style="color:rgb(17,65,112)" target="_blank">http://www.10Pines.com</a></span></font></span></span></span></strong></span></div><div><font face="tahoma, sans-serif"><span style="border-collapse:collapse">Address: Alem 896</span></font>, Floor 6, Buenos Aires, Argentina</div></div></div></div></div></div></div></div></div></div></div></div>