Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-2337

Remove ClusterContext requirement from service clients

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Major
    • 3.0.0
    • None
    • library
    • None
    • 1

    Description

      The only reason we inject ClusterContext into the various service HTTP clients is to get the service URI. This is overcomplicating a lot of unit tests, as they are forced to build a large portion of the ClusterContext's backing configuration to support this. Instead, we can create IServiceUriProvider to provide random service URIs, and inject this. Then it is much more easily mockable in unit tests.

      Attachments

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

        Activity

          People

            btburnett3 Brant Burnett
            btburnett3 Brant Burnett
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty