TextBoxMask TextChanging fix for #3279

[puppetsw]

Fixes #3279

Applies the fix suggest by oenarap in issue #3279 .

PR Type

What kind of change does this PR introduce?

• Bugfix

What is the current behavior?

TextBoxMask leaves an artifact (i.e., old Text value) when a new value is set. (Through binding)

What is the new behavior?

PR Checklist

Please check if your PR fulfills the following requirements:

• Tested code with current supported SDKs
• Pull Request has been submitted to the documentation repository instructions. Link:
• Sample in sample app has been added / updated (for bug fixes / features)
  • Icon has been created (if new sample) following the Thumbnail Style Guide and templates
• New major technical changes in the toolkit have or will be added to the Wiki e.g. build changes, source generators, testing infrastructure, sample creation changes, etc...
• Tests for the changes have been added (for bug fixes / features) (if applicable)
• Header has been added to all new source files (run build/UpdateHeaders.bat)
• Contains NO breaking changes

Other information

First attempt at a PR and fix. I hope I'm doing this right.

[ghost]

Thanks puppetsw for opening a Pull Request! The reviewers will test the PR and highlight if there is any conflict or changes required. If the PR is approved we will proceed to merge the pull request 🙌

[net-foundation-cla]

All CLA requirements met.

[Nirmal4G]

Since, the text is masked, new text with different casing should be considered different than the old text.

But if we need to ignore casing, then it should be an option that a developer can set.

Let me test this and get back to you.

[ghost]

This PR has been marked as "needs attention 👋" and awaiting a response from the team.

[Nirmal4G]

👍🚀

[michael-hawker]

@puppetsw would you be up for adding a new unit test for guarding regressions as well? We have info in our wiki now about this here. There's also an existing UI Test for checking binding you could copy as well, but think it may be simpler to have a unit test for this too since you could update the binding text via code in the test as well with the VisualUITestBase.

[puppetsw]

@michael-hawker Sure can do. I'll write up something shortly.

Edit: I'm not sure if this was the best way to handle this I couldn't figure out a way to test this with VisualUITestBase. So I piggy-backed onto the existing TextBoxMask UI test.