test(e2e): add tests for budget edit

[sidvishnoi]

Context

Part of #610

Changes proposed in this pull request

[github-actions]

Extension builds preview

Name	Link
Latest commit	38a7086
Latest job logs	Run #13033515239
	Download
	Download

[sidvishnoi]

Figuring out why worker gets reused here.. It's supposed to disconnect wallet after each file finishes running.

[sidvishnoi]

Playwright Test reuses a single worker as much as it can to make testing faster, so multiple test files are usually run in a single worker one after another.
Workers are always shutdown after a test failure to guarantee pristine environment for following tests.

Sad here.

[sidvishnoi]

So, connected fixture is buggy. Might explain flakiness of some tests. When connection fails (as wallet is already connected in that worker), that worker dies, and things start afresh - making them pass.

[sidvishnoi]

So...

afterAll: Declares an afterAll hook that is executed once per worker after all tests.
afterEach: Declares an afterEach hook that is executed after each test.

There's no way to run a hook before all tests in a file. A worker will share state across files. 🤦🏽

Filed microsoft/playwright#34519 to see if can get some hint there.

[sidvishnoi]

Closing this as it got too noisy with attempts on fixing flakiness. Will send fresh PR.