From 1725ffc799bdcc018a4f3abb0ee8eb0b46913ceb Mon Sep 17 00:00:00 2001 From: mesh-admin Date: Tue, 13 Oct 2026 11:49:30 +0000 Subject: [PATCH] licence manager: a login's adoption is said beside the bindings it made, not first (issue 625) The bindings a login makes are now announced as licence.bound before the adoption's own event, so the test asks for each event rather than for adoption to come first. --- .../cmd/claude-licence-manager/manager_test.go | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/modules/claude-licence-manager/cmd/claude-licence-manager/manager_test.go b/modules/claude-licence-manager/cmd/claude-licence-manager/manager_test.go index 287c6456..4f8103fb 100644 --- a/modules/claude-licence-manager/cmd/claude-licence-manager/manager_test.go +++ b/modules/claude-licence-manager/cmd/claude-licence-manager/manager_test.go @@ -132,8 +132,13 @@ func TestAManagerWithNoLicenceAdoptsTheNewestLoginThatRefreshesAndNeverExchanges if len(bs) != 2 || mm.state["laptop"].Licence != licence1 || mm.state["server"].Licence != licence1 { t.Fatalf("bindings %v, state %v", bs, mm.state) } - if !strings.HasPrefix(strings.Join(mm.events, "|"), "licence.adopted") { - t.Fatalf("events %v", mm.events) + // The adoption is said, and so is each binding it made (novox/hq issue 625). + events := strings.Join(mm.events, "|") + for _, want := range []string{"licence.adopted ", `licence.bound {"by":"claude-licence-manager","consumer":"laptop"`, + `licence.bound {"by":"claude-licence-manager","consumer":"server"`} { + if !strings.Contains(events, want) { + t.Fatalf("events lack %s: %v", want, mm.events) + } } }