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

Created JUnit test for HibernateFormDAO.getFormCriteria's containingAnyF... #631

Closed
wants to merge 1 commit into from

Conversation

sashrika
Copy link
Contributor

Created JUnit test for HibernateFormDAO.getFormCriteria's containingAnyFormField parameter
https://tickets.openmrs.org/browse/TRUNK-3195

@dkayiwa
Copy link
Member

dkayiwa commented Feb 12, 2014

@sashrika does mvn clean install work without errors on your side?

@dkayiwa dkayiwa closed this Feb 12, 2014
@sashrika
Copy link
Contributor Author

@dkayiwa, When I removed this two test cases, mvn clean install work without errors on my side(that means without any modification to the original cloned copy).
But as I mentioned in https://tickets.openmrs.org/browse/TRUNK-3195, my tests failed after I added them. Since they fail, mvn clean install doesn't work.

I doubt whether https://tickets.openmrs.org/browse/TRUNK-4189 is fixed correctly. Any ideas?

@sashrika
Copy link
Contributor Author

@dkayiwa I checked https://travis-ci.org/openmrs/openmrs-core/jobs/18668823, the reason for the build failure is my junit test.

@dkayiwa dkayiwa reopened this Feb 14, 2014
@dkayiwa
Copy link
Member

dkayiwa commented Feb 17, 2014

@sashrika you are right about this. I have made a follow up fix for TRUNK-4189
So pull and try again. :)

@dkayiwa dkayiwa closed this Feb 17, 2014
@sashrika
Copy link
Contributor Author

@dkayiwa Great :) . I'll try again

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