Issue #854: Fix conversion error when Handle clause property differs by case from actual property

[jrmoreno1]

Link to issue(s) this covers
#854

Problem

When the property declaration and the property reference in the Handles clause, differ by case, the conversion fails trying to create two event containers.

Solution

• Pass the semantic model to CreateEventContainer so it can be used to get the name from the property declaration
• Change is relatively minor, all unit test continue to pass
• Added new unit test, unit test fails without change, passes after change.
• Copied "hasLineCommentConversionIssue: true /Fields re-ordered/);" from another test, not sure what that actually means.

[GrahamTheCoder]

Looks good, thanks for that!

I've removed hasLineCommentConversionIssue: true /Fields re-ordered/);. In fact I've removed it from all but one of the few other places it was used in favour of being more specific about which comments move about. I've added a bit more explanation in a doc comment and made the test fail when it's incorrectly used:

By default tests run a second time with a numbered comment added to each line (that doesn't already have a comment) and checks the comments come out in the same order. If the order significantly changes, or there are input lines where a line comment is invalid (e.g. multiline xml literal) you can use incompatibleWithAutomatedCommentTesting to skip the check