Projects:ARRA:VTKWidgetsTesting

Preliminary Steps

Working from the Slicer3 VTK56 git repository.

First goal is to set up a similar infrastructure as used in Slicer3/Libs/MRML/Testing, which includes a file that defines basic testing macros (Slicer3/TestingMacros.h) that calls methods on super classes and chains macros together as you get down to testing subclasses. The file

 VTK/Widgets/Testing/Cxx/WidgetTestingMacros.h

is set up to define:

  EXERCISE_BASIC_OBJECT_METHODS (copied from Slicer, thanks to Luis Ibanez)
  TEST_SET_GET for various variable types (INT, BOOLEAN, STRING, DOUBLE, VECTOR3, CHAR)
  EXERCISE_BASIC_INTERACTOR_OBSERVER_METHODS for the superclass (could be moved/copied to VTK/Rendering/Testing, no tests yet)
  EXERCISE_BASIC_ABSTRACT_METHODS
  EXERCISE_BASIC_BORDER_METHODS to test widgets that inherit from vtkBorderWidget (Text)
  EXERCISE_BASIC_HOVER_METHOD to test widgets that inherit from vtkHoverWidget (Balloon)
  EXERCISE_BASIC_PROP_METHODS to test representations that inherit from the vtkProp superclass
  EXERCISE_BASIC_REPRESENTATION_METHODS to test representations that inherit from the vtkWidgetRepresentation superclass
  EXERCISE_BASIC_ANGLE_REPRESENTATION_METHODS to test representations that inherit from the vtkAngleRepresentation superclass
  EXERCISE_BASIC_BORDER_REPRESENTATION_METHODS to test representations that inherit from the vtkBorderRepresentation

Then new testing .cxx files were crafted and added to the CMakeLists.txt to use these basic macros to test every call.

WORK IN PROGRESS

Widgets

Notes made while writing the Widget tests: (priorities taken from here: Priority Widgets, indentation shows inheritance)

• vtkInteractorObserver defines the Enabled variable and the On/Off methods. Testing them in isolation shows that there are missing checks for having an interactor defined, as the test crashes consistently even after creating a render window and interactor to pass to the widget
  • vtkAbstractWidget Set/Get Parent doesn't work in stand alone tests
    • vtkAngleWidget
      • Would like GetWidgetState macro, GetCurrentHandle macro.
      • Access to HandleWidgets?
    • vtkBidimensionalWidget
      • Would like Get macros for WidgetState, CurrentHandle, HandleLine1Selected, HandleLine2Selected, Line1InnerSelected, Line1OuterSelected, Line2InnerSelected, Line2OuterSelected, CenterSelected.
      • Access to HandleWidgets?
    • vtkImplicitPlaneWidget2 Would like Get macro for WidgetState
    • vtkLineWidget2
      • Would like GetWidgetState macro, GetCurrentHandle macro
      • Access to HandleWidgets?
    • vtkSeedWidget
      • Would like Get macros for WidgetState, Defining.
      • Calling CreateNewHandle crashes if HandleRepresentation is null, need to test for that in *vtkSeedRepresentation::GetHandleRepresentation and in vtkSeedWidget::CreateNewHandle. Test currently failing, suggest that a default handle representation is created.
    • vtkSplineWidget2 Would like Get macro for WidgetState
  • vtkBorderWidget Access to SetCursor?
    • vtkCaptionWidget
      • Access to HandleWidget?
      • Only uses text actor 2d
    • vtkTextWidget
  • vtkHoverWidget Would like GetWidgetState macro
    • vtkBalloonWidget
      • Would like SetBalloonString
      • Failing the basic interactor observer tests
      • seems to be failing AddBalloon(prop1, cstr) after did a few other kinds of add balloon calls, the text string doesn't match when get it
      • Problem with changing text on this widget, doesn't update the representation

General notes:

• when writing the macro for GetWidgetState, need to move the enum of the states to be public so can check for them.
• would be very very very useful to have GetXRepresentation defined on the subclasses to return the expected subclass of vtkWidgetRepresentation so that don't have to safe down cast the return values to get at subclass methods.

Representations

Testing the representations:

• vtkProp
  • vtkWidgetRepresentation
    • no Get macros for Placed, ValidPick
    • declares ComputeInteractionState, clash with Widget
    • vtkAngleRepresentation
      • BuildRepresenation doesn't check for null handle pointers
      • SetXDisplayPosition doesn't have versions with three doubles
      • GetXDisplayPosition doesn't have a version that returns a double*
      • SetHandleRepresentation should be higher up in the .h file as mostly nothing works w/o it being called
      • vtkAngleRepresentation2D
        • BuildRepresentation doesn't check for null handle pointers.
        • GetPoint1WorldPosition, GetCenterWorldPosition, GetPoint2WorldPosition, SetPoint1DisplayPosition, SetCenterDisplayPosition, SetPoint2DisplayPosition, GetPoint1DisplayPosition, GetCenterDisplayPosition, GetPoint2DisplayPosition don't check for null handle pointers
        • HandleSize test fails with the lower clamp bound
      • vtkAngleRepresentation3D
        • TextActorScale is a convenience method that could use set macro that takes x,y,z
        • Also many failures to check for null handle representations
        • Code I'm testing doesn't have set/get color macros for the lines, arcs, text, have to Get the actors and then get properties. Convenience get/set methods would be good.
    • vtkBalloonRepresentation
      • why no HoverRepresentation as a super class?
      • Would like Get macro for TextActor (necessary?)
    • vtkBiDimensionalRepresentation2D
      • missing 3D representation
      • the Get/Set Point 1-4 Display and World positions could use methods with signatures of x,y,z
      • SetLabelText?
      • No Get macros for 4 Point handle representations
      • No Set/Get for CenterWorld
      • Missing some null pointer checks on handle representations
      • crashes in Point3WidgetInteraction, GetWorldLabelPosition - renderer not set up?
      • Label text not updating?
    • vtkBorderRepresentation
      • no Get macro for interaction state
      • defines ComputeInteractionState, may clash with the Widget
      • no Set macro for size
      • uses vtkCoordinate and macro for Position and Position2, missing Get(pos[2]) macro
      • vtkCaptionRepresentation
        • no Get macro for PointWidgetState
        • no Get macro for actor2d's properties
      • vtkTextRepresentation
        • no Get macro for TextProperty, but does have a macro for GetTextActor
    • vtkImplicitPlaneRepresentation
      • does it match to vtkImplicitPlaneWidget2?
      • no GetPolyData() to return vtkPolyData, ditto GetPlane (but GetPolyDataAlgorithm returns a pointer)
      • has all the GetProperty macros (no set)
      • What are valid values for RepresentationState and how does it clash with InteractionState?
      • No EdgesSelected defined
      • testing fails in ComputeInteraction state with a seg fault if the renderer doesn't have a render window (VTK/Rendering/vtkPicker.cxx:235 doesn't check for null pointer, and in Widgets/vtkWidgetRepresentation.cxx:105 if put in the check, basically BuildRepresentation is fragile)
    • vtkLineRepresentation
      • does it match to LineWidget2?
      • could use SetPointNWorld/DisplayPosition(double, double, double)
      • setting/getting display positions causes a seg fault in vtkRenderer::GetActiveCamera
      • What are valid values for RepresentationState and how does it clash with InteractionState?
      • request ticks on line to show resolution
      • SetDistanceAnnotationScale needs Set(double, double, double) (added)
    • vtkSeedRepresentation
      • defines ComputeInteractionState (NearSeed, Outside), clash with the Widget since one is calculated in screen space and one in world space)
      • might be useful to have IsHandlePresent (GetHandleRepresentation creates one if it doesn't exist).
      • when GetHandleRepresentation, have to safe down cast returns from vtkHandleRepresentation, some utility methods to GetPointHandleRepresentation, etc would be useful
      • need more docs on CreateHandle, e is display position
      • no SetSeedWorldPosition?
      • if don't set handle representation before creating new handle, seg fault
    • vtkSplineRepresentation
      • defines ComputeInteractionState, clash with the Widget
      • no access to individual handles to highlight or change colors
      • no Get macro for Centroid
      • no access to MovePoint, Scale, Translate
      • seg faults when call SetProjectionPosition, SetNumberOfHandles, etc, build representation triggers an get active camera call