fix(laboratory): register the canary upload hook once per suite

Playwright scopes afterEach to the suite, not to the test object it was
registered on, so registering per canary fixture ran both hooks for every
test and uploaded each datapoint twice. Registering on the shared
timingFixture covers every test object in the file and fires once.

Applies the hook placement from #5753.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: enesozturk

apps/laboratory/tests/verify.spec.ts

@@ -15,17 +15,12 @@ import { afterEachCanary, getCanaryTagAndAnnotation } from './shared/utils/metri
 import { ModalValidator } from './shared/validators/ModalValidator'
 
 /*
- * Upload canary metrics for the @canary-tagged verify tests below. This hook is a
- * no-op for tests without the @canary tag and outside canary environments.
+ * Uploads canary metrics for the @canary-tagged tests below; a no-op for untagged tests and
+ * outside canary environments. Registered on the shared timingFixture rather than on each canary
+ * fixture: Playwright scopes hooks to the suite, so per-fixture registration would run every hook
+ * for every test in the file and upload each datapoint twice.
  */
-testMWagmiVerifyValid.afterEach(async ({ browserName, timingRecords }, testInfo) => {
-  if (browserName === 'firefox') {
-    return
-  }
-  await afterEachCanary(testInfo, timingRecords)
-})
-
-testMWagmiVerifyEvil.afterEach(async ({ browserName, timingRecords }, testInfo) => {
+timingFixture.afterEach(async ({ browserName, timingRecords }, testInfo) => {
   if (browserName === 'firefox') {
     return
   }