fix tests and force color in unit tests

Author: eablack

package.json

@@ -398,8 +398,8 @@
     "test": "npm run test:all",
     "test:all": "npm run pretest && c8 --no-clean -- mocha \"test/**/*.unit.test.ts\"",
     "test:file": "npm run pretest && c8 --no-clean -- mocha",
-    "test:ci": "FORCE_COLOR=true npm run test:ci:unit",
-    "test:ci:unit": "npm run pretest && mocha --forbid-only \"test/**/*.unit.test.ts\"",
+    "test:ci": "npm run test:ci:unit",
+    "test:ci:unit": "FORCE_COLOR=true npm run pretest && mocha --forbid-only \"test/**/*.unit.test.ts\"",
     "test:ci:unit:coverage": "npm run pretest && c8 --all --check-coverage --reporter=lcov --reporter=text-summary mocha --forbid-only \"test/**/*.unit.test.ts\"",
     "test:ci:acceptance": "npm run pretest && mocha --forbid-only \"test/**/*.acceptance.test.ts\" && node ./bin/bats-test-runner",
     "test:ci:integration": "npm run pretest && mocha --forbid-only \"test/**/*.integration.test.ts\"",

test/unit/analytics-telemetry/backboard-otel-client.unit.test.ts

@@ -10,15 +10,23 @@ const isDev = process.env.IS_DEV_ENVIRONMENT === 'true'
 describe('backboard-otel-client', function () {
   let sandbox: sinon.SinonSandbox
   let client: BackboardOtelClient
+  let originalTestEnv: string | undefined
 
   beforeEach(function () {
     sandbox = sinon.createSandbox()
+    // Temporarily enable telemetry for these tests
+    originalTestEnv = process.env.IS_HEROKU_TEST_ENV
+    delete process.env.IS_HEROKU_TEST_ENV
     client = new BackboardOtelClient()
   })
 
   afterEach(function () {
     sandbox.restore()
     nock.cleanAll()
+    // Restore test environment
+    if (originalTestEnv !== undefined) {
+      process.env.IS_HEROKU_TEST_ENV = originalTestEnv
+    }
   })
 
   describe('send', function () {

test/unit/analytics-telemetry/telemetry-manager.unit.test.ts

@@ -6,8 +6,20 @@ import {telemetryManager} from '../../../src/lib/analytics-telemetry/telemetry-m
 const isDev = process.env.IS_DEV_ENVIRONMENT === 'true'
 
 describe('telemetry-manager', function () {
+  let originalTestEnv: string | undefined
+
+  beforeEach(function () {
+    // Temporarily enable telemetry for these tests
+    originalTestEnv = process.env.IS_HEROKU_TEST_ENV
+    delete process.env.IS_HEROKU_TEST_ENV
+  })
+
   afterEach(function () {
     nock.cleanAll()
+    // Restore test environment
+    if (originalTestEnv !== undefined) {
+      process.env.IS_HEROKU_TEST_ENV = originalTestEnv
+    }
   })
 
   describe('setupTelemetry', function () {

test/unit/lib/spaces/format.unit.test.ts

@@ -29,7 +29,7 @@ describe('spaces/format', function () {
       expect(hostStatus('under-assessment')).to.eq('\u001B[38;5;43munder-assessment\u001B[39m')
       expect(hostStatus('permanent-failure')).to.eq('\u001B[38;2;255;135;135mpermanent-failure\u001B[39m')
       expect(hostStatus('released-permanent-failure')).to.eq('\u001B[38;2;255;135;135mreleased-permanent-failure\u001B[39m')
-      expect(hostStatus('released')).to.eq('\u001B[90mreleased\u001B[39m')
+      expect(hostStatus('released')).to.eq('\u001B[38;5;248mreleased\u001B[39m')
       expect(hostStatus('foo')).to.eq('foo')
     })
   })