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

Missing test case for SpherePoint

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Story Points:
      2
    • Epic Link:
    • Sprint:
      Alert Production S17 - 1, Alert Production S17 - 2
    • Team:
      Alert Production

      Description

      One of the changes introduced during review of DM-5529 was a mismatch between the behavior of indexing in Python and C++ – negative indices are allowed in the former but not the latter, consistent with either language's conventions. However, indexing is tested only in the Python test suite against the Python specification.

      A test case should be added to testSpherePoint.cc to verify that operator[] behaves as specified when called directly.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          krzys Krzysztof Findeisen added a comment -

          Hi Colin Slater, would you be willing to do a quick review of this ticket (the new test case is pretty trivial)?

          Show
          krzys Krzysztof Findeisen added a comment - Hi Colin Slater , would you be willing to do a quick review of this ticket (the new test case is pretty trivial)?
          Hide
          Permalink
          ctslater Colin Slater added a comment -

          Looks good; no comments.

          Show
          ctslater Colin Slater added a comment - Looks good; no comments.

            People

            Assignee:
            krzys Krzysztof Findeisen
            Reporter:
            krzys Krzysztof Findeisen
            Reviewers:
            Colin Slater
            Watchers:
            Colin Slater, Krzysztof Findeisen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                CI Builds

                No builds found.