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
JBIDE-24963-5 - add new testclasses for CDI 2.0 #1962
Conversation
320d68f
to
e1003b6
Compare
import org.eclipse.reddeer.eclipse.ui.views.markers.ProblemsView.ProblemType; | ||
import org.jboss.tools.cdi.reddeer.annotation.ValidationType; | ||
|
||
public class BeansXmlUIValidationProviderCDI20 extends AbstractValidationProvider { |
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.
Isn't this file the exact same as for CDI11 except JSR number? Wouldn't it be easier to just have one class definition with problems definition and abstract method that would return proper JSR number in children classes?
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.
Fixed, now testing on Jenkins.
974338a
to
e71e4d5
Compare
private final String jsr = "JSR-346"; | ||
|
||
public BeansXmlUIValidationProviderCDI11() { | ||
private static String jsr; |
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 would suggest not to use static here. you are assigning it in constructor.
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.
Fixed.
b8dd6c7
to
d569b51
Compare
@jkopriva giving +1, merge this please. |
@zcervink please add javadoc with author in new clasess, otherwise we can merge. |
- BeansXMLAsYouTypeValidationTestCDI20.class - BeansXMLOpenOnTestCDI20.class - BeansXMLUITestCDI20.class Signed-off-by: Zbynek Cervinka <zcervink@redhat.com>
testPR |
@zcervink thanks for contribution! Merging. |
JBIDE-24963-5 - add new testclasses for CDI 2.0:
Signed-off-by: Zbynek Cervinka zcervink@redhat.com
Jenkins: https://dev-platform-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/cdi20.itests/98/
Jira: https://issues.jboss.org/browse/JBIDE-24963