Skip to content

Commit 7d1ce8f

Browse files
committed
ENH: Register OOC format from DataIOCollection ctor; fix in-core Data Format tests
* DataIOCollection's constructor registers the OOC format via SimplnxOoc::registerIOManager under SIMPLNX_USE_OOC, so getManager("HDF5-OOC") resolves in the compile-time-switch OOC build. * IOFormat: guard the in-core large-data-format preference tests to #ifndef SIMPLNX_USE_OOC (the OOC-build defaults are covered by SimplnxOoc's DataFormatPreferenceTest) and update the "not configured" assertion to the seeded k_InMemoryFormat default. Signed-off-by: Joey Kleingers <joey.kleingers@bluequartz.net>
1 parent 427e40d commit 7d1ce8f

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

test/IOFormat.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,8 @@ TEST_CASE("Data Format: Not configured defaults to InMemory store", "[IOTest][Da
5151
REQUIRE(prefs->largeDataFormat() == Preferences::k_InMemoryFormat);
5252
REQUIRE_FALSE(prefs->useOocData());
5353

54-
// The resolver-aware CreateDataStore should produce an InMemory store under
55-
// these conditions because the registered resolver (none in the in-core build)
56-
// returns "" → default in-memory.
54+
// The resolver-aware CreateDataStore should produce an InMemory store: with no
55+
// OOC compiled in, the resolver is bypassed and storage is always in-memory.
5756
DataStructure ds;
5857
DataPath dp({"TestArray"});
5958
auto store = DataStoreUtilities::CreateDataStore<float32>(ds, dp, {100, 100, 100}, {1}, IDataAction::Mode::Execute);

0 commit comments

Comments
 (0)