Details
-
Type:
Improvement
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: afw
-
Labels:
-
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.
Hi Colin Slater, would you be willing to do a quick review of this ticket (the new test case is pretty trivial)?