Couchbase .NET client library

Restructure .NET tests

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

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 :)

Issue Links

depends on

Bug - A problem which impairs or prevents the functions of the product. NCBC-177 Consolidate test options from App.config

  • Major - Major loss of function.
  • Resolved - A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Saakshi Manocha added a comment -
http://review.couchbase.org/#/c/23202/
Show
Saakshi Manocha added a comment - http://review.couchbase.org/#/c/23202/
Hide
Permalink
Mark Nunberg added a comment -
I've sectioned out the previous bug into an NCBC issue. In general any changes to the SDKs themselves should be branched out into issues which touch those specific projects, with the SDKQE task tracking them.
Show
Mark Nunberg added a comment - I've sectioned out the previous bug into an NCBC issue. In general any changes to the SDKs themselves should be branched out into issues which touch those specific projects, with the SDKQE task tracking them.
Hide
Permalink
Saakshi Manocha added a comment - - edited
updated files attached for review :
couchbase.tests.changedfiles.zip
Show
Saakshi Manocha added a comment - - edited updated files attached for review : couchbase.tests.changedfiles.zip
Hide
Permalink
Saakshi Manocha added a comment -
please review:
http://review.couchbase.org/#/c/23430/
Show
Saakshi Manocha added a comment - please review: http://review.couchbase.org/#/c/23430/
Hide
Permalink
Saakshi Manocha added a comment -
Mark to review the changes updated in Gerrit
Show
Saakshi Manocha added a comment - Mark to review the changes updated in Gerrit
Hide
Permalink
Saakshi Manocha added a comment -
moved this task from SDKQE project to NCBC, as per mail from Matt so that we can use NCBC in comment while submitting code in gerrit.
updated comment in gerrit and submitted a new patch set.,
please review http://review.couchbase.org/#/c/23604/
Show
Saakshi Manocha added a comment - moved this task from SDKQE project to NCBC, as per mail from Matt so that we can use NCBC in comment while submitting code in gerrit. updated comment in gerrit and submitted a new patch set., please review http://review.couchbase.org/#/c/23604/
Hide
Permalink
Matt Ingenthron added a comment -
Saakshi: it's not so much that this task needed to be moved, but an underlying NCBC enhancement issue needed to be created-- even one with just a one line summary of what's to be done.

NCBC tracks this product/project, so we should always be able to see the changes in a version via this issue tracker and cross reference that with the code changes.

Thanks!
Show
Matt Ingenthron added a comment - Saakshi: it's not so much that this task needed to be moved, but an underlying NCBC enhancement issue needed to be created-- even one with just a one line summary of what's to be done. NCBC tracks this product/project, so we should always be able to see the changes in a version via this issue tracker and cross reference that with the code changes. Thanks!

People

  • Assignee:
    Saakshi Manocha
    Reporter:
    Mark Nunberg
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: