Skip to content
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

Remove of deprecated .xtext.junit4 and .xbase.junit4 #3026

Open
LorenzoBettini opened this issue May 2, 2024 · 9 comments
Open

Remove of deprecated .xtext.junit4 and .xbase.junit4 #3026

LorenzoBettini opened this issue May 2, 2024 · 9 comments

Comments

@LorenzoBettini
Copy link
Contributor

I propose to ditch for good those deprecated projects, whose mechanisms are provided by the corresponding split projects .testing and .ui.testing.

The only thing to address is the AbstractXtextTests and ResourceLoadHelper, which don't have a counterpart in the new projects. We still use them a lot, by duplicating them, in several test projects in Xtext. We could re-introduce them in the new testing projects before the proposed removal of the deprecated ones. This can be achieved in a separate PR.

The removal will allow us to also get rid of the corresponding .tests projects.

@cdietrich
Copy link
Member

should we add another reminder to release notes?
is all marked forRemoval?

@cdietrich
Copy link
Member

cdietrich commented May 3, 2024

isnt ResourceLoadHelper unused ? (the deprecated one)

@LorenzoBettini
Copy link
Contributor Author

@cdietrich I'll check for the forRemoval later.
ResourceLoadHelper seems to be just an interface...

What I still have to understand is whether the AbstractXtextTests copied in most of our test projects is always the same.

@cdietrich
Copy link
Member

i assume yes

@LorenzoBettini
Copy link
Contributor Author

@cdietrich indeed forRemoval seems everywhere on those older projects

@LorenzoBettini
Copy link
Contributor Author

By the way, we already have org.eclipse.xtext.ui.testing.util.ResourceLoadHelper in ui.testing, so probably that's the right place to be. In any case, we can't remove that one: it's in the new project and it's used in some of content assist base test classes.

In the new AbstractXtextTests base class, I won't make it implement that interface, though, since it's not used in that context

@LorenzoBettini
Copy link
Contributor Author

@cdietrich With #3038 merged, should we go on removing the old projects for 2.35?

@cdietrich
Copy link
Member

cdietrich commented May 16, 2024

I would Annonce in rel notes and remove in 2.36

@szarnekow
Copy link
Contributor

I would Annonce in rel notes and remove in 2.36

I'd do the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants