Re: [cdt-dev] New DSF test structure

[Marc Khouzam <marc.khouzam@xxxxxxxxxxxx>]

I want to thank Elena for her recent efforts to cleanup the debug test suite.
We had a lot of classes simply to have different names for different gdb versions.
She used her JUnit expertise to clean that up and remove 89% of the test classes,
while keeping the same number of actual tests!

Not only will it bec easier to maintain, but it makes it much easier when a new GDB version
comes out.

I'll let her explain the new way to launch the tests, to make sure I also fully understand :)

Nice job Elena!

Marc

---

From: cdt-dev-bounces@xxxxxxxxxxx [cdt-dev-bounces@xxxxxxxxxxx] on behalf of Alena Laskavaia [elaskavaia.cdt@xxxxxxxxx]
Sent: March 17, 2016 10:42 AM
To: CDT General developers list.
Subject: Re: [cdt-dev] New DSF test structure

I merged the SourceLookup test changes already in my next commit.

Its important to make writing junits easy. Once I am done refactoring I will write some info on how to use it from now on on dev list.

On Wed, Mar 16, 2016 at 7:04 PM, Jonah Graham <jonah@xxxxxxxxxxxxxxxx> wrote:

Hi Alena,

The new test structure [1] looks great. I will update the
SourceLookupTest once my big outstanding gerrits that touch them them
are merged.

I think it is great how with this structure I can easily run one test
on all GDB versions as a single JUnit run without having to create a
manual suite as I have been doing up until now [2]. It also looks
great in the JUnit UI.

Thanks!
Jonah


[1] https://mattermost-test.eclipse.org/eclipse/pl/1yphynpzn7g4upk4sbr5kixykh
and https://git.eclipse.org/r/#/c/68371/

[2] https://git.eclipse.org/r/#/c/52068/1/dsf-gdb/org.eclipse.cdt.tests.dsf.gdb/src/org/eclipse/cdt/tests/dsf/gdb/tests/SourceLookupSuite.java
- this file is not part of the review anymore, but only public link I
had to it.

~~~
Jonah Graham
Kichwa Coders Ltd.
www.kichwacoders.com
_______________________________________________
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