fix: Account for case where they are 0

Author: AbhiPrasad

packages/tracing/src/browser/metrics.ts

@@ -233,13 +233,13 @@ export function addResourceSpans(
   }
 
   const data: Record<string, any> = {};
-  if (entry.transferSize) {
+  if ('transferSize' in entry) {
     data.transferSize = entry.transferSize;
   }
-  if (entry.encodedBodySize) {
+  if ('encodedBodySize' in entry) {
     data.encodedBodySize = entry.encodedBodySize;
   }
-  if (entry.decodedBodySize) {
+  if ('decodedBodySize' in entry) {
     data.decodedBodySize = entry.decodedBodySize;
   }
 
@@ -249,7 +249,7 @@ export function addResourceSpans(
   _startChild(transaction, {
     description: resourceName,
     endTimestamp,
-    op: entry.initiatorType && entry.initiatorType !== '' ? `resource.${entry.initiatorType}` : 'resource',
+    op: entry.initiatorType ? `resource.${entry.initiatorType}` : 'resource',
     startTimestamp,
     data,
   });

packages/tracing/test/browser/metrics.test.ts

@@ -142,4 +142,28 @@ describe('addResourceSpans', () => {
       );
     }
   });
+
+  it('allows for enter size of 0', () => {
+    const entry: ResourceEntry = {
+      initiatorType: 'css',
+      transferSize: 0,
+      encodedBodySize: 0,
+      decodedBodySize: 0,
+    };
+
+    addResourceSpans(transaction, entry, '/assets/to/css', 100, 23, 345);
+
+    // eslint-disable-next-line @typescript-eslint/unbound-method
+    expect(transaction.startChild).toHaveBeenCalledTimes(1);
+    // eslint-disable-next-line @typescript-eslint/unbound-method
+    expect(transaction.startChild).toHaveBeenLastCalledWith(
+      expect.objectContaining({
+        data: {
+          decodedBodySize: entry.decodedBodySize,
+          encodedBodySize: entry.encodedBodySize,
+          transferSize: entry.transferSize,
+        },
+      }),
+    );
+  });
 });