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

Add cross-language GenericMap unit tests

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Templates:
    • Story Points:
      4
    • Epic Link:
    • Sprint:
      AP F19-4 (2H September)
    • Team:
      Alert Production

      Description

      GenericMap has unit tests in both C++ and Python, but they do not actually pass data between the two languages. It would be helpful to verify that a Python object can be correctly retrieved (as a Storable) in C++, and that objects added from C++ can be modified from Python.

      However, I don't see a way to write such tests without pybind11-wrapping test functions written in C++, so some caution should be taken to ensure the wrappers don't pollute the Python API.

        Attachments

          Issue Links

          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-17981 Create heterogenous map

          • Done
          relates to

          Bug - A problem which impairs or prevents the functions of the product. DM-21216 GenericMap converts C-strings to bool

          • To Do

          Bug - A problem which impairs or prevents the functions of the product. DM-21268 GenericMap can't distinguish int/float sizes in Python

          • To Do

          Bug - A problem which impairs or prevents the functions of the product. DM-21314 Investigate GC problems with Storable

          • Done

          Improvement - An improvement or enhancement to an existing feature or task. DM-21382 Separate GenericMap's visitor API from its implementation

          • To Do

            Activity

              People

              • Assignee:
                krzys Krzysztof Findeisen
                Reporter:
                krzys Krzysztof Findeisen
                Reviewers:
                Jim Bosch
                Watchers:
                Jim Bosch, Krzysztof Findeisen
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel