「フォローしているユーザーのファイル付きノートのみ含まれる」e2eテストがタイムアウトする

[sakuhanight]

💡 Summary

e2eテストで「フォローしているユーザーのファイル付きノートのみ含まれる」が正常であってもマシンスペックなどの事情によりタイムアウト(10s)する。

🥰 Expected Behavior

正常な場合にタイムアウトしない

🤬 Actual Behavior

タイムアウトする

📝 Steps to Reproduce

1. マシンスペックやIOの弱い環境でe2eテストを実行する

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey:

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

• Yes, I will patch the bug myself and send a pull request

[sakuhanight]

そもそもなぜタイムアウトするほど時間が掛かっているかの調査をするのが先かもしれません。

[anatawa12]

#15282 のようにactionsでも落ちるようになった

[sakuhanight]

Issue起票当時もActionsでも落ちるときがあった記憶

[samunohito]

（#15282 の文脈から）

ざっくりとこんな感じのコードを書いてログ出力の時間を計測してみました。

const logger = new MyCustomLogger({ disableQueryTruncation: true });

test('logQuery', () => {
	const query = 'metaテーブルをselectする長いクエリ'

	const start = new Date().getTime();
	for (let i = 0; i < 1000; i++) {
		logger.logQuery(query, []);
	}
	const end = new Date().getTime();

	console.log(`Time: ${end - start}ms`);
});

• disableQueryTruncation === true
  • truncateSqlしてからhighlightSql -> 2815ms
  • highlightSqlしてからtruncateSql -> 2787ms
• disableQueryTruncation === false
  • truncateSqlしてからhighlightSql -> 89ms
  • highlightSqlしてからtruncateSql -> 2634ms

「highlightSqlしてからtruncateSql」は今までのロジックで、 #15268 以降「truncateSqlしてからhighlightSql」に変わっています。
上記から、ログの出力はタイムアウトとは直接関係ないと考えます

[samunohito]

（とりあえず、 #15283 を暫定対処として取り込んでしまいたい気持ち）

[anatawa12]

logのハイライト関係ないのか...謎だな...

とりあえず取り込んじゃっていいと思う