Remove isDev() environment unit test, changes value depending on whether run locally in husky pre-commit hook or on GH Actions workflow

Author: webstackdev

src/components/scripts/utils/__tests__/environmentClient.spec.ts

@@ -4,7 +4,7 @@
  * Tests the environment detection functions in a Vitest/jsdom context
  */
 import { describe, it, expect } from 'vitest'
-import { isUnitTest, isE2eTest, isTest, isDev, isProd } from '@lib/config/environmentServer'
+import { isUnitTest, isE2eTest, isTest, isProd } from '@lib/config/environmentServer'
 
 describe('Environment detection utilities', () => {
   it('isUnitTest should return true when running in Vitest', () => {
@@ -19,9 +19,7 @@ describe('Environment detection utilities', () => {
     expect(isTest()).toBe(true)
   })
 
-  it('isDev() should return true when running in Vitest', () => {
-    expect(isDev()).toBe(true)
-  })
+  // Can't test isDev() - result varies between local and GH Action runner
 
   it('isProd() should return false when running in Vitest', () => {
     expect(isProd()).toBe(false)

src/lib/config/__tests__/environmentServer.spec.ts

@@ -3,7 +3,7 @@
  * Tests the wrapper functions around js-cookie library
  */
 import { describe, it, expect } from 'vitest'
-import { isDev, isE2eTest, isProd, isTest, isUnitTest } from '@lib/config'
+import { isE2eTest, isProd, isTest, isUnitTest } from '@lib/config'
 
 describe('Build-time environment handling utilities', () => {
   it('should always report running in Vitest in a unit test', () => {
@@ -18,9 +18,7 @@ describe('Build-time environment handling utilities', () => {
     expect(isTest()).toBeTruthy()
   })
 
-  it('isDev() should return true when running in Vitest', () => {
-    expect(isDev()).toBeTruthy()
-  })
+  // Can't test isDev() - result varies between local and GH Action runner
 
   it('isProd() should return false when running in Vitest', () => {
     expect(isProd()).toBeFalsy()