test(service-worker): remove redundant ternary operator from MockRequest initialization (#46912)

gkalpak · AndrewKushnir · commit 99e5fa157dd9 · 2022-08-01T09:51:56.000-07:00
Since the `init` parameter has a default value of `{}`, it can never be `!== undefined`. Thus, it is not necessary to account for that case. PR Close #46912
diff --git a/packages/service-worker/worker/testing/fetch.ts b/packages/service-worker/worker/testing/fetch.ts
@@ -115,7 +115,7 @@ export class MockRequest extends MockBody implements Request {
   url: string;
 
   constructor(input: string|Request, init: RequestInit = {}) {
-    super(init !== undefined ? (init.body as (string | null)) || null : null);
+    super((init.body as string | null) ?? null);
     if (typeof input !== 'string') {
       throw 'Not implemented';
     }

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ export class MockRequest extends MockBody implements Request {`
`115`	`115`	`url: string;`
`116`	`116`
`117`	`117`	`constructor(input: string\|Request, init: RequestInit = {}) {`
`118`		`- super(init !== undefined ? (init.body as (string \| null)) \|\| null : null);`
	`118`	`+ super((init.body as string \| null) ?? null);`
`119`	`119`	`if (typeof input !== 'string') {`
`120`	`120`	`throw 'Not implemented';`
`121`	`121`	`}`