Skip to content

Commit 64d9067

Browse files
committed
test(uat): assert aws lane train phase directly
The aggregate runsJourney("training", "train") check passes on gcp's enabled train step even if the aws lane lost the phase, so assert awsLane.phases["train"] directly to prove the aws pipeline runs it. Signed-off-by: Nathan Hensley <nhensley@nvidia.com>
1 parent 9ac0b5d commit 64d9067

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

tools/coverage/build_test.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,12 @@ func TestScanWiredUATFromRegistry(t *testing.T) {
285285
t.Errorf("aws lane intents = %v, want the union to include %q", awsLane.intents, intent)
286286
}
287287
}
288+
// And the aws pipeline actually runs the train phase — asserted on the lane
289+
// itself, since the aggregate runsJourney("training", "train") below would
290+
// pass on gcp's enabled train step even if the aws lane lost the phase.
291+
if !awsLane.phases["train"] {
292+
t.Errorf("aws lane phases = %v, want the enabled train phase", awsLane.phases)
293+
}
288294

289295
// Both intents run somewhere, so both stacks are stood up nightly...
290296
for _, intent := range []string{"training", "inference"} {

0 commit comments

Comments
 (0)