Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-10022

dev guide should say that tests of C++ should be written in python

    XMLWordPrintable

    Details

    • Story Points:
      0.5

      Description

      Our unittest policy does not describe how to balance tests between python and C+, for code that is mostly or entirely C+ (with pybind11 wrappers). After this conversation on Slack, it was clear that the consensus is that tests of our C++ layer should be written almost entirely in python, with C++ tests only for purely internal C++ APIs or tricky things in the pybind11 layer (both of which should be rare).

      Filing as a ticket instead of an RFC, given the clear consensus in the Slack conversation. I can make an RFC if someone does object.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            We probably still need this.

            Show
            tjenness Tim Jenness added a comment - We probably still need this.
            Hide
            tjenness Tim Jenness added a comment -

            Looks okay.

            Show
            tjenness Tim Jenness added a comment - Looks okay.

              People

              Assignee:
              ktl Kian-Tat Lim
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Tim Jenness
              Watchers:
              Jim Bosch, John Parejko, Kian-Tat Lim, Krzysztof Findeisen, Pim Schellart [X] (Inactive), Robert Lupton, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.