From a9a2741ec69d42397119e239b73037a27ae35a83 Mon Sep 17 00:00:00 2001 From: Miles Yucht Date: Wed, 15 Nov 2023 13:11:47 +0100 Subject: [PATCH] Skip recipients tests in Azure (#692) ## Changes The recipient activation flow in the SDK doesn't work properly today because the API requires that this request is sent to the host specified in the activation URL of the token generated when creating the recipient. Currently, the SDK only supports using the host from the static configuration. This does work in AWS and GCP today but purely by coincidence, so for now we will stop running these tests in Azure. ## Tests - [ ] `make test` passing - [ ] `make fmt` applied - [ ] relevant integration tests applied Signed-off-by: Tomo Cesnik --- internal/sharing_test.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/internal/sharing_test.go b/internal/sharing_test.go index 8db0ae977..b68bf10f4 100644 --- a/internal/sharing_test.go +++ b/internal/sharing_test.go @@ -55,6 +55,9 @@ func TestUcAccProviders(t *testing.T) { // TODO: remove NoTranspile func TestUcAccRecipientActivationNoTranspile(t *testing.T) { ctx, w := ucwsTest(t) + if w.Config.IsAzure() { + skipf(t)("temporarily skipping this test on Azure until RetrieveToken uses the same host as specified in the activation URL") + } created, err := w.Recipients.Create(ctx, sharing.CreateRecipient{ Name: RandomName("go-sdk-"),