refactor: Added ability to inject custom directory for DataStoreFile

[asmeili]

Summary

• Extended DataStoreFile to allow for injection of custom FileManager.SearchPathDirectory
• Extended DefaultEventDispatcher to allow for injection of custom FileManager.SearchPathDirectory for DataStoreType.file

Users have been raising the issue that they see these technical files inside of their Files app, because the directory for DataStoreFile defaults to .documentsDirectory, which is visible to users. This change allows us to update the directory to .applicationSupportDirectory in DefaultEventDispatcher using DataStoreType.file(directory: .applicationSupportDirectory) and in DefaultDatafileHandler by overriding open func createDataStore(sdkKey: String) -> OPTDataStore.

Test plan

• Added a new test to verify the custom directory is used by DataStoreFile

Issues

• None

[asmeili]

Hi @muzahidul-opti, could you please review this PR and let me know if any changes are required?

[100hp4ever]

lgtm

[muzahidul-opti]

Hi @muzahidul-opti, could you please review this PR and let me know if any changes are required?

Sure, we will review it.

[asmeili]

Hi @muzahidul-opti, are there any updates on this?

[muzahidul-opti]

Looks good to me, CI failing could you please fix it?

[asmeili]

Hi @muzahidul-opti, I fixed the build issue. Could you approve the CI runs to verify?

[asmeili]

Hi @raju-opti, could you take a look at this PR and approve the CI runs?

[coveralls]

Coverage is 93.864% — asmeili:artesmichael/make-data-store-more-flexible into optimizely:master. No base build found for optimizely:master.

[asmeili]

Thanks for the review! I've merged the latest master into the branch, which should take care of the testThreadSafe failure (it's in LruCacheTests, unrelated to this change, looks flaky).

The remaining failures don't seem fixable from my side as a fork PR.

Could you check again @muzahidul-opti?