Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: library
    • 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

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

          Show
          saakshi.manocha Saakshi Manocha added a comment - http://review.couchbase.org/#/c/23430/
          Hide
          saakshi.manocha Saakshi Manocha added a comment -

          updated files attached here for review

          Show
          saakshi.manocha Saakshi Manocha added a comment - updated files attached here for review
          Hide
          mnunberg Mark Nunberg added a comment -

          [saakshi]

          I copied this bug over as a NCBC issue

          http://review.couchbase.org/#/c/23202/

          Show
          mnunberg Mark Nunberg added a comment - [saakshi] I copied this bug over as a NCBC issue http://review.couchbase.org/#/c/23202/

            People

            • Assignee:
              saakshi.manocha Saakshi Manocha
              Reporter:
              mnunberg Mark Nunberg
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Gerrit Reviews

                There are no open Gerrit changes