couchbase.tests.changedfiles.zip
|
Dependency |
|||
| blocks |
|
Restructure .NET tests | Resolved |
[NCBC-177] Consolidate test options from App.config Created: 11/Dec/12 Updated: 19/Dec/12 Resolved: 18/Dec/12 |
|
| Status: | Resolved |
| Project: | Couchbase .NET client library |
| Component/s: | library |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major |
| Reporter: | Mark Nunberg | Assignee: | Saakshi Manocha |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
couchbase.tests.changedfiles.zip
|
||||||||
| Issue Links: |
|
||||||||
| Description |
|
Eliminate redundant tests. I see there are some repetitive tests which differ only by 1-2 parameters. For example, we don't need individual functions for each scenario of observe, and it should be possible to abstract this all into a single function, with different tests just calling into that function with different parameters.
I recall that you brought this up as a concern before; now it's a task :) |
| Comments |
| Comment by Mark Nunberg [ 11/Dec/12 ] |
|
[saakshi]
I copied this bug over as a NCBC issue :) http://review.couchbase.org/#/c/23202/ |
| Comment by Saakshi Manocha [ 18/Dec/12 ] |
| updated files attached here for review |
| Comment by Saakshi Manocha [ 19/Dec/12 ] |
| http://review.couchbase.org/#/c/23430/ |