non-zero size estimate (#5139)

voxel51 · Nov 18, 2024 · 9a5eb8e · 9a5eb8e
1 parent cb08bcc
commit 9a5eb8e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/app/packages/utilities/src/size-bytes-estimate.test.ts b/app/packages/utilities/src/size-bytes-estimate.test.ts
@@ -32,7 +32,7 @@ describe("sizeBytesEstimate tests", () => {
     expect(sizeBytesEstimate("chars")).toBe(10);
   });
 
-  test("non-objects return 0", () => {
-    expect(sizeBytesEstimate(() => null)).toBe(0);
+  test("non-objects return non-zero", () => {
+    expect(sizeBytesEstimate(() => null)).toBe(1);
   });
 });
diff --git a/app/packages/utilities/src/size-bytes-estimate.ts b/app/packages/utilities/src/size-bytes-estimate.ts
@@ -64,5 +64,6 @@ const sizer = (object: SizerTypes) => {
 };
 
 export default function sizeBytesEstimate(object: SizeTypes) {
-  return sizer(object);
+  // return value > 0;
+  return Math.max(sizer(object), 1);
 }