From e1780f322a2674f97e09817eeb9c4f17f5e60ecc Mon Sep 17 00:00:00 2001 From: Doug Date: Tue, 13 Oct 2026 15:24:26 +0100 Subject: [PATCH] Remove un-necessary plan file. --- TIDY.md | 5 ----- 1 file changed, 5 deletions(-) delete mode 100644 TIDY.md diff --git a/TIDY.md b/TIDY.md deleted file mode 100644 index bd7944744..000000000 --- a/TIDY.md +++ /dev/null @@ -1,5 +0,0 @@ -# Tidy - -Follow-ups that aren't worth their own PR yet. Do each item in one go, not piecemeal. - -- **Enterprise hook tests should use `EnterpriseESSServiceMock(configuration:)` instead of the on-disk cache.** Today `EnterpriseCreateRoomScreenHookTests`, `EnterpriseSecurityAndPrivacyScreenHookTests`, `EnterpriseUserSessionHookTests` and parts of `EnterpriseRemoteSettingsHookTests` seed a real `EnterpriseESSService` by writing an `EnterpriseESSConfiguration` to `URL.cachedESSConfigurationFile(for:)`, and each wipes `.enterpriseApplicationSupportBaseDirectory` in `init`. Passing the mock removes the disk dependency and the shared-directory coupling. Switch all of them together so the pattern stays consistent, and leave `EnterpriseESSServiceTests` alone since the cache is what it tests.