passing tests for wb wpp

GoogleChrome · Jan 3, 2022 · 39d313b · 39d313b
1 parent 3ded4ab
commit 39d313b
Show file tree

Hide file tree

Showing 5 changed files with 30 additions and 24 deletions.
diff --git a/packages/workbox-webpack-plugin/src/generate-sw.ts b/packages/workbox-webpack-plugin/src/generate-sw.ts
@@ -359,4 +359,4 @@ class GenerateSW {
   }
 }
 
-export {GenerateSW}
+export {GenerateSW};
diff --git a/packages/workbox-webpack-plugin/src/inject-manifest.ts b/packages/workbox-webpack-plugin/src/inject-manifest.ts
@@ -212,7 +212,8 @@ class InjectManifest {
 
     const childCompiler = compilation.createChildCompiler(
       this.constructor.name,
-      outputOptions, [],
+      outputOptions,
+      [],
     );
 
     childCompiler.context = parentCompiler.context;
@@ -260,7 +261,9 @@ class InjectManifest {
     parentCompiler: webpack.Compiler,
   ): void {
     // eslint-disable-next-line
-    const source = (parentCompiler.inputFileSystem as any).readFileSync(this.config.swSrc)
+    const source = (parentCompiler.inputFileSystem as any).readFileSync(
+      this.config.swSrc,
+    );
     compilation.emitAsset(this.config.swDest!, new RawSource(source));
   }
 
@@ -278,7 +281,7 @@ class InjectManifest {
       this.config = validateWebpackInjectManifestOptions(this.config);
     } catch (error) {
       // eslint-disable-next-line
-      if(error instanceof Error){
+      if (error instanceof Error) {
         throw new Error(
           `Please check your ${this.constructor.name} plugin ` +
             // eslint-disable-next-line
@@ -331,7 +334,9 @@ class InjectManifest {
             warning instanceof Error && warning.message === warningMessage,
         )
       ) {
-        compilation.warnings.push(new Error(warningMessage) as webpack.WebpackError);
+        compilation.warnings.push(
+          new Error(warningMessage) as webpack.WebpackError,
+        );
       }
     } else {
       this.alreadyCalled = true;
@@ -341,7 +346,7 @@ class InjectManifest {
 
     // Ensure that we don't precache any of the assets generated by *any*
     // instance of this plugin.
-     config.exclude!.push((name:string) => _generatedAssetNames.has(name));
+    config.exclude!.push((name: string) => _generatedAssetNames.has(name));
 
     // See https://webpack.js.org/contribute/plugin-patterns/#monitoring-the-watch-graph
     const absoluteSwSrc = upath.resolve(this.config.swSrc);
@@ -362,7 +367,7 @@ class InjectManifest {
       throw new Error(
         `Multiple instances of ${config.injectionPoint ?? ''} were ` +
           `found in your SW source. Include it only once. For more info, see ` +
-        `https://github.com/GoogleChrome/workbox/issues/2681`,
+          `https://github.com/GoogleChrome/workbox/issues/2681`,
       );
     }
 
@@ -427,4 +432,4 @@ class InjectManifest {
   }
 }
 
-export {InjectManifest}
+export {InjectManifest};
diff --git a/packages/workbox-webpack-plugin/src/lib/get-manifest-entries-from-compilation.ts b/packages/workbox-webpack-plugin/src/lib/get-manifest-entries-from-compilation.ts
@@ -36,7 +36,7 @@ function checkConditions(
 ): boolean {
   for (const condition of conditions) {
     if (typeof condition === 'function') {
-      return condition(asset.name)
+      return condition(asset.name);
     } else {
       if (ModuleFilenameHelpers.matchPart(asset.name, condition)) {
         return true;

diff --git a/packages/workbox-webpack-plugin/src/lib/get-sourcemap-asset-name.ts b/packages/workbox-webpack-plugin/src/lib/get-sourcemap-asset-name.ts
@@ -28,7 +28,11 @@ import type {Compilation} from 'webpack';
  *
  * @private
  */
-export function getSourcemapAssetName(compilation: Compilation, swContents: string, swDest: string): string | undefined {
+export function getSourcemapAssetName(
+  compilation: Compilation,
+  swContents: string,
+  swDest: string,
+): string | undefined {
   // eslint-disable-next-line
   const url = sourceMapURL.getFrom(swContents);
   if (url) {
@@ -39,7 +43,8 @@ export function getSourcemapAssetName(compilation: Compilation, swContents: stri
     // See https://github.com/GoogleChrome/workbox/issues/2250
     const swAssetDirname = upath.dirname(swDest);
     const sourcemapURLAssetName = upath.normalize(
-        upath.join(swAssetDirname, url));
+      upath.join(swAssetDirname, url),
+    );
 
     // Not sure if there's a better way to check for asset existence?
     if (compilation.getAsset(sourcemapURLAssetName)) {

diff --git a/packages/workbox-webpack-plugin/tsconfig.json b/packages/workbox-webpack-plugin/tsconfig.json
@@ -1,18 +1,14 @@
 {
   "extends": "../../tsconfig",
   "compilerOptions": {
-      "esModuleInterop": true,
-      "module": "CommonJS",
-      "outDir": "./build",
-      "resolveJsonModule": true,
-      "rootDir": "./src",
-      "target": "ES2018",
-      "tsBuildInfoFile": "./tsconfig.tsbuildinfo"
+    "esModuleInterop": true,
+    "module": "CommonJS",
+    "outDir": "./build",
+    "resolveJsonModule": true,
+    "rootDir": "./src",
+    "target": "ES2018",
+    "tsBuildInfoFile": "./tsconfig.tsbuildinfo"
   },
-  "include": [
-    "src/**/*.ts"
-  ],
-  "references": [
-    {"path": "../workbox-build/"}
-  ]
+  "include": ["src/**/*.ts"],
+  "references": [{"path": "../workbox-build/"}]
 }