Flaky test "fix"

We've seen some flaky behavior in the beforeEach for VE logging. Drop
the beforeEach so that flake exoneration can do its job.

Bug: none
Change-Id: I530e37a6135f90bb9cbd3fa27d810c6e10a6bae1
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6343237
Commit-Queue: Philip Pfaffe <[email protected]>
Auto-Submit: Philip Pfaffe <[email protected]>
Reviewed-by: Danil Somsikov <[email protected]>
Commit-Queue: Danil Somsikov <[email protected]>

Author: pfaffe

test/e2e/application/service-worker-update_test.ts

@@ -4,7 +4,7 @@
 import {assert} from 'chai';
 
 import {unregisterAllServiceWorkers} from '../../conductor/hooks.js';
-import {step, waitFor} from '../../shared/helper.js';
+import {waitFor} from '../../shared/helper.js';
 import {
   navigateToApplicationTab,
   navigateToServiceWorkers,
@@ -15,20 +15,14 @@ const TEST_HTML_FILE = 'service-worker-network';
 const SERVICE_WORKER_UPDATE_TIMELINE_SELECTOR = '.service-worker-update-timing-table';
 
 describe('The Application Tab', function() {
-  beforeEach(async function() {
+  it('Navigate to a page with service worker we should find service worker update timeline info', async () => {
     await navigateToApplicationTab(TEST_HTML_FILE);
     await navigateToServiceWorkers();
-  });
 
-  afterEach(async () => {
-    await unregisterAllServiceWorkers();
-  });
+    const timeline = await waitFor(SERVICE_WORKER_UPDATE_TIMELINE_SELECTOR);
+    assert.isDefined(timeline);
 
-  it('Navigate to a page with service worker we should find service worker update timeline info', async () => {
-    await step('wait and locate service worker update time line', async () => {
-      const timeline = await waitFor(SERVICE_WORKER_UPDATE_TIMELINE_SELECTOR);
-      assert.isDefined(timeline);
-    });
     await unregisterServiceWorker();
+    await unregisterAllServiceWorkers();
   });
 });