CPLAT-1053 Cancel the transitionend timer when AbstractTransitionComponent unmounts

[aaronlademann-wf]

• To prevent ReactJS setState warnings involving unmounted components

Motivation

ReactJS setState warnings are being produced by components that extend from AbstractTransitionComponent.

Changes

Cancel the transitionend timer in-addition-to the subscription within componentWillUnmount.

Release Notes

Fix issue that causes ReactJS setState warnings to appear in the browser console.

Review

See CONTRIBUTING.md for more details on review types (+1 / QA +1 / +10) and code review process.

Please review: @greglittlefield-wf

QA Checklist

• Tests were updated and provide good coverage of the changeset and other affected code
• Manual testing was performed if needed
  • Steps from PR author:
  • Anything falling under manual testing criteria outlined in CONTRIBUTING.md

Merge Checklist

While we perform many automated checks before auto-merging, some manual checks are needed:

• A Client Platform member has reviewed these changes
• There are no unaddressed comments - this check can be automated if reviewers use the "Request Changes" feature
• For release PRs - Version metadata in Rosie comment is correct

[aviary2-wf]

Security Insights

No security relevant content was detected by automated scans.

Action Items

• Review PR for security impact; comment "security review required" if needed or unsure
• Verify aviary.yaml coverage of security relevant code

---

Questions or Comments? Reach out on Slack: #support-infosec.

[codecov-io]

Codecov Report

Merging #294 into 1.x_release_base will increase coverage by 0.01%.
The diff coverage is 100%.

@@                 Coverage Diff                  @@
##           1.x_release_base     #294      +/-   ##
====================================================
+ Coverage             90.27%   90.27%   +0.01%     
====================================================
  Files                    36       36              
  Lines                  1807     1808       +1     
====================================================
+ Hits                   1631     1632       +1     
  Misses                  176      176

[greglittlefield-wf]

+1

[greglittlefield-wf]

+1

[joebingham-wk]

+1

[aaronlademann-wf]

QA +1

@Workiva/release-management-pp