chore(util-body-length): throw Error if body length can't be computed (…

…#3401)
aws · Mar 9, 2022 · 813ff8a · 813ff8a
1 parent 88f8cc2
commit 813ff8a
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 0 deletions.
diff --git a/packages/util-body-length-browser/src/calculateBodyLength.spec.ts b/packages/util-body-length-browser/src/calculateBodyLength.spec.ts
@@ -33,4 +33,10 @@ describe(calculateBodyLength.name, () => {
     };
     expect(calculateBodyLength(mockFileObject)).toEqual(mockFileObject.size);
   });
+
+  it.each([true, 1, {}, []])("throws error if Body Length computation fails for: %s", (body) => {
+    expect(() => {
+      expect(calculateBodyLength(body));
+    }).toThrowError(`Body Length computation failed for ${body}`);
+  });
 });
diff --git a/packages/util-body-length-browser/src/calculateBodyLength.ts b/packages/util-body-length-browser/src/calculateBodyLength.ts
@@ -16,4 +16,5 @@ export const calculateBodyLength = (body: any): number | undefined => {
     // handles browser File object
     return body.size;
   }
+  throw new Error(`Body Length computation failed for ${body}`);
 };
diff --git a/packages/util-body-length-node/src/calculateBodyLength.spec.ts b/packages/util-body-length-node/src/calculateBodyLength.spec.ts
@@ -61,4 +61,10 @@ describe(calculateBodyLength.name, () => {
       }
     });
   });
+
+  it.each([true, 1, {}, []])("throws error if Body Length computation fails for: %s", (body) => {
+    expect(() => {
+      expect(calculateBodyLength(body));
+    }).toThrowError(`Body Length computation failed for ${body}`);
+  });
 });
diff --git a/packages/util-body-length-node/src/calculateBodyLength.ts b/packages/util-body-length-node/src/calculateBodyLength.ts
@@ -18,4 +18,5 @@ export const calculateBodyLength = (body: any): number | undefined => {
     // handles fd readable streams
     return fstatSync(body.fd).size;
   }
+  throw new Error(`Body Length computation failed for ${body}`);
 };