ValidatorTester and AssertableDiagnostics deprecated
LorenzoBettini opened this issue · comments
https://github.com/eclipse/xtext/blob/652c2c41499387ab87737143d12939a8bdab41ab/org.eclipse.xtext.junit4/src/org/eclipse/xtext/junit4/validation/AssertableDiagnostics.java and https://github.com/eclipse/xtext/blob/652c2c41499387ab87737143d12939a8bdab41ab/org.eclipse.xtext.junit4/src/org/eclipse/xtext/junit4/validation/ValidatorTester.java were deprecated in 2.30 as part as the old org.eclipse.xtext.junit4 bundle.
In org.eclipse.xtext.testing we now have
https://github.com/eclipse/xtext/blob/652c2c41499387ab87737143d12939a8bdab41ab/org.eclipse.xtext.testing/src/org/eclipse/xtext/testing/validation/AssertableDiagnostics.java but no corresponding ValidatorTester, which was the only one creating objects of AssertableDiagnostics.
We still use ValidatorTester in our own tests (we copied it there):
Why don't we put the latter in org.eclipse.xtext.testing?
It is not straightforward to port existing tests using ValidatorTester to ValidationTestHelper.
Please add a convenient substitute for the deprecated ValidationTester.
I was suggesting to use the one we already use in our tests
I mean: put that one in the org.eclipse.xtext.testing. I seem to understand that it is exactly the same as the deprecated one.
Yes, please.