pk/atst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tenant admin pw reset use secrets lib, csp calls do not return the result data class

Browse Source
This commit is contained in:
Philip Kalinsky 2020-03-06 10:50:06 -05:00
parent 8734a35bf8
commit f9a45e68bb
4 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
atat/domain/csp/cloud/azure_cloud_provider.py
View File

@ -1,5 +1,4 @@
import json import json
import random
import string import string
from secrets import token_hex, token_urlsafe from secrets import token_hex, token_urlsafe
from uuid import uuid4 from uuid import uuid4
@ -1603,8 +1602,7 @@ class AzureCloudProvider(CloudProviderInterface):
"passwordProfile": { "passwordProfile": {
"forceChangePasswordNextSignIn": True, "forceChangePasswordNextSignIn": True,
"forceChangePasswordNextSignInWithMfa": False, "forceChangePasswordNextSignInWithMfa": False,
"password": payload.new_password "password": payload.new_password or token_urlsafe(16),
or "".join(random.choice(string.ascii_letters) for i in range(15)),
} }
} }

2
atat/domain/csp/cloud/mock_cloud_provider.py
View File

@ -52,7 +52,6 @@ from .models import (
TenantAdminOwnershipCSPPayload, TenantAdminOwnershipCSPPayload,
TenantAdminOwnershipCSPResult, TenantAdminOwnershipCSPResult,
TenantAdminCredentialResetCSPPayload, TenantAdminCredentialResetCSPPayload,
TenantAdminCredentialResetCSPResult,
TenantCSPPayload, TenantCSPPayload,
TenantCSPResult, TenantCSPResult,
TenantPrincipalAppCSPPayload, TenantPrincipalAppCSPPayload,
@ -343,7 +342,6 @@ class MockCloudProvider(CloudProviderInterface):
self._maybe_raise(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION) self._maybe_raise(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
self._maybe_raise(self.SERVER_FAILURE_PCT, self.SERVER_EXCEPTION) self._maybe_raise(self.SERVER_FAILURE_PCT, self.SERVER_EXCEPTION)
self._maybe_raise(self.UNAUTHORIZED_RATE, self.AUTHORIZATION_EXCEPTION) self._maybe_raise(self.UNAUTHORIZED_RATE, self.AUTHORIZATION_EXCEPTION)
return TenantAdminCredentialResetCSPResult()
def create_tenant_principal_ownership( def create_tenant_principal_ownership(
self, payload: TenantPrincipalOwnershipCSPPayload self, payload: TenantPrincipalOwnershipCSPPayload

1
tests/domain/cloud/test_azure_csp.py
View File

@ -57,7 +57,6 @@ from atat.domain.csp.cloud.models import (
TenantAdminOwnershipCSPPayload, TenantAdminOwnershipCSPPayload,
TenantAdminOwnershipCSPResult, TenantAdminOwnershipCSPResult,
TenantAdminCredentialResetCSPPayload, TenantAdminCredentialResetCSPPayload,
TenantAdminCredentialResetCSPResult,
TenantCSPPayload, TenantCSPPayload,
TenantCSPResult, TenantCSPResult,
TenantPrincipalAppCSPPayload, TenantPrincipalAppCSPPayload,

2
tests/domain/test_portfolio_state_machine.py
View File

@ -305,9 +305,9 @@ def test_fsm_transition_start(
FSMStates.INITIAL_MGMT_GROUP_CREATED, FSMStates.INITIAL_MGMT_GROUP_CREATED,
FSMStates.INITIAL_MGMT_GROUP_VERIFICATION_CREATED, FSMStates.INITIAL_MGMT_GROUP_VERIFICATION_CREATED,
FSMStates.TENANT_ADMIN_OWNERSHIP_CREATED, FSMStates.TENANT_ADMIN_OWNERSHIP_CREATED,
FSMStates.TENANT_ADMIN_CREDENTIAL_RESET_CREATED,
FSMStates.TENANT_PRINCIPAL_OWNERSHIP_CREATED, FSMStates.TENANT_PRINCIPAL_OWNERSHIP_CREATED,
FSMStates.BILLING_OWNER_CREATED, FSMStates.BILLING_OWNER_CREATED,
FSMStates.TENANT_ADMIN_CREDENTIAL_RESET_CREATED,
FSMStates.COMPLETED, FSMStates.COMPLETED,
] ]