テストが実際の実装をテストしていない

[khirotaka]

テストが実際の実装をテストしていない

parseBooleanEnv 関数をテストファイル内で再実装しており、実際の config.ts の実装をテストしていません。

理由:

• 実際の config.ts の parseBooleanEnv にバグがあっても、このテストは通過してしまう
• テストの価値が低く、偽陽性（false positive）を生む可能性がある
• コードカバレッジが実際の実装を反映しない

修正案:

オプション1: parseBooleanEnv を別ファイルにエクスポートしてテスト

// src/utils/env.ts (新規ファイル)
export function parseBooleanEnv(...) { ... }

// tests/unit/env.test.ts
import { parseBooleanEnv } from '../../src/utils/env.ts';

オプション2: config オブジェクトを通じた統合テスト

Deno.test({
  name: 'config.auth.enabled - Parses AUTH_ENABLED correctly',
  fn: () => {
    // 環境変数を設定してプロセスを再起動する必要があるため、
    // この方法は実用的ではない可能性がある
  },
});

オプション3: このテストファイルを削除

• 現状では実装の検証になっていないため、削除しても問題ない
• parseBooleanEnv の動作は他の統合テストで間接的に検証される

Originally posted by @claude[bot] in #149 (comment)