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

Tests are failing #127

Open
VaclavDedik opened this issue Aug 18, 2016 · 3 comments
Open

Tests are failing #127

VaclavDedik opened this issue Aug 18, 2016 · 3 comments
Assignees
Labels

Comments

@VaclavDedik
Copy link
Member

See e.g.:

https://travis-ci.org/redhatcz/Thesis-management-system/builds/153227439

Not sure why it worked as PRs, and now it doesn't :/

@VaclavDedik
Copy link
Member Author

Looks like reverting #55 fixed it. There is still an error in the log though, so maybe it would be a good idea to have a look at it:

https://travis-ci.org/redhatcz/Thesis-management-system/builds/153238281

@pehala pehala added bug and removed blocker labels Sep 5, 2016
@pehala
Copy link
Contributor

pehala commented Sep 5, 2016

NotificationIntegrationSpec and ApplicationIntegrationSpec are colliding for no reason whatsoever. I found out that tests won't save new Feed and connection to Mongo would just time out. I tried to get error, but was unable to do so and only thing I got was unspecified timeout. I tried looking into Mongo logs but without luck, no sign of any attempt to save anything and no error. I also made sure that both tests remove whole testing DB, so each test has clean environment. I think this might be bug in MongoDB driver, because when I switched Feed from Mongo to Postgre tests were successful. For now I removed NotificationIntegrationSpec and will resolve this after we migrate all things to Postegre.

@VaclavDedik
Copy link
Member Author

Mongo is rubbish, we have to get rid of it at some point :)

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

No branches or pull requests

2 participants