From 56b8b23391cd59a47da3cd751b67bb8b176d3dc0 Mon Sep 17 00:00:00 2001 From: Ross McFarland Date: Tue, 24 Aug 2021 09:18:14 -0700 Subject: [PATCH] Delete second ns1 record to make sure cache clears w/o zone --- tests/test_octodns_provider_ns1.py | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a/tests/test_octodns_provider_ns1.py b/tests/test_octodns_provider_ns1.py index bd13e97..1d3f49b 100644 --- a/tests/test_octodns_provider_ns1.py +++ b/tests/test_octodns_provider_ns1.py @@ -2615,6 +2615,24 @@ class TestNs1Client(TestCase): 'sub.unit.tests': 'bar', }, client._zones_cache) + # Delete the other record, no zone this time, record should still go + # away + reset() + record_delete_mock.side_effect = [{}] + self.assertEquals({}, client.records_delete('unit.tests', + 'a.unit.tests', 'A')) + record_delete_mock.assert_has_calls([call('unit.tests', 'a.unit.tests', + 'A')]) + self.assertEquals({ + 'unit.tests': { + 'a.unit.tests': {}, + 'aaaa.unit.tests': {}, + } + }, client._records_cache) + self.assertEquals({ + 'sub.unit.tests': 'bar', + }, client._zones_cache) + # Record update removes zone and caches result record_update_mock.side_effect = ['done'] self.assertEquals('done', client.records_update('sub.unit.tests', @@ -2625,9 +2643,7 @@ class TestNs1Client(TestCase): 'AAAA', key='val')]) self.assertEquals({ 'unit.tests': { - 'a.unit.tests': { - 'A': 'baz' - }, + 'a.unit.tests': {}, 'aaaa.unit.tests': {}, }, 'sub.unit.tests': {