-
Notifications
You must be signed in to change notification settings - Fork 317
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add AbstractXtextTests
in .xtext.testing
#3038
Conversation
Copied from the one in .xtext.tests
there are no Xtend tests in this project anymore
org.eclipse.xtext.testing/src/org/eclipse/xtext/testing/AbstractXtextTests.java
Show resolved
Hide resolved
@cdietrich so do you want me to mark it as beta or is the PR ok? |
i want to be safe and thus mark it as beta |
@cdietrich done: d10ede9 |
*/ | ||
public abstract class AbstractXtextTests extends Assert implements ResourceLoadHelper { | ||
@Beta |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd appreciate a comment that points to the "newer" way of writing tests based on @RunWith
and friends.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@szarnekow Right! To think I also thought about that and then forgot it ;)
Done that here f6a299a
I also took the chance to clean a few other bits.
This is a preparation for #3026.
AbstractXtextTests
is now in.xtext.testing
.It doesn't implement
ResourceLoadHelper
anymore: it wasn't used in a headless context. In fact,ResourceLoadHelper
had already been inserted in.xtext.ui.testing
.The current
AbstractXtextTests
has been taken from duplicate occurrences of our test projects.In some cases, the
implements ResourceLoadHelper
had to be manually added for some of our older content assist tests. But that wasn't a big deal.