Jestがランダムに落ちる

[syuilo]

解決した方に報奨金を進呈

[tamaina]

ん、どれ

[syuilo]

例えば
https://github.com/misskey-dev/misskey/actions/runs/4906993811/jobs/8761818955

[Nanashia]

    QueryFailedError: relation "muting" does not exist

（再現コード書けてないけど、）BullのRepeat JobとDBの初期化がタイミング的にぶつかると不味そう？

CheckExpiredMutingsProcessorService は5分毎に実行されます。

misskey/packages/backend/src/queue/QueueProcessorService.ts

Lines 160 to 164 in 8f36a80

	this.queueService.systemQueue.add('checkExpiredMutings', {
	}, {
	repeat: { cron: '*/5 * * * *' },
	removeOnComplete: true,
	});

一方でテスト環境ではPostgresはstartServer毎に中身を捨てて毎回migrationをすべて実行するようになっています。

misskey/packages/backend/src/postgres.ts

Lines 221 to 223 in 65a597a

	synchronize: process.env.NODE_ENV === 'test',
	dropSchema: process.env.NODE_ENV === 'test',
	cache: !config.db.disableCache && process.env.NODE_ENV !== 'test' ? { // dbをcloseしても何故かredisのコネクションが内部的に残り続けるようで、テストの際に支障が出るため無効にする(キャッシュも含めてテストしたいため本当は有効にしたいが...)

---

類似パターンとして
https://github.com/misskey-dev/misskey/actions/runs/4915818443/jobs/8778777119

    QueryFailedError: relation "following" does not exist

みたいなのもあるけど、こっちは分針が55の時にぶつかると起きる感じ？

misskey/packages/backend/src/queue/QueueProcessorService.ts

Lines 130 to 134 in 8f36a80

	this.queueService.systemQueue.add('tickCharts', {
	}, {
	repeat: { cron: '55 * * * *' },
	removeOnComplete: true,
	});

[syuilo]

あーーー

[syuilo]

テスト中はそれらのキュー無効化してみるか

[syuilo]

いや通常のテストではjobQueue起動しないようにすでになっている

[syuilo]

どこで呼ばれてるんだ

[syuilo]

あーAccount Moveで呼んでるな

[syuilo]

これ呼ぶ必要あるのかしら

[syuilo]

なさそう

[syuilo]

あーいや

await sleep(1000 * 3); // wait for jobs to finish

がある

[tamaina]

Account Moveはキューを多用する

[Nanashia]

https://github.com/misskey-dev/misskey/actions/runs/4923097917/jobs/8794536924

PASS test/e2e/api-visibility.ts (33.251 s, 740 MB heap size)
PASS test/e2e/move.ts (46.698 s, 850 MB heap size) 　　← ★★★ AccountMoveのテストでjq起動
PASS test/e2e/2fa.ts (27.001 s, 808 MB heap size)
FAIL test/e2e/fetch-resource.ts (29.898 s, 898 MB heap size　　← ★別のテストがFAIL
  ● Webリソース › /@:user/pages/:page › はGETできる。(存在しないIDでも。)

    TypeORMError: Driver not Connected

      36 | 			.innerJoinAndSelect('muting.mutee', 'mutee')
      37 | 			.getMany();
    > 38 |
         | ^
      39 | 		if (expired.length > 0) {
      40 | 			await this.userMutingService.unmute(expired);
      41 | 		}

...
      at SelectQueryBuilder.getMany (../../node_modules/.pnpm/[email protected][email protected][email protected]/src/query-builder/SelectQueryBuilder.ts:1759:36)
      at CheckExpiredMutingsProcessorService.process (src/queue/processors/CheckExpiredMutingsProcessorService.ts:38:56)
      at Queue.<anonymous> (src/queue/SystemQueueProcessorsService.ts:43:96)　← ★★★ なぜかjqが生き残っている？？？
      at Queue.Object.<anonymous>.Queue.processJob (../../node_modules/.pnpm/[email protected]/node_modules/bull/lib/queue.js:1166:22) 

5c54e12 で jq.close()を呼ぶようになりましたが、このJobQueueの実体（queue:systemなどのBull.Queue）はどこからもclose()されていないので、結局停止できていない気がします。
setIntervalやRedisのようにonApplicationShutdownを使って破棄する必要があるかも？（手元で検証中）

[syuilo]

onApplicationShutdownで停止するようにした

[syuilo]

直ったかも？