refactor(component-testing): simplify vue mount

[elevatebart]

• Closes CT-367

User facing changelog

Fixes some of the memory leaks in with @cypress/vue

Additional details

• Avoid recreating the root element at each test
• Make sure we properly destroy the vue instance after each iteration

This might fix some issues with devtools

PR Tasks

• Has the original issue or this PR been tagged with a release in ZenHub?

[cypress-bot]

Thanks for taking the time to open a PR!

• Create a Draft Pull Request if your PR is not ready for review. Mark the PR as Ready for Review when you're ready for a Cypress team member to review the PR.
• Become familiar with the Code Review Checklist for guidelines on coding standards and what needs to be done before a PR can be merged.

[lmiller1990]

We could use the feature that is built in to test utils: https://vue-test-utils.vuejs.org/api/#enableautodestroy-hook

[lmiller1990]

I think we could consider using the version built into test utils: We could use the feature that is built in to test utils: https://vue-test-utils.vuejs.org/api/#enableautodestroy-hook

[elevatebart]

@lmiller1990 I do not think we can put anything in afterEach without reaching for a support file.
It would uselessly complicate the installation process.

Where were you thinking we call this enableAutoDestroy function?

[elevatebart]

I understand after looking at your vite PR what you mean.
Tell me if my implementation seems fine.

[cypress]

---

Test summary

9392 • 0 • 119 • 3 • 1

---

Run details

Project	cypress
Status	Passed
Commit	b7a380b
Started	Mar 19, 2021 12:23 AM
Ended	Mar 19, 2021 12:34 AM
Duration	10:46 💡
OS	Linux Debian - 10.5
Browser	Multiple

View run in Cypress Dashboard ➡️

---

Flakiness

	cypress/integration/retries.ui.spec.js		1
1	runner/cypress retries.ui.spec > opens attempt on each attempt failure for the screenshot, and closes after test passes

---

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

[JessicaSachs]

This looks okay to me. It's an improvement on where we currently are. There are some fundamental issues with how we are cleaning up, but those will be addressed when we refactor these to use Cypress's internal events.

[lmiller1990]

✅

[JessicaSachs]

Manually tested this. It's looking good.

[cypress-bot]

Released in 7.0.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v7.0.0, please open a new issue.