[JAVA-1498] Testing Aggregation using the new API Created: 09/Oct/14 Updated: 06/May/15 Resolved: 06/May/15 |
|
| Status: | Closed |
| Project: | Java Driver |
| Component/s: | Test Coverage |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Trisha Gee | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Epic Link: | Test Coverage |
| Description |
|
Currently, the only thing that tests aggregation via the new API is CollectionAcceptance test. We should:
In addition, there was an Ignored test in JavaClientOldTest: public void testAggregateOnSecondary() As it was ignored, this was not being run. But it's not clear why it was ignored or what it was trying to test. If this is an important test, we need to implement it using the new API. |
| Comments |
| Comment by Jeffrey Yemin [ 06/May/15 ] |
|
MongoCollection is now well covered with unit tests. |
| Comment by Jeffrey Yemin [ 09/Oct/14 ] |
|
But this ticket is about the legacy API, which doesn't look well covered either. We need to test both old and new. |
| Comment by Jeffrey Yemin [ 09/Oct/14 ] |
|
There is also com.mongodb.MongoCollectionSpecification#aggregate should use AggregationOperation properly We should add tests of all the overloads in that specification. |