get back to 100% test coverage

Author: GrantBirki

__tests__/functions/prechecks.test.js

@@ -1,5 +1,5 @@
 import {prechecks} from '../../src/functions/prechecks.js'
-import {vi, expect, describe, test, beforeEach, afterEach} from 'vitest'
+import {vi, expect, test, beforeEach} from 'vitest'
 import {COLORS} from '../../src/functions/colors.js'
 import * as isAdmin from '../../src/functions/admin.js'
 import * as isOutdated from '../../src/functions/outdated-check.js'

__tests__/functions/string-to-array.test.js

@@ -1,8 +1,9 @@
 import {stringToArray} from '../../src/functions/string-to-array.js'
-import {vi, expect, describe, test, beforeEach, afterEach} from 'vitest'
+import {vi, expect, test, beforeEach} from 'vitest'
 import * as core from '@actions/core'
 
 const debugMock = vi.spyOn(core, 'debug')
+const errorMock = vi.spyOn(core, 'error')
 
 beforeEach(() => {
   vi.clearAllMocks()
@@ -33,3 +34,9 @@ test('successfully converts an empty string to an empty array', async () => {
 test('successfully converts garbage to an empty array', async () => {
   expect(stringToArray(',,,')).toStrictEqual([])
 })
+
+test('throws an error when string processing fails', async () => {
+  // Pass a non-string value to trigger the error
+  expect(() => stringToArray(null)).toThrow('could not convert String to Array')
+  expect(errorMock).toHaveBeenCalled()
+})

badges/coverage.svg

@@ -326,12 +326,13 @@ 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
       core.debug(
         'Could not output raw graphql result for debugging - This is bad'
       )
     }
+    /* c8 ignore stop */
   }
 
   // Get admin data

dist/index.js

@@ -860,7 +860,7 @@ export async function run() {
   }
 }
 
-/* istanbul ignore next */
+/* c8 ignore start */
 if (core.getState('isPost') === 'true') {
   post()
 } else {
@@ -871,3 +871,4 @@ if (core.getState('isPost') === 'true') {
     run()
   }
 }
+/* c8 ignore stop */