authz.schemaRegistry test suite setup/teardown is flaky

[shouples]

No idea what causes this, but we see the beforeEach()/afterEach() in https://github.com/confluentinc/vscode/blob/main/src/authz/schemaRegistry.test.ts fail occasionally in CI:

Test output:

  207 passing (15s)02:28
  2 failing02:28
  1) authz.schemaRegistry02:28
       "before each" hook for "canAccessSchemaTypeForTopic() should return true if environmentId is not present":02:28
     Error: Timeout of 10000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/semaphore/vscode/out/src/authz/schemaRegistry.test.js)02:28
  	at listOnTimeout (node:internal/timers:581:17)02:28
  	at process.processTimers (node:internal/timers:519:7)02:28
 02:28
  2) authz.schemaRegistry02:28
       "after each" hook for "canAccessSchemaTypeForTopic() should return true if environmentId is not present":02:28
     TypeError: Cannot read properties of undefined (reading 'restore')02:28
  	at Context.<anonymous> (out/src/authz/schemaRegistry.test.js:56:13)02:28
  	at process.processImmediate (node:internal/timers:483:21)02:28
  	at process.callbackTrampoline (node:internal/async_hooks:130:17)02:28
 02:28
Error: 2 tests failed.02:28
	at Object.run (/home/semaphore/vscode/out/src/testing.js:9:1062)02:28
	at process.processTicksAndRejections (node:internal/process/task_queues:95:5)02:28
Extension host test runner error Error: 2 tests failed.02:28
	at Object.run (/home/semaphore/vscode/out/src/testing.js:9:1062)02:28
	at process.processTicksAndRejections (node:internal/process/task_queues:95:5)02:28
Asking native host service to exit with code 1.

Rerunning the workflow usually fixes it, but seeing them fail a few times a week isn't great.

The second failure (Cannot read properties of undefined (reading 'restore')) could probably be solved by creating the sandbox sooner or check whether it's a SinonSandbox vs undefined before calling .restore(), but the timeout may be related to latency activating the extension between tests, and may be better placed in a before() block instead of beforeEach().