Skip to content

Commit

Permalink
Fixed management API version mismatch (#2385)
Browse files Browse the repository at this point in the history
* Fixed management API version mismatch

* Changed test settings to use the new endpoint

* Changed test data jsons
  • Loading branch information
jsorohova committed Feb 8, 2024
1 parent 1f68f15 commit 9fe526d
Show file tree
Hide file tree
Showing 3 changed files with 31 additions and 23 deletions.
4 changes: 2 additions & 2 deletions src/services/tenantService.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@ export class TenantService {
* Returns tenant settings.
*/
public async getSettings(): Promise<TenantSettings> {
const result = await this.mapiClient.get("/tenant/settings?api-version=2019-12-01", [await this.mapiClient.getPortalHeader("getSettings")]);
return result && result["settings"];
const result = await this.mapiClient.get("/settings/public", [await this.mapiClient.getPortalHeader("getSettings")]);
return result && result["properties"] && result["properties"]["settings"];
}

/**
Expand Down
26 changes: 15 additions & 11 deletions tests/mocks/defaultStaticData.json
Original file line number Diff line number Diff line change
Expand Up @@ -1562,7 +1562,7 @@
"count": 1
}
},
"https://contoso.management.azure-api.net/subscriptions/sid/resourceGroups/rgid/providers/Microsoft.ApiManagement/service/sid/tenant/settings": {
"https://contoso.management.azure-api.net/subscriptions/sid/resourceGroups/rgid/providers/Microsoft.ApiManagement/service/sid/settings/public": {
"headers": [
{
"name": "content-type",
Expand All @@ -1572,16 +1572,20 @@
"statusCode": 200,
"statusText": "OK",
"body": {
"id": "/settings/public",
"settings": {
"CustomPortalSettings.UserRegistrationTerms": "",
"CustomPortalSettings.UserRegistrationTermsEnabled": "False",
"CustomPortalSettings.UserRegistrationTermsConsentRequired": "False",
"CustomPortalSettings.DelegationEnabled": "False",
"CustomPortalSettings.DelegationUrl": "",
"CustomPortalSettings.DelegatedSubscriptionEnabled": "False",
"CustomPortalSettings.RequireUserSigninEnabled": "True"
}
"id": "/subscriptions/sid/resourceGroups/rgid/providers/Microsoft.ApiManagement/service/sid/settings/public",
"type": "Microsoft.ApiManagement/service/settings",
"name": "public",
"properties": {
"settings": {
"CustomPortalSettings.UserRegistrationTerms": "",
"CustomPortalSettings.UserRegistrationTermsEnabled": "False",
"CustomPortalSettings.UserRegistrationTermsConsentRequired": "False",
"CustomPortalSettings.DelegationEnabled": "False",
"CustomPortalSettings.DelegationUrl": "",
"CustomPortalSettings.DelegatedSubscriptionEnabled": "False",
"CustomPortalSettings.RequireUserSigninEnabled": "True"
}
}
}
},
"https://contoso.management.azure-api.net/subscriptions/sid/resourceGroups/rgid/providers/Microsoft.ApiManagement/service/sid/products/unlimited": {
Expand Down
24 changes: 14 additions & 10 deletions tests/mocks/mockServerData.json
Original file line number Diff line number Diff line change
Expand Up @@ -93,18 +93,22 @@
"count": 0
}
},
"/subscriptions/sid/resourceGroups/rgid/providers/Microsoft.ApiManagement/service/sid/tenant/settings":{
"/subscriptions/sid/resourceGroups/rgid/providers/Microsoft.ApiManagement/service/sid/settings/public":{
"statusCode": 200,
"body": {

"id": "/settings/public",
"settings": {
"CustomPortalSettings.UserRegistrationTerms": null,
"CustomPortalSettings.UserRegistrationTermsEnabled": "False",
"CustomPortalSettings.UserRegistrationTermsConsentRequired": "False",
"CustomPortalSettings.DelegationEnabled": "False",
"CustomPortalSettings.DelegationUrl": "",
"CustomPortalSettings.DelegatedSubscriptionEnabled": "False"
"id": "/subscriptions/sid/resourceGroups/rgid/providers/Microsoft.ApiManagement/service/sid/settings/public",
"type": "Microsoft.ApiManagement/service/settings",
"name": "public",
"properties": {
"settings": {
"CustomPortalSettings.UserRegistrationTerms": "",
"CustomPortalSettings.UserRegistrationTermsEnabled": "False",
"CustomPortalSettings.UserRegistrationTermsConsentRequired": "False",
"CustomPortalSettings.DelegationEnabled": "False",
"CustomPortalSettings.DelegationUrl": "",
"CustomPortalSettings.DelegatedSubscriptionEnabled": "False",
"CustomPortalSettings.RequireUserSigninEnabled": "True"
}
}
}
},
Expand Down

0 comments on commit 9fe526d

Please sign in to comment.