Skip to content

Commit 387fef1

Browse files
hakatashiclaude
andcommitted
test: map-guessr/nmpzのスモークテスト除外を解除
旧PR #1231 (jestベース) ではts-jestがpuppeteer-core(ESM形式)を パースできず起動時にクラッシュしていたため両botを除外していたが、 vitestはESMをネイティブに扱えるためこの制約は存在しない。実際に テストを走らせたところ、map-guessrはそのままpassした。nmpzは GOOGLE_MAPS_API_KEY/CLOUDINARY_URLが未設定だとモジュール読み込み時に 即座にthrowする実装だったため、テスト側でダミー値を設定して対応した。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BuziKVfh8Zx1WC9bRy2MYu
1 parent e517039 commit 387fef1

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

lib/startup.test.ts

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -78,11 +78,6 @@ vi.mock('axios');
7878
vi.mock('dotenv/config', () => ({}));
7979

8080
const EXCLUDED_BOTS = new Set([
81-
// puppeteer (ESM形式のpuppeteer-coreに依存) が実際にヘッドレスブラウザの
82-
// 起動を試みる。Fastify登録クラスのクラッシュ検知という本テストの目的とは
83-
// 無関係な実ブラウザ依存のため対象外とする。
84-
'map-guessr',
85-
'nmpz',
8681
// mahjong は deploy 経由で @octokit/webhooks の Webhooks インスタンスを
8782
// 生成し、起動直後に実GitHub APIへの接続を試みる副作用がある。
8883
'deploy',
@@ -136,6 +131,10 @@ describe.each(testedBots)('%s', (name) => {
136131
const slack = new SlackMock();
137132
process.env.CHANNEL_SANDBOX = slack.fakeChannel;
138133
process.env.CHANNEL_GAMES = slack.fakeChannel;
134+
// nmpz はモジュール読み込み時に GOOGLE_MAPS_API_KEY / CLOUDINARY_URL の
135+
// 存在を無条件にチェックし、無ければ即座にthrowする。
136+
process.env.GOOGLE_MAPS_API_KEY ||= 'dummy-google-maps-api-key';
137+
process.env.CLOUDINARY_URL ||= 'cloudinary://dummy:dummy@dummy';
139138

140139
const plugin = await import(`../${name}`);
141140

0 commit comments

Comments
 (0)