list_agents function argument location_id has changed to location

[xsgbloom]

dfcx-scrapi/src/dfcx_scrapi/core/agents.py

Line 89 in 1c9f813

location: str = None) -> List[types.Agent]:

This broke one of my customers in production. Underlying code uses parameter location_id. The documentation for this function still calls the argument "location_id", so I'm guessing this was an inadvertent change. Is there unit testing to catch/prevent this kind of thing?

[kmaphoenix]

@xsgbloom this was an inadvertent change.
I will revert it back to the original.

Re: unit tests, unfortunately none are running on this file yet.
We're in the process of building out unit tests for all files to prevent breaks like this from happening.