opticedge-cloud-utils 1.1.23 → 1.1.24

package/dist/env.d.ts

@@ -1 +1 @@
-export declare function getEnv(name: string, fallback?: any): any;
+export declare function requireEnv(name: string, defaultValue?: string): string;

package/dist/env.js

@@ -1,13 +1,13 @@
 "use strict";
-/* eslint-disable @typescript-eslint/no-explicit-any */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getEnv = getEnv;
-function getEnv(name, fallback) {
+exports.requireEnv = requireEnv;
+function requireEnv(name, defaultValue) {
     const value = process.env[name];
-    if (!value) {
-        if (fallback !== undefined)
-            return fallback;
-        throw new Error(`Missing env var ${name}`);
+    if (value === undefined || value === '') {
+        if (defaultValue !== undefined) {
+            return defaultValue;
+        }
+        throw new Error(`${name} env var must be set`);
     }
     return value;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opticedge-cloud-utils",
-  "version": "1.1.23",
+  "version": "1.1.24",
   "description": "Common utilities for cloud functions",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/env.ts

@@ -1,10 +1,12 @@
-/* eslint-disable @typescript-eslint/no-explicit-any */
-
-export function getEnv(name: string, fallback?: any): any {
+export function requireEnv(name: string, defaultValue?: string): string {
   const value = process.env[name]
-  if (!value) {
-    if (fallback !== undefined) return fallback
-    throw new Error(`Missing env var ${name}`)
+
+  if (value === undefined || value === '') {
+    if (defaultValue !== undefined) {
+      return defaultValue
+    }
+    throw new Error(`${name} env var must be set`)
   }
+
   return value
-}
+}

package/tests/env.test.ts

@@ -1,18 +1,65 @@
-import { getEnv } from '../src/env'
+import { requireEnv } from '../src/env'
 
-describe('getEnv', () => {
-  it('returns the value of an existing env var', () => {
-    process.env.TEST_KEY = 'test-value'
-    expect(getEnv('TEST_KEY')).toBe('test-value')
+describe('requireEnv', () => {
+  const ORIGINAL_ENV = process.env
+
+  beforeEach(() => {
+    jest.resetModules()
+    process.env = { ...ORIGINAL_ENV }
+  })
+
+  afterAll(() => {
+    process.env = ORIGINAL_ENV
+  })
+
+  // ============================================================
+  // Success cases
+  // ============================================================
+
+  it('returns env value when set', () => {
+    process.env.TEST_VAR = 'hello'
+    expect(requireEnv('TEST_VAR')).toBe('hello')
+  })
+
+  it('returns "0" correctly (should not treat as missing)', () => {
+    process.env.TEST_VAR = '0'
+    expect(requireEnv('TEST_VAR')).toBe('0')
   })
 
-  it('returns fallback if env var is not set', () => {
-    delete process.env.OPTIONAL_KEY
-    expect(getEnv('OPTIONAL_KEY', 'default')).toBe('default')
+  it('returns "false" correctly (should not treat as missing)', () => {
+    process.env.TEST_VAR = 'false'
+    expect(requireEnv('TEST_VAR')).toBe('false')
+  })
+
+  // ============================================================
+  // Default handling
+  // ============================================================
+
+  it('returns default when env is undefined', () => {
+    delete process.env.TEST_VAR
+    expect(requireEnv('TEST_VAR', 'fallback')).toBe('fallback')
+  })
+
+  it('returns default when env is empty string', () => {
+    process.env.TEST_VAR = ''
+    expect(requireEnv('TEST_VAR', 'fallback')).toBe('fallback')
+  })
+
+  // ============================================================
+  // Error cases
+  // ============================================================
+
+  it('throws when env is undefined and no default provided', () => {
+    delete process.env.TEST_VAR
+    expect(() => requireEnv('TEST_VAR')).toThrow(
+      'TEST_VAR env var must be set'
+    )
   })
 
-  it('throws error if env var is not set and no fallback provided', () => {
-    delete process.env.MISSING_KEY
-    expect(() => getEnv('MISSING_KEY')).toThrow('Missing env var MISSING_KEY')
+  it('throws when env is empty string and no default provided', () => {
+    process.env.TEST_VAR = ''
+    expect(() => requireEnv('TEST_VAR')).toThrow(
+      'TEST_VAR env var must be set'
+    )
   })
-})
+})