-
Notifications
You must be signed in to change notification settings - Fork 1.3k
CSHARP-5592: Separate unit tests into dedicated variants #1695
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
Conversation
@@ -18,10 +18,12 @@ | |||
using MongoDB.Bson; | |||
using MongoDB.Driver.Linq.Linq3Implementation; | |||
using MongoDB.Driver.Linq.Linq3Implementation.Translators.ExpressionToExecutableQueryTranslators; | |||
using MongoDB.TestHelpers.XunitExtensions; |
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.
Why was this added?
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 also in some other files
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.
Let me check. There were several iteration of switching in between TraitAttribute
and custom attribute. It might be a leftovers.
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.
Removed. Thanks!
using MongoDB.Driver.Linq.Linq3Implementation; | ||
using MongoDB.Driver.Linq.Linq3Implementation.Translators.ExpressionToExecutableQueryTranslators; | ||
using Xunit; | ||
|
||
using MongoDB.TestHelpers.XunitExtensions;using Xunit; |
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.
They should not be on the same line. (And maybe the first using is not necessary?)
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.
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.
Overall it looks good, just a small question.
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.
LGTM
In this PR I've marked all tests that require MongoDB connection with Category=Integration. Then I've adjusted cake to run ONLY Integration tests and created a new variants to run unit tests (tests that do not belong to Integration category).