refactor: refine circular buffer error message

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

diff --git a/src/circular-buffer.ts b/src/circular-buffer.ts
index 55f5f9bfc22e48f6f6e42da61c252ec124ac0bca..7abb982ea6adb3ebcd2881efbb9d1b7ef8c66682 100644 (file)
--- a/src/circular-buffer.ts
+++ b/src/circular-buffer.ts
@@ -87,7 +87,7 @@ export class CircularBuffer {
   private checkSize (size: number): void {
     if (!Number.isSafeInteger(size)) {
       throw new TypeError(
-        `Invalid circular buffer size: ${size} is not an integer`
+        `Invalid circular buffer size: '${size}' is not an integer`
       )
     }
     if (size < 0) {

diff --git a/src/fixed-priority-queue.ts b/src/fixed-priority-queue.ts
index 075986e26b09acbd5b30f1d3247725f6ccc52588..fadaee750515b3de72bcc6c5a9f6cd70de0b68cd 100644 (file)
--- a/src/fixed-priority-queue.ts
+++ b/src/fixed-priority-queue.ts
@@ -130,7 +130,7 @@ export class FixedPriorityQueue<T> {
   private checkSize (size: number): void {
     if (!Number.isSafeInteger(size)) {
       throw new TypeError(
-        `Invalid fixed priority queue size: ${size} is not an integer`
+        `Invalid fixed priority queue size: '${size}' is not an integer`
       )
     }
     if (size < 0) {

diff --git a/tests/circular-buffer.test.mjs b/tests/circular-buffer.test.mjs
index 2d275237413c9f91741cf3f09170a40e9446c1bd..f6afdda9cd63999aacc817ec1878d7dd3b7dde94 100644 (file)
--- a/tests/circular-buffer.test.mjs
+++ b/tests/circular-buffer.test.mjs
@@ -24,16 +24,16 @@ describe('Circular buffer test suite', t => {
 
   it('Verify that circular buffer size is valid at instance creation', () => {
     expect(() => new CircularBuffer(0.25)).toThrow(
-      new TypeError('Invalid circular buffer size: 0.25 is not an integer')
+      new TypeError("Invalid circular buffer size: '0.25' is not an integer")
     )
     expect(() => new CircularBuffer(-1)).toThrow(
       new RangeError('Invalid circular buffer size: -1 < 0')
     )
     expect(() => new CircularBuffer(Number.MAX_SAFE_INTEGER + 1)).toThrow(
       new TypeError(
-        `Invalid circular buffer size: ${
+        `Invalid circular buffer size: '${
           Number.MAX_SAFE_INTEGER + 1
-        } is not an integer`
+        }' is not an integer`
       )
     )
   })

diff --git a/tests/fixed-priority-queue.test.mjs b/tests/fixed-priority-queue.test.mjs
index 7f8e380f013f4a8af1970021f93223259a46419d..7a1506906817ee1c0c8022b5708777b69ad672f2 100644 (file)
--- a/tests/fixed-priority-queue.test.mjs
+++ b/tests/fixed-priority-queue.test.mjs
@@ -8,7 +8,7 @@
 // describe('Fixed priority queue test suite', () => {
 //   it('Verify constructor() behavior', () => {
 //     expect(() => new FixedPriorityQueue('')).toThrow(
-//       new TypeError('Invalid fixed priority queue size:  is not an integer')
+//       new TypeError('Invalid fixed priority queue size: '' is not an integer')
 //     )
 //     expect(() => new FixedPriorityQueue(-1)).toThrow(
 //       new RangeError('Invalid fixed priority queue size: -1 < 0')