octodns / octodns-ns1

Ns1Provider provider for octoDNS
MIT License
4 stars 13 forks source link

Switch to assertLogs over hacky getLogger test #60

Closed ross closed 1 year ago

ross commented 1 year ago

geo records have been deprecated and there's new log warnings about that. Those warnings trip up the hacky mess that I previously did to check for some log messages around unsupported geo records here. This PR reworks that test to use the built-in assertLogs method and cleans all that up.

/cc https://github.com/octodns/octodns/pull/1032 which this fixes