duosecurity / duo_client_python

Python library for interacting with the Duo Auth, Admin, and Accounts APIs
https://duo.com/docs/
Other
136 stars 136 forks source link

Add child_api_host to AccountAdmin tests to fix failing tests #250

Closed jeffreyparker closed 9 months ago

jeffreyparker commented 9 months ago

Description

I don't know why this suddenly started failing, but during the AccountAdmin tests, it looks like AccountAdmin init is trying to really reach out to the API to fetch the child accounts. This pre-populates child_api_host so it doesn't make a request.

I don't know that this is necessarily the correct fix (e.g. vs ensuring that HTTP call is mocked), but it is a fix.

Motivation and Context

Fix the automated tests

How Has This Been Tested?

Tests now pass

Types of Changes