Purpose

DYN-1710

This PR adds testing coverage to another open PR DynamoDS#9572 which has not yet been merged. I have already reviewed @RyaPorter 's work which looks good. Once this PR is reviewed it can be merged into DynamoDS#9572 followed by DynamoDS#9572 being merged into master.

Testing

This functionality is currently partially covered in HelixWatch3DViewModelTests.cs which are VisualizationTests. Unfortunately, these don't yet run in the automated pipeline so I modified the logic into a unit test that will be run in the pipeline. These tests confirm the appropriate command methods are called for each lacing operation. The final test verifies all 4 lacing options appear in the context menu.

Declarations

Check these if you believe they are true

• The code base is in a better state after this PR
• Is documented according to the standards
• The level of testing this PR includes is appropriate
• User facing strings, if any, are extracted into *.resx files
• All tests pass using the self-service CI.
• Snapshot of UI changes, if any.
• Changes to the API follow Semantic Versioning, and are documented in the API Changes document.

Reviewers

@RyaPorter

FYIs

@QilongTang @mjkkirschner