DYN-7919: Fix stray helpdoc filenames in infobubbles, remove dangling reference to inexistent helpdocs.

[chubakueno]

Purpose

#14514 Introduced a bug in which the linked helpdoc file appears concatenated at the end of the message info bubbles (See DereferencingNonPointer.html in the PR screenshots). This PR fixes that issue by properly sanitizing the message content and it also removes dangling references to a testing.html helpdoc that never existed.

Message with bug:

Message without bug in case of several warnings:

Message without bug in case of a single warning:

Declarations

Check these if you believe they are true

• The codebase 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.
• This PR modifies some build requirements and the readme is updated
• This PR contains no files larger than 50 MB

Release Notes

Fix stray helpdoc filename at the end of infobubble messages, remove dangling reference to inexistent helpdocs.

Reviewers

@QilongTang
@RobertGlobant20

FYIs

@avidit

[github-actions]

See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-7919

[github-actions]

UI Smoke Tests

Test: success. 11 passed, 0 failed.
TestComplete Test Result
Workflow Run: UI Smoke Tests
Check: UI Smoke Tests

[QilongTang]

@chubakueno Would you test after sanitizing the warning message, if clicking on learn more would still pop up the warning help content in the right side panel?

[chubakueno]

@chubakueno Would you test after sanitizing the warning message, if clicking on learn more would still pop up the warning help content in the right side panel?

Sure. Some warnings will no longer have the Learn more button because the testing.html link was removed, but others shouldn't be affected by the message sanitization (InfoBubbleDataPacket.ParseLinkFromText and InfoBubbleDataPacket.RemoveLinkFromText are not dependent on each other).

Duplicate warning after removal of the dangling testing.html link.

[chubakueno]

@chubakueno Would you test after sanitizing the warning message, if clicking on learn more would still pop up the warning help content in the right side panel?

Added tests to verify links are properly sanitized and extended the sanitizer to be more lenient (it will now accept everything until a space, newline, or end of string).

[QilongTang]

@chubakueno Thank you, we will have a follow up task to add back the content missed