test(service-worker): do not create testing artifacts on environments that do not support SW (#27080)

The tests will not be run anyway, so the artifacts are never used and
there might be errors if creating the testing artifacts relies on APIs
that are not available in that environment (e.g. `URL`).

PR Close #27080

packages/service-worker/test/integration_spec.ts

@@ -21,6 +21,12 @@ import {take} from 'rxjs/operators';
 
 import {async_beforeEach, async_fit, async_it} from './async';
 
+(function() {
+  // Skip environments that don't support the minimum APIs needed to run the SW tests.
+  if (!SwTestHarness.envIsSupported()) {
+    return;
+  }
+
   const dist = new MockFileSystemBuilder().addFile('/only.txt', 'this is only').build();
 
   const distUpdate = new MockFileSystemBuilder().addFile('/only.txt', 'this is only v2').build();
@@ -66,11 +72,7 @@ const server = new MockServerStateBuilder().withStaticFiles(dist).withManifest(m
   const serverUpdate =
       new MockServerStateBuilder().withStaticFiles(distUpdate).withManifest(manifestUpdate).build();
 
-(function() {
-  // Skip environments that don't support the minimum APIs needed to run the SW tests.
-  if (!SwTestHarness.envIsSupported()) {
-    return;
-  }
+
   describe('ngsw + companion lib', () => {
     let mock: MockServiceWorkerContainer;
     let comm: NgswCommChannel;

packages/service-worker/worker/test/data_spec.ts

@@ -15,6 +15,12 @@ import {SwTestHarness, SwTestHarnessBuilder} from '../testing/scope';
 
 import {async_beforeEach, async_fit, async_it} from './async';
 
+(function() {
+  // Skip environments that don't support the minimum APIs needed to run the SW tests.
+  if (!SwTestHarness.envIsSupported()) {
+    return;
+  }
+
   const dist = new MockFileSystemBuilder()
                    .addFile('/foo.txt', 'this is foo')
                    .addFile('/bar.txt', 'this is bar')
@@ -112,15 +118,7 @@ const serverSeqUpdate = new MockServerStateBuilder()
 
   const scope = new SwTestHarnessBuilder().withServerState(server).build();
 
-function asyncWrap(fn: () => Promise<void>): (done: DoneFn) => void {
-  return (done: DoneFn) => { fn().then(() => done(), err => done.fail(err)); };
-}
 
-(function() {
-  // Skip environments that don't support the minimum APIs needed to run the SW tests.
-  if (!SwTestHarness.envIsSupported()) {
-    return;
-  }
   describe('data cache', () => {
     let scope: SwTestHarness;
     let driver: Driver;

packages/service-worker/worker/test/happy_spec.ts

@@ -18,6 +18,12 @@ import {SwTestHarness, SwTestHarnessBuilder} from '../testing/scope';
 
 import {async_beforeEach, async_fit, async_it} from './async';
 
+(function() {
+  // Skip environments that don't support the minimum APIs needed to run the SW tests.
+  if (!SwTestHarness.envIsSupported()) {
+    return;
+  }
+
   const dist =
       new MockFileSystemBuilder()
           .addFile('/foo.txt', 'this is foo')
@@ -42,7 +48,8 @@ const distUpdate =
           .addFile('/quuux.txt', 'this is quuux v2')
           .addFile('/lazy/unchanged1.txt', 'this is unchanged (1)')
           .addFile('/lazy/unchanged2.txt', 'this is unchanged (2)')
-        .addUnhashedFile('/unhashed/a.txt', 'this is unhashed v2', {'Cache-Control': 'max-age=10'})
+          .addUnhashedFile(
+              '/unhashed/a.txt', 'this is unhashed v2', {'Cache-Control': 'max-age=10'})
           .addUnhashedFile('/ignored/file1', 'this is not handled by the SW')
           .addUnhashedFile('/ignored/dir/file2', 'this is not handled by the SW either')
           .build();
@@ -69,7 +76,7 @@ const brokenManifest: Manifest = {
 
   // Manifest without navigation urls to test backward compatibility with
   // versions < 6.0.0.
-export interface ManifestV5 {
+  interface ManifestV5 {
     configVersion: number;
     appData?: {[key: string]: string};
     index: string;
@@ -214,11 +221,7 @@ const server404 = new MockServerStateBuilder().withStaticFiles(dist).build();
   const manifestHash = sha1(JSON.stringify(manifest));
   const manifestUpdateHash = sha1(JSON.stringify(manifestUpdate));
 
-(function() {
-  // Skip environments that don't support the minimum APIs needed to run the SW tests.
-  if (!SwTestHarness.envIsSupported()) {
-    return;
-  }
+
   describe('Driver', () => {
     let scope: SwTestHarness;
     let driver: Driver;

packages/service-worker/worker/test/idle_spec.ts

@@ -15,6 +15,7 @@ import {async_beforeEach, async_fit, async_it} from './async';
   if (!SwTestHarness.envIsSupported()) {
     return;
   }
+
   describe('IdleScheduler', () => {
     let scope: SwTestHarness;
     let idle: IdleScheduler;

packages/service-worker/worker/test/prefetch_spec.ts

@@ -14,6 +14,12 @@ import {SwTestHarness, SwTestHarnessBuilder} from '../testing/scope';
 
 import {async_fit, async_it} from './async';
 
+(function() {
+  // Skip environments that don't support the minimum APIs needed to run the SW tests.
+  if (!SwTestHarness.envIsSupported()) {
+    return;
+  }
+
   const dist = new MockFileSystemBuilder()
                    .addFile('/foo.txt', 'this is foo')
                    .addFile('/bar.txt', 'this is bar')
@@ -28,12 +34,6 @@ const scope = new SwTestHarnessBuilder().withServerState(server).build();
   const db = new CacheDatabase(scope, scope);
 
 
-
-(function() {
-  // Skip environments that don't support the minimum APIs needed to run the SW tests.
-  if (!SwTestHarness.envIsSupported()) {
-    return;
-  }
   describe('prefetch assets', () => {
     let group: PrefetchAssetGroup;
     let idle: IdleScheduler;