v0.16.3

[vasqued2]

• Don't save last_update to Recorder to slow growth (thanks @duczz for the idea)
• Refactor DataCoordinator in preparation for support of non-ESPN APIs
• Refactor regression tests to use mock API calls
• Refactor regression tests to use snapshots instead of expected results files

This was a major refactor of key components of the sensor in preparation for the addition of new functionality.
Please report any defects to the Issues log.

---

This discussion was created from the release v0.16.3.

[vasqued2]

This was a major refactor to prep for several new contributions.

@irishpadres - I refactored your "all leagues" code to fit into a structure that will work well with non-ESPN APIs. I moved most of your code into async_fetch_espn_all_leagues_data() which does the API calls w/ the tightened date ranges and moved the population of the league_name to process_event() which handles the parsing and attribute values. All the regression tests passed so I think we are OK. Lmk if there is an edge case that broke or feel free to fix yourself.

@iAmWayward - The bad news is I cleaned up the DataCoordinator and I doubt your current PR merges cleanly. The good news, I refactored the code to make it very easy to integrate non-ESPN APIs using your approach. It should be easy to call your code from async_call_sport_apis() and get it working quickly. The improvements to the testing should make it easy to add tests for your new API. Using snapshots and mocks now instead of the old file overrides.

@duczz - This version will not save last_update to the Recorder as you suggested. Shifting the tests to mocks and snapshots should make it much easier to add additional functionality and attribute changes without a lot of manual work.

Thanks to all of you for your contributions and patience as I did some long needed clean-up to make sure there was a solid base to build upon.