jsontests.cpp is hard to read and maintain

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Minor - P4
    • 4.3.1
    • Affects Version/s: 4.3 Desired
    • Component/s: Internal Code
    • None
    • Fully Compatible
    • Dev Tools 2019-06-17
    • 0
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      jsontests.cpp uses inheritance to parse each string that it wants to test.

      This is really wasteful and makes it difficult to see the differences among the tests in a group. The tests don't share common code. The file is about 3000 lines, and doesn't need to be.

      I'd like to see us make some progress toward phasing out this kind of boilerplate-dominated testing.

      The test for SERVER-8544 was too hard to add.

            Assignee:
            Billy Donahue
            Reporter:
            Billy Donahue
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: