From 211b81bbd558ea38b331613b4466e433e0b136a3 Mon Sep 17 00:00:00 2001 From: Astrid Gealer Date: Tue, 20 Oct 2026 16:22:52 +0100 Subject: [PATCH] chore(test): point Azure RI-exchange credential test comments at cli#932 The Azure credential-resolution tests in handler_ri_exchange_test.go cite issue #871, but that work is LeanerCloud/cloud-commitments-cli#932 (fixed by cli#934, whose branch name carried the wrong number). cli#871 is the RI Exchange global-filter design question, so the other #871 references in the file and in handler_ri_exchange.go are correct and stay as they are. Use the full repo-qualified form so the reference cannot be read as this repo's #932. Comment-only change; no behavior change. Closes #26 Co-Authored-By: Claude Opus 5.5 --- internal/api/handler_ri_exchange_test.go | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/internal/api/handler_ri_exchange_test.go b/internal/api/handler_ri_exchange_test.go index 499923dd2..c0ed9f790 100644 --- a/internal/api/handler_ri_exchange_test.go +++ b/internal/api/handler_ri_exchange_test.go @@ -1422,7 +1422,7 @@ func TestListExchangeableAzureRIs_SubscriptionIDFiltersToOwnRows(t *testing.T) { } } -// --- Azure credential-resolution path tests (issue #871) --- +// --- Azure credential-resolution path tests (issue LeanerCloud/cloud-commitments-cli#932) --- // // These tests exercise the production path of buildAzureExchangeClient, which // was previously hardcoded to azidentity.NewDefaultAzureCredential. The fix @@ -1440,8 +1440,9 @@ func TestListExchangeableAzureRIs_SubscriptionIDFiltersToOwnRows(t *testing.T) { // TestListExchangeableAzureRIs_NoAzureAccountRegistered verifies that when no // Azure CloudAccount is registered for the requested subscription, the handler // returns an empty reservations list (graceful empty state) rather than a 500. -// This is the fix for issue #871: Lambda has no ambient Azure identity so the -// old DefaultAzureCredential call failed, producing an opaque 500. +// This is the fix for issue LeanerCloud/cloud-commitments-cli#932: Lambda has no +// ambient Azure identity so the old DefaultAzureCredential call failed, +// producing an opaque 500. func TestListExchangeableAzureRIs_NoAzureAccountRegistered(t *testing.T) { mockStore := &MockConfigStore{} // GetCloudAccountByExternalID returns (nil, nil) — no account for this sub.