Skip to content

Commit

Permalink
fix issue with missing live test parameter
Browse files Browse the repository at this point in the history
  • Loading branch information
xirzec committed Jun 22, 2023
1 parent 136d49e commit 66b7091
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ describe("Get user", function () {

it("should obtain user data", async function () {
// If using a test app, it needs to be the oid.
const userId = env.AZURE_CLIENT_OID ?? env.AZURE_CLIENT_ID ?? "";
const userId = env.AZURE_CLIENT_OID;
if (!userId) {
this.skip();
}
Expand Down
1 change: 1 addition & 0 deletions sdk/confidentialledger/test-resources.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ resource baseName_resource 'Microsoft.ConfidentialLedger/ledgers@2020-12-01-prev
}
}

output CONFIDENTIALLEDGER_CLIENT_OID string = testApplicationOid
output LEDGER_NAME string = ledgerName
output LEDGERURI string = ledgerUri
output IDENTITY_SERVICE_URL string = 'https://identity.confidential-ledger.core.azure.com'
1 change: 1 addition & 0 deletions sdk/confidentialledger/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ stages:
EnvVars:
AZURE_CLIENT_ID: $(CONFIDENTIALLEDGER_CLIENT_ID)
AZURE_TENANT_ID: $(CONFIDENTIALLEDGER_TENANT_ID)
AZURE_CLIENT_OID: $(CONFIDENTIALLEDGER_CLIENT_OID)
AZURE_CLIENT_SECRET: $(CONFIDENTIALLEDGER_CLIENT_SECRET)
LEDGER_URI: $(LEDGERURI)
LEDGER_NAME: $(LEDGER_NAME)
Expand Down

0 comments on commit 66b7091

Please sign in to comment.