chore: wip

Author: chrisbbreuer

packages/launchpad/test/services.test.ts

@@ -8,15 +8,12 @@ import { createDefaultServiceConfig, getAllServiceDefinitions, getServiceDefinit
 import { disableService, enableService, getServiceStatus, initializeServiceManager, listServices, restartService, startService, stopService } from '../src/services/manager'
 import { generateLaunchdPlist, generateSystemdService, getServiceManagerName, isPlatformSupported } from '../src/services/platform'
 
-// Helper function to detect if we're in a CI environment
+// Helper function to detect if we're in a CI environment where services aren't available
 function isRunningInCI(): boolean {
   return process.env.CI === 'true'
     || process.env.GITHUB_ACTIONS === 'true'
     || process.env.RUNNER_OS !== undefined
     || process.env.GITHUB_RUN_ID !== undefined
-  // Check if PostgreSQL tools are actually available
-    || !process.env.PATH?.includes('/usr/bin')
-    || process.env.NODE_ENV === 'test' // Fallback for test environments
 }
 
 describe('Service Management', () => {
@@ -501,7 +498,7 @@ describe('Service Management', () => {
       }
 
       // Skip actual service operations in CI where services aren't installed
-      if (process.env.CI || process.env.GITHUB_ACTIONS) {
+      if (isRunningInCI()) {
         const serviceName = 'redis'
         expect(serviceName).toBe('redis')
         return
@@ -568,7 +565,7 @@ describe('Service Management', () => {
       }
 
       // Skip actual service operations in CI where services aren't installed
-      if (process.env.CI || process.env.GITHUB_ACTIONS) {
+      if (isRunningInCI()) {
         // Just test configuration setting without actually starting services
         const mockConfig = {
           'maxmemory': '128mb',
@@ -612,7 +609,7 @@ describe('Service Management', () => {
         expect(isPlatformSupported()).toBe(true)
 
         // Skip actual service operations in CI where services aren't installed
-        if (process.env.CI || process.env.GITHUB_ACTIONS) {
+        if (isRunningInCI()) {
           // Just test that we can import and call the platform check
           expect(isPlatformSupported()).toBe(true)
         }