fix: fix error handling in .table()

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

Author: jerome-benoit

package.json

@@ -39,7 +39,9 @@
       "import": "./dist/index.js"
     }
   },
-  "files": ["dist/**"],
+  "files": [
+    "dist/**"
+  ],
   "repository": "tinylibs/tinybench",
   "license": "MIT",
   "devDependencies": {
@@ -65,5 +67,9 @@
     "typescript": "~5.6.3",
     "vitest": "^2.1.5"
   },
-  "keywords": ["benchmark", "tinylibs", "tiny"]
+  "keywords": [
+    "benchmark",
+    "tinylibs",
+    "tiny"
+  ]
 }

src/bench.ts

@@ -233,8 +233,6 @@ export class Bench extends EventTarget {
               Error: task.result.error.message,
               // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
               Stack: task.result.error.stack!,
-              // eslint-disable-next-line perfectionist/sort-objects
-              Samples: task.result.latency.samples.length,
             }
           : (convert?.(task) ?? {
               'Task name': task.name,

test/index.test.ts

@@ -64,6 +64,24 @@ test('bench table', async () => {
       Samples: expect.any(Number),
     },
   ])
+
+  bench.remove('foo')
+  bench.add('bar', () => {
+    throw new Error('fake')
+  })
+
+  await bench.run()
+
+  expect(bench.table()).toStrictEqual([
+    {
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+      'Task name': expect.any(String),
+      // eslint-disable-next-line perfectionist/sort-objects, @typescript-eslint/no-unsafe-assignment
+      Error: expect.any(String),
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+      Stack: expect.any(String),
+    },
+  ])
 })
 
 test('bench task runs and time consistency', async () => {