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 fixed with MongoDB 3.6 #234

Merged
merged 4 commits into from
Jul 15, 2018

Conversation

IlyaSkriblovsky
Copy link
Contributor

@IlyaSkriblovsky IlyaSkriblovsky commented Jul 6, 2018

Fixing some test failures when running against MongoDB 3.6. TxMongo works fine with 3.6, this issue only affects testing code that hardcodes some error message texts and other internal MongoDB stuff from 3.4. No user-visible changes here.

This PR is based on #233 and includes its commits, so it should be merged only after that.

@coveralls
Copy link

coveralls commented Jul 6, 2018

Coverage Status

Coverage decreased (-0.8%) to 94.425% when pulling 7de99f0 on IlyaSkriblovsky:mongodb-3.6-compatibility into 2074262 on twisted:master.

@@ -68,7 +68,6 @@ def start(self):
b"--noprealloc", b"--nojournal",
b"--smallfiles", b"--nssize", b"1",
b"--oplogSize", b"1",
b"--nohttpinterface",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this! I had to do this for an internal project too. :)

@IlyaSkriblovsky
Copy link
Contributor Author

IlyaSkriblovsky commented Jul 14, 2018

Can we merge this?

@psi29a psi29a merged commit 5a2792f into twisted:master Jul 15, 2018
@psi29a
Copy link
Contributor

psi29a commented Jul 15, 2018

Done and done.

@IlyaSkriblovsky IlyaSkriblovsky deleted the mongodb-3.6-compatibility branch July 19, 2018 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants