Fix: Remove semicolons and add tests

Author: trentonallan

packages/pg/lib/utils.js

@@ -97,9 +97,8 @@ function dateToString(date) {
   // Invalid dates can occur from new Date(undefined), new Date(NaN), etc.
   // See https://github.com/brianc/node-postgres/issues/3318
   if (!date || !(date instanceof Date) || isNaN(date.getTime())) {
-    throw new TypeError('Cannot serialize invalid date');
+    throw new TypeError('Cannot serialize invalid date')
   }
-  
   let offset = -date.getTimezoneOffset()
   let year = date.getFullYear()
   const isBCYear = year < 1
@@ -137,7 +136,7 @@ function dateToStringUTC(date) {
   // Invalid dates can occur from new Date(undefined), new Date(NaN), etc.
   // See https://github.com/brianc/node-postgres/issues/3318
   if (!date || !(date instanceof Date) || isNaN(date.getTime())) {
-    throw new TypeError('Cannot serialize invalid date');
+    throw new TypeError('Cannot serialize invalid date')
   }
 
   let year = date.getUTCFullYear()

packages/pg/test/unit/utils-tests.js

@@ -314,3 +314,28 @@ testEscapeIdentifier(
   'hello \\ \' " world',
   '"hello \\ \' "" world"'
 )
+
+test('prepareValue: invalid date from undefined throws error', function () {
+  assert.throws(() => {
+    utils.prepareValue(new Date(undefined))
+  }, /Cannot serialize invalid date/)
+})
+
+test('prepareValue: invalid date from NaN throws error', function () {
+  assert.throws(() => {
+    utils.prepareValue(new Date(NaN))
+  }, /Cannot serialize invalid date/)
+})
+
+test('prepareValue: invalid date from invalid string throws error', function () {
+  assert.throws(() => {
+    utils.prepareValue(new Date('invalid'))
+  }, /Cannot serialize invalid date/)
+})
+
+test('prepareValue: valid date still works correctly', function () {
+  const date = new Date('2024-01-01T12:00:00Z')
+  const result = utils.prepareValue(date)
+  assert(typeof result === 'string')
+  assert(result.includes('2024'))
+})