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-2 - Create CDI 2.0 integration tests - step 2 #1949
Conversation
|
||
@Before | ||
public void changeDiscoveryMode(){ | ||
validationProvider = new BeansXmlValidationProviderCDI11(); |
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.
Are you sure that this class contains all that is needed/changed/new in cdi 2.0 specs? Aren't they any different?
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.
@zcervink Do you dare to address the required changes?
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.
|
||
@Before | ||
public void setValidationProvider(){ | ||
validationProvider = new BeansXmlValidationProviderCDI11(); |
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.
Same question as above. Probably same answer ;)
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
f5b8253
to
ea58be8
Compare
testPR |
|
||
@Before | ||
public void changeDiscoveryMode(){ | ||
validationProvider = new BeansXmlValidationProviderCDI11(); |
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.
@zcervink Do you dare to address the required changes?
Good. Let's wait for the build to finish. |
@zcervink Tests are still failing. |
Current Jenkins build: https://dev-platform-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/cdi20.itests/58/ |
+1. @jkopriva Merge this please. |
BeansXMLBeansEditorTestCDI20.class BeansXMLValidationTestCDI20.class BeansXMLValidationQuickFixTestCDI20.class Signed-off-by: Zbynek Cervinka <zcervink@redhat.com>
Merging, we should create JIRA for duplicated strings and fix it. |
JBIDE-24963-2 - Create CDI 2.0 integration tests - step 2; following testclasses are added:
- BeansXMLBeansEditorTestCDI20.class
- BeansXMLValidationTestCDI20.class
- BeansXMLValidationQuickFixTestCDI20.class
Signed-off-by: Zbynek Cervinka zcervink@redhat.com
@odockal please review
Jira: https://issues.jboss.org/browse/JBIDE-24963
Jenkins: https://dev-platform-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/cdi20.itests/16/