Fix tests flakiness.

[nakul-py]

Description

This PR updates the beforeEach function in ui-tests/tests/contextmenu.spec.ts to delete and upload a new document before each test. So this removes the flakiness of the test as discussed in #1119 (comment)

Checklist

• PR has a descriptive title and content.
• PR description contains references to any issues the PR resolves, e.g. Resolves #XXX.
• PR has one of the labels: documentation, bug, enhancement, feature, maintenance
• Checks are passing.
  Failing lint checks can be resolved with:
  • pre-commit run --all-files
  • jlpm run lint

---

📚 Documentation preview: https://jupytergis--1121.org.readthedocs.build/en/1121/
💡 JupyterLite preview: https://jupytergis--1121.org.readthedocs.build/en/1121/lite

[github-actions]

👈 Launch a Binder on branch nakul-py/jupytergis/fix1119

[github-actions]

Integration tests report: appsharing.space

[mfisher87]

I think this actually wouldn't close #1119, since that bug happens outside of a test context. But fixing the test flakiness is still good :) Could you provide some background on why this fixes the test flakiness? 🤔

Would fixing the underlying but #1119 also fix the flaky test? If the flaky test is telling us about a bug, I think I'd rather not address the flakiness and instead address the bug. That way if the bug ever regresses, we'll know because the test will become flaky again. What do you think?

[nakul-py]

This doesn’t fix #1119 itself, but it fixes test flakiness.

Previously we only closed activities between tests, which resets the UI but not the underlying JupyterGIS document. Since all tests reopened the same .jGIS file path, model state (layers, selection, rename state) leaked across tests, making locators ambiguous and causing F2/rename timeouts.

Re-uploading the .jGIS file in beforeEach guarantees a fresh document + model per test, so every test starts from the same baseline and the flakiness goes away.

[martinRenou]

Thanks! waitForTimeout(1000) is bound to generate flakyness, happy that it's removed