Remove code duplication from description.NewServer and IsMaster.Result

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Major - P3
    • 1.4.0
    • Affects Version/s: None
    • Component/s: Core API
    • None
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      The description.NewServer function currently takes a result.IsMaster, but that adds an extra step of decoding indirection. Instead, take the code from operation.Result and use it to update the description.NewServer method.

      The description.NewServer function is only being used in the SDAM spec tests and operation.IsMaster.Result duplicates the code to create a new server description. We should change this so IsMaster.Result uses this function and remove the duplication.

       

            Assignee:
            Isabella Siu (Inactive)
            Reporter:
            Kristofer Brandow (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: