Skip to content

Commit d252705

Browse files
Pigbibicodex
andauthored
fix: reconcile dependency matrix baseline (#196)
Co-authored-by: Codex <noreply@openai.com>
1 parent 407223c commit d252705

2 files changed

Lines changed: 24 additions & 21 deletions

File tree

‎internal_dependency_matrix.json‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@
300300
"path": "pyproject.toml",
301301
"package": "quant-platform-kit",
302302
"source_repo": "QuantPlatformKit",
303-
"ref": "ff09c889ed21e2eb6fcb37f6cdaa159190ec82da"
303+
"ref": "8ba8276948ff71a8cc0a810f98b7437a1311c671"
304304
},
305305
{
306306
"consumer_repo": "UsEquitySnapshotPipelines",
@@ -314,14 +314,14 @@
314314
"path": "pyproject.toml",
315315
"package": "us-equity-strategies",
316316
"source_repo": "UsEquityStrategies",
317-
"ref": "b0b590ce5ac0233a40b8fb957c249cf375a6ff21"
317+
"ref": "337f66bab1f88a7c4181d65ce8598ab182d2396c"
318318
},
319319
{
320320
"consumer_repo": "UsEquitySnapshotPipelines",
321321
"path": "uv.lock",
322322
"package": "quant-platform-kit",
323323
"source_repo": "QuantPlatformKit",
324-
"ref": "ff09c889ed21e2eb6fcb37f6cdaa159190ec82da"
324+
"ref": "8ba8276948ff71a8cc0a810f98b7437a1311c671"
325325
},
326326
{
327327
"consumer_repo": "UsEquitySnapshotPipelines",
@@ -335,21 +335,21 @@
335335
"path": "uv.lock",
336336
"package": "us-equity-strategies",
337337
"source_repo": "UsEquityStrategies",
338-
"ref": "b0b590ce5ac0233a40b8fb957c249cf375a6ff21"
338+
"ref": "337f66bab1f88a7c4181d65ce8598ab182d2396c"
339339
},
340340
{
341341
"consumer_repo": "UsEquityStrategies",
342342
"path": "pyproject.toml",
343343
"package": "quant-platform-kit",
344344
"source_repo": "QuantPlatformKit",
345-
"ref": "92458590a463e7219f0369a3505031ee74414135"
345+
"ref": "8ba8276948ff71a8cc0a810f98b7437a1311c671"
346346
},
347347
{
348348
"consumer_repo": "UsEquityStrategies",
349349
"path": "uv.lock",
350350
"package": "quant-platform-kit",
351351
"source_repo": "QuantPlatformKit",
352-
"ref": "92458590a463e7219f0369a3505031ee74414135"
352+
"ref": "8ba8276948ff71a8cc0a810f98b7437a1311c671"
353353
}
354354
]
355355
}

‎python/tests/test_internal_dependency_matrix.py‎

Lines changed: 18 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -99,28 +99,31 @@ def test_current_matrix_matches_local_workspace(self):
9999
self.assertEqual(report.missing_files, [])
100100
self.assertEqual(report.issues, [])
101101

102-
def test_qpk_migrated_consumers_use_current_canonical_pin(self):
103-
canonical_pin = "92458590a463e7219f0369a3505031ee74414135"
104-
migrated_consumers = {
105-
"BinancePlatform",
106-
"CharlesSchwabPlatform",
107-
"CnEquityStrategies",
108-
"CryptoStrategies",
109-
"FirstradePlatform",
110-
"HkEquityStrategies",
111-
"InteractiveBrokersPlatform",
112-
"LongBridgePlatform",
113-
"UsEquityStrategies",
102+
def test_qpk_migrated_consumers_use_current_baseline_pins(self):
103+
expected_pins_by_consumer = {
104+
"BinancePlatform": "92458590a463e7219f0369a3505031ee74414135",
105+
"CharlesSchwabPlatform": "92458590a463e7219f0369a3505031ee74414135",
106+
"CnEquityStrategies": "92458590a463e7219f0369a3505031ee74414135",
107+
"CryptoStrategies": "92458590a463e7219f0369a3505031ee74414135",
108+
"FirstradePlatform": "92458590a463e7219f0369a3505031ee74414135",
109+
"HkEquityStrategies": "92458590a463e7219f0369a3505031ee74414135",
110+
"InteractiveBrokersPlatform": "92458590a463e7219f0369a3505031ee74414135",
111+
"LongBridgePlatform": "92458590a463e7219f0369a3505031ee74414135",
112+
"UsEquityStrategies": "8ba8276948ff71a8cc0a810f98b7437a1311c671",
114113
}
115114
matrix_pins = check_internal_dependency_matrix.load_matrix(ROOT / "internal_dependency_matrix.json")
116115
refs = {
117116
(pin.consumer_repo, pin.path): pin.ref
118117
for pin in matrix_pins
119-
if pin.consumer_repo in migrated_consumers and pin.source_repo == "QuantPlatformKit"
118+
if pin.consumer_repo in expected_pins_by_consumer and pin.source_repo == "QuantPlatformKit"
119+
}
120+
expected_refs = {
121+
(consumer_repo, path): ref
122+
for consumer_repo, ref in expected_pins_by_consumer.items()
123+
for path in ("pyproject.toml", "uv.lock")
120124
}
121125

122-
self.assertEqual(len(refs), len(migrated_consumers) * 2)
123-
self.assertEqual(set(refs.values()), {canonical_pin})
126+
self.assertEqual(refs, expected_refs)
124127

125128
def test_require_consumer_files_treats_missing_paths_as_issues(self):
126129
projects_root = self._make_projects_root({})

0 commit comments

Comments
 (0)