DYN-6507: crash when uploading package

[dnenov]

Purpose

Revert change that was introduced to try and fix failing tests before. The change was a BeingInvoke wrap around a clean up of a property which was used inside async code. The use of the BeginInvoke was causing a crash in production code in Dynamo.

This jira issue picks it up https://jira.autodesk.com/browse/DYN-6507?filter=-1

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

• attempt to fix a crash when uploading a package and clearing the uploadhandle on success

Reviewers

@reddyashish

FYIs

@avidit
@mjkkirschner

[reddyashish]

@dnenov 1 failure here: https://master-15.jenkins.autodesk.com/view/DYN/job/DYN-DevCI_Self_Service/1329/. No test runner crash now

[reddyashish]

1 failure

[dnenov]

I disabled CancelCommandClearsAllData test as the command it is supposed to be testing, is currently restricted under testing environment. It crashes master-15 (sequentially running of tests) because of thread affinity issue. So this test does not make sense at the moment.

[github-actions]

UI Smoke Tests

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

[github-actions]

UI Smoke Tests

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

[reddyashish]

https://master-15.jenkins.autodesk.com/view/DYN/job/DYN-DevCI_Self_Service/1333/ passed.