CSMAP Read-only directory fix for itwinjs-backlog 1118

[tcobbs-bentley]

Fix was suggested by Alain Robert
See https://github.com/iTwin/itwinjs-backlog/issues/1118#issuecomment-2115811221

[tcobbs-bentley]

I will cherry-pick this into release/4.x once merged.

[aruniverse]

@Mergifyio backport release/4.x

[mergify]

backport release/4.x

🟠 Waiting for conditions to match

• merged [📌 backport requirement]

[AlainRobertAtBentley]

I have made all tests run correctly on a local build. The failing tests must be related to something in the pipeline. I also noted the presence of a few windows slashes in tests paths. I think these should be changed from \ to / specifically at:

imodel-native/iModelCore/GeoCoord/Tests/NonPublished/GeoCoordTestCommon.cpp

Line 31 in 64949e5

path.AppendToPath(L"Assets\\DgnGeoCoord");

[tcobbs-bentley]

I have made all tests run correctly on a local build. The failing tests must be related to something in the pipeline. I also noted the presence of a few windows slashes in tests paths. I think these should be changed from \ to / specifically at:

imodel-native/iModelCore/GeoCoord/Tests/NonPublished/GeoCoordTestCommon.cpp

Line 31 in 64949e5

path.AppendToPath(L"Assets\\DgnGeoCoord");

As far as I can tell, BeFileName automatically fixes paths to be correct on Windows and non-Windows. (On Windows, it converts all forward slashes to backslashes, and on non-Windows, it converts all backslashes to forward slashes.) There are four total places in the GeoCoord C++ code that use Assets\\, and none that use Assets/, so it seems like leaving it alone is the best thing here.