Rename "throw" to "raise"
This commit is contained in:
parent
8347e4e387
commit
fd65a3a972
@ -141,8 +141,8 @@ class MockCloudProvider(CloudProviderInterface):
|
|||||||
self._authorize(auth_credentials)
|
self._authorize(auth_credentials)
|
||||||
|
|
||||||
self._delay(1, 5)
|
self._delay(1, 5)
|
||||||
self._maybe_throw(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
self._maybe_raise(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
||||||
self._maybe_throw(
|
self._maybe_raise(
|
||||||
self.ENV_CREATE_FAILURE_PCT,
|
self.ENV_CREATE_FAILURE_PCT,
|
||||||
GeneralCSPException("Could not create environment."),
|
GeneralCSPException("Could not create environment."),
|
||||||
)
|
)
|
||||||
@ -153,8 +153,8 @@ class MockCloudProvider(CloudProviderInterface):
|
|||||||
self._authorize(auth_credentials)
|
self._authorize(auth_credentials)
|
||||||
|
|
||||||
self._delay(1, 5)
|
self._delay(1, 5)
|
||||||
self._maybe_throw(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
self._maybe_raise(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
||||||
self._maybe_throw(
|
self._maybe_raise(
|
||||||
self.ATAT_ADMIN_CREATE_FAILURE_PCT,
|
self.ATAT_ADMIN_CREATE_FAILURE_PCT,
|
||||||
GeneralCSPException("Could not create admin user."),
|
GeneralCSPException("Could not create admin user."),
|
||||||
)
|
)
|
||||||
@ -165,8 +165,8 @@ class MockCloudProvider(CloudProviderInterface):
|
|||||||
self._authorize(auth_credentials)
|
self._authorize(auth_credentials)
|
||||||
|
|
||||||
self._delay(1, 5)
|
self._delay(1, 5)
|
||||||
self._maybe_throw(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
self._maybe_raise(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
||||||
self._maybe_throw(
|
self._maybe_raise(
|
||||||
self.ATAT_ADMIN_CREATE_FAILURE_PCT,
|
self.ATAT_ADMIN_CREATE_FAILURE_PCT,
|
||||||
GeneralCSPException("Could not create environment baseline."),
|
GeneralCSPException("Could not create environment baseline."),
|
||||||
)
|
)
|
||||||
@ -182,8 +182,8 @@ class MockCloudProvider(CloudProviderInterface):
|
|||||||
self._authorize(auth_credentials)
|
self._authorize(auth_credentials)
|
||||||
|
|
||||||
self._delay(1, 5)
|
self._delay(1, 5)
|
||||||
self._maybe_throw(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
self._maybe_raise(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
||||||
self._maybe_throw(
|
self._maybe_raise(
|
||||||
self.ATAT_ADMIN_CREATE_FAILURE_PCT,
|
self.ATAT_ADMIN_CREATE_FAILURE_PCT,
|
||||||
GeneralCSPException("Could not create user."),
|
GeneralCSPException("Could not create user."),
|
||||||
)
|
)
|
||||||
@ -191,11 +191,11 @@ class MockCloudProvider(CloudProviderInterface):
|
|||||||
return {"id": self._id()}
|
return {"id": self._id()}
|
||||||
|
|
||||||
def suspend_user(self, auth_credentials, csp_user_id):
|
def suspend_user(self, auth_credentials, csp_user_id):
|
||||||
self._maybe_throw(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
self._maybe_raise(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
||||||
return self._maybe(12)
|
return self._maybe(12)
|
||||||
|
|
||||||
def delete_user(self, auth_credentials, csp_user_id):
|
def delete_user(self, auth_credentials, csp_user_id):
|
||||||
self._maybe_throw(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
self._maybe_raise(self.NETWORK_FAILURE_PCT, self.NETWORK_EXCEPTION)
|
||||||
return self._maybe(12)
|
return self._maybe(12)
|
||||||
|
|
||||||
def get_calculator_url(self):
|
def get_calculator_url(self):
|
||||||
@ -217,7 +217,7 @@ class MockCloudProvider(CloudProviderInterface):
|
|||||||
def _maybe(self, pct):
|
def _maybe(self, pct):
|
||||||
return not self._with_failure or self._random.randrange(0, 100) < pct
|
return not self._with_failure or self._random.randrange(0, 100) < pct
|
||||||
|
|
||||||
def _maybe_throw(self, pct, exc):
|
def _maybe_raise(self, pct, exc):
|
||||||
if self._with_failure and self._maybe(pct):
|
if self._with_failure and self._maybe(pct):
|
||||||
raise exc
|
raise exc
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user