test(main): ダウンロード側の関門にテストを足す

[hal-shu-sato]

内容

install 側の同型の関門(resolveInside / safeRemove)は install.test.ts と apmPath.test.ts で覆われているのに、ダウンロード側は 1 件も覆われていませんでした。

download.ts が守っているのは 2 つです。

// subDir 経由でデータフォルダ外に書き込ませない
if (!isParent(dataDir, retFilePath)) { … return undefined; }

// 現状ファイル名を decode する箇所は無いが、将来どこかで decode されると
// パス区切りに化ける値(%2f・%5c)は入口で拒否しておく
if (/%2f|%5c/i.test(opt.filename)) { … return undefined; }

tempFile.ts の existsTempFile も同じ基点で守っています。

テストの作り

実際のダウンロードは行いません。url にローカルパスを渡してコピー経路(url が http で始まらない側)だけを通します。electron-dl の download は呼ばれたら例外を投げるモックにしてあるので、「ネットワーク経路へ入っていないこと」も同時に保証されます。

8 件:

• データフォルダの中へ保存する
• subDir でデータフォルダの外へ出る指定を拒む
• ファイル名にエンコードされたパス区切りがあれば拒む(%2f / %5c / 大文字 %2F)
• アーカイブ拡張子は archive サブディレクトリへ分ける
• keyText を渡すとハッシュを前置した名前になる
• loadCache は既存ファイルがあればコピーせずそのパスを返す(中身を見ないことも明示)
• existsTempFile がデータフォルダの中を解決する / 外へ出るパスを拒む

効いていることの確認

関門を実際に外して、対応するテストが落ちることを確認しました。

脱出先は userData の親を掃除対象の root に含める形にしてあります。os.tmpdir() 直下に置くと、関門が壊れたときに実ファイルが残り、次回以降のテストがそれを拾って誤って落ちます(実際に一度踏みました)。

挙動は変えていません。

---

調査と文面の作成に Claude Code を使用しています。