Refactor tests and prechecks for improved clarity and coverage

Author: GrantBirki

__tests__/functions/timestamp.test.js

@@ -1,18 +1,19 @@
 import {timestamp} from '../../src/functions/timestamp.js'
-import {vi, expect, test, beforeEach} from 'vitest'
+import {vi, expect, test, beforeEach, afterEach} from 'vitest'
 
 beforeEach(() => {
   vi.clearAllMocks()
 })
 
+afterEach(() => {
+  vi.useRealTimers()
+})
+
 test('should return the current date in ISO 8601 format', () => {
   const mockDate = new Date('2025-01-01T00:00:00.000Z')
-  vi.spyOn(global, 'Date').mockImplementation(() => mockDate)
+  vi.setSystemTime(mockDate)
 
   const result = timestamp()
 
-  expect(result).toBe(mockDate.toISOString())
-
-  // Restore the original Date implementation
-  global.Date.mockRestore()
+  expect(result).toBe('2025-01-01T00:00:00.000Z')
 })

src/functions/prechecks.js

@@ -326,13 +326,11 @@ export async function prechecks(context, octokit, data) {
     try {
       core.debug('raw graphql result for debugging:')
       core.debug(result)
-      /* c8 ignore start */
-    } catch {
+    } /* istanbul ignore next */ catch {
       core.debug(
         'Could not output raw graphql result for debugging - This is bad'
       )
     }
-    /* c8 ignore stop */
   }
 
   // Get admin data

src/main.js

@@ -860,15 +860,15 @@ export async function run() {
   }
 }
 
-/* c8 ignore start */
+/* istanbul ignore next */
 if (core.getState('isPost') === 'true') {
   post()
 } else {
+  /* istanbul ignore next */
   if (
     process.env.CI === 'true' &&
     process.env.BRANCH_DEPLOY_VITEST_TEST !== 'true'
   ) {
     run()
   }
 }
-/* c8 ignore stop */