Zone._remove_record shim removed in favor of long standing Zone.remove_record

CHANGELOG.md

@@ -6,6 +6,8 @@
    * Provider, Source, and Processor shims removed, they've been warnings for >
      1yr.  Everything should be using and referring to provider-specific
      modules now.
+   * Zone._remove_record shim removed in favor of long standing
+     Zone.remove_record
 
 #### Stuff
 

octodns/zone.py

@@ -132,9 +132,6 @@ class Zone(object):
 
         self._records[record.name].discard(record)
 
-    # TODO: delete this
-    _remove_record = remove_record
-
     def changes(self, desired, target):
         self.log.debug('changes: zone=%s, target=%s', self, target)
 

tests/test_octodns_zone.py

@@ -125,7 +125,7 @@ class TestZone(TestCase):
         # add a record, delete a record -> [Delete, Create]
         c = ARecord(before, 'c', {'ttl': 42, 'value': '1.1.1.1'})
         after.add_record(c)
-        after._remove_record(b)
+        after.remove_record(b)
         self.assertEqual(after.records, set([a, c]))
         changes = before.changes(after, target)
         self.assertEqual(2, len(changes))