発見の経緯
ERM (Explicit Resource Management) 導入作業の事前調査で見つかったスコープ外項目。
内容
packages/@burger-editor/client/src/__tests__/vr/vr-helper.ts:66-77 の cleanUp() は dialog close + remove + document.body.innerHTML = '' + head 内 style 全削除を一括で行う。各 VR spec が beforeEach(injectCSS) / afterEach(cleanUp) で使っているが、「DOM 全部なぎ倒す」実装は他テストの副作用も消してしまう乱暴な後始末になっている。
対応方針(要検討)
injectCSS() / renderDialog() / renderElement() を Disposable を返す形にし、「作った本人が消す」設計に変更する
- 効果対リスク(VR スクリーンショットに影響するリファクタ)を踏まえ、着手前に影響範囲を精査すること
発見の経緯
ERM (Explicit Resource Management) 導入作業の事前調査で見つかったスコープ外項目。
内容
packages/@burger-editor/client/src/__tests__/vr/vr-helper.ts:66-77のcleanUp()は dialog close + remove +document.body.innerHTML = ''+ head 内 style 全削除を一括で行う。各 VR spec がbeforeEach(injectCSS)/afterEach(cleanUp)で使っているが、「DOM 全部なぎ倒す」実装は他テストの副作用も消してしまう乱暴な後始末になっている。対応方針(要検討)
injectCSS()/renderDialog()/renderElement()をDisposableを返す形にし、「作った本人が消す」設計に変更する