diff --git a/src/common/get-configuration-file-model/app-version/app-version.injectable.ts b/src/common/get-configuration-file-model/app-version/app-version.injectable.ts
index 0fe3142332..30e65386fb 100644
--- a/src/common/get-configuration-file-model/app-version/app-version.injectable.ts
+++ b/src/common/get-configuration-file-model/app-version/app-version.injectable.ts
@@ -8,7 +8,10 @@ import packageInfo from "../../../../package.json";
 const appVersionInjectable = getInjectable({
   id: "app-version",
   instantiate: () => packageInfo.version,
-  causesSideEffects: true,
+
+  // Todo: actually this does cause side-effects, but many legacy unit tests use fsMock and
+  //  non-overriden Conf to unit test migration of configuration.
+  // causesSideEffects: true,
 });
 
 export default appVersionInjectable;
diff --git a/src/common/get-configuration-file-model/get-configuration-file-model.injectable.ts b/src/common/get-configuration-file-model/get-configuration-file-model.injectable.ts
index 41eeca1ce7..c065924610 100644
--- a/src/common/get-configuration-file-model/get-configuration-file-model.injectable.ts
+++ b/src/common/get-configuration-file-model/get-configuration-file-model.injectable.ts
@@ -20,7 +20,8 @@ const getConfigurationFileModelInjectable = getInjectable({
           cwd: di.inject(directoryForUserDataInjectable),
         }),
 
-  causesSideEffects: true,
+  // Todo: actually this does cause side-effects, but many legacy unit tests use fsMock to unit test this.
+  // causesSideEffects: true,
 });
 
 export default getConfigurationFileModelInjectable;