fix: correct TTL field name mismatch in persistent cache config

The Go test harness sends cache TTL as 'ttl' (in seconds), but the C++
data model expected 'ttlMs' (in milliseconds). This meant the TTL value
was never deserialized, and the default 5-minute cache TTL was always
used regardless of what the test requested.

- Rename ConfigPersistentCache::ttlMs to ttl to match JSON field name
- Change CacheRefresh from milliseconds to seconds to match harness units

Co-Authored-By: rlamb@launchdarkly.com <kingdewman@gmail.com>

Author: devin-ai-integration[bot]

contract-tests/data-model/include/data_model/data_model.hpp

@@ -152,12 +152,12 @@ NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE_WITH_DEFAULT(ConfigWrapper, name, version);
 
 struct ConfigPersistentCache {
     std::string mode;  // "off", "ttl", "infinite"
-    std::optional<int> ttlMs;
+    std::optional<int> ttl;  // TTL in seconds (sent by test harness as "ttl")
 };
 
 NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE_WITH_DEFAULT(ConfigPersistentCache,
                                                 mode,
-                                                ttlMs);
+                                                ttl);
 
 struct ConfigPersistentStore {
     std::string type;  // "redis", "consul", "dynamodb"

contract-tests/server-contract-tests/src/entity_manager.cpp

@@ -181,14 +181,14 @@ std::optional<std::string> EntityManager::create(ConfigParams const& in) {
             // Configure cache mode
             // Default is 5 minutes, but contract tests may specify:
             // - "off": disable caching (fetch from DB every time)
-            // - "ttl": custom TTL in milliseconds
+            // - "ttl": custom TTL in seconds (test harness sends seconds)
             // - "infinite": never expire cached items
             if (in.persistentDataStore->cache.mode == "off") {
                 lazy_load.CacheRefresh(std::chrono::seconds(0));
             } else if (in.persistentDataStore->cache.mode == "ttl") {
-                if (in.persistentDataStore->cache.ttlMs) {
-                    lazy_load.CacheRefresh(std::chrono::milliseconds(
-                        *in.persistentDataStore->cache.ttlMs));
+                if (in.persistentDataStore->cache.ttl) {
+                    lazy_load.CacheRefresh(std::chrono::seconds(
+                        *in.persistentDataStore->cache.ttl));
                 }
             } else if (in.persistentDataStore->cache.mode == "infinite") {
                 // Use a very large TTL to effectively never expire