Re: [cdt-dev] help needed: CDT refactoring test suite results

[Martin Oberhuber <mober.at+eclipse@xxxxxxxxx>]

FYI, I've started to have a quick look at how the CDT does with the refactoring suite:

https://github.com/moberhuber/refactor-test-suite

Doing so, I've found an 8-year-old bugzilla that includes a patch as well as testcases 

for one of the refactorings that currently fails (rename a class when positioned on the constructor):

https://bugs.eclipse.org/bugs/show_bug.cgi?id=319506

@Nate, Thomas or other committers - could you work with the submitter on rebasing

the patch onto latest and getting it merged? It looks safe and simple to me, and the

submitter is still responsive. He just had his ECA filed for us, so we are good

to proceed from a legal standpoint...

Cheers,
Martin

On Mon, Mar 5, 2018 at 2:36 AM, Nathan Ridge <zeratul976@xxxxxxxxxxx> wrote:

>> We can write automated tests for refactorings if we know what
>> the code should look like after the refactoring.
>
> Take a look at the results/diffs folder.  This contains diffs for the
> expected results of the refactoring.

Thanks; I filed bug 531987 [1] to track adding these test cases
to CDT's refactoring test suite.

I'm unlikely to have time to work on this in the near future.
As always, contributions are welcome!

Regards,
Nate

[1] https://bugs.eclipse.org/bugs/show_bug.cgi?id=531987

_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cdt-dev