New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Identity] Testing service connections on pipeline #29451
Conversation
d9a92c2
to
c78dbdf
Compare
API change check APIView has identified API level changes in this PR and created following API reviews. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few things I noticed but otherwise LGTM
sdk/identity/identity/test/public/node/azurePipelinesServiceConnection.spec.ts
Outdated
Show resolved
Hide resolved
describe("AzurePipelinesServiceConnectionCredential", function () { | ||
const scope = "https://vault.azure.net/.default"; | ||
const tenantId = env.IDENTITY_SP_TENANT_ID || env.AZURE_TENANT_ID!; | ||
const tenantId = "72f988bf-86f1-41af-91ab-2d7cd011db47"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when we record these tests, you'd want any env.
calls to be made after the recorder is initialized (so like in the test body or a beforeEach
block after creating the recorder. Just FYI. I could be wrong but I remember this being a problem for me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True. Though with this particular test, it's difficult to have recordings, since I cannot run locally :(
…nnection.spec.ts Co-authored-by: Maor Leger <maorleger@users.noreply.github.com>
Response = {"$id":"1","innerException":null,"message":"The requested version "7.1" of the resource is under preview. The -preview flag must be supplied in the api-version for such requests. For example: "7.1-preview"","typeName":"Microsoft.VisualStudio.Services.WebApi.VssInvalidPreviewVersionException, Microsoft.VisualStudio.Services.WebApi","typeKey":"VssInvalidPreviewVersionException","errorCode":0,"eventId":3000}