isort import formatting

octodns/cmds/args.py

@@ -7,6 +7,7 @@ from logging import DEBUG, INFO, WARNING, Formatter, StreamHandler, getLogger
 from logging.config import dictConfig
 from logging.handlers import SysLogHandler
 from sys import stderr, stdout
+
 from yaml import safe_load
 
 from octodns import __VERSION__

octodns/cmds/compare.py

@@ -3,8 +3,8 @@
 Octo-DNS Comparator
 '''
 
-from pprint import pprint
 import sys
+from pprint import pprint
 
 from octodns.cmds.args import ArgumentParser
 from octodns.manager import Manager

octodns/cmds/report.py

@@ -3,12 +3,13 @@
 Octo-DNS Reporter
 '''
 
+import re
 from concurrent.futures import ThreadPoolExecutor
-from dns.exception import Timeout
-from dns.resolver import NXDOMAIN, NoAnswer, NoNameservers, Resolver, query
 from logging import getLogger
 from sys import stdout
-import re
+
+from dns.exception import Timeout
+from dns.resolver import NXDOMAIN, NoAnswer, NoNameservers, Resolver, query
 
 from octodns.cmds.args import ArgumentParser
 from octodns.manager import Manager

octodns/idna.py

@@ -4,7 +4,9 @@
 
 from collections.abc import MutableMapping
 
-from idna import IDNAError as _IDNAError, decode as _decode, encode as _encode
+from idna import IDNAError as _IDNAError
+from idna import decode as _decode
+from idna import encode as _encode
 
 # Providers will need to to make calls to these at the appropriate points,
 # generally right before they pass names off to api calls. For an example of

octodns/manager.py

@@ -18,13 +18,10 @@ from .record import Record
 from .yaml import safe_load
 from .zone import Zone
 
-
 # TODO: this can go away once we no longer support python 3.7
 try:
-    from importlib.metadata import (
-        PackageNotFoundError,
-        version as module_version,
-    )
+    from importlib.metadata import PackageNotFoundError
+    from importlib.metadata import version as module_version
 except ModuleNotFoundError:  # pragma: no cover
 
     class PackageNotFoundError(Exception):

octodns/processor/ownership.py

@@ -6,7 +6,6 @@ from collections import defaultdict
 
 from ..provider.plan import Plan
 from ..record import Record
-
 from .base import BaseProcessor
 
 

octodns/provider/base.py

@@ -4,8 +4,8 @@
 
 from ..source.base import BaseSource
 from ..zone import Zone
-from .plan import Plan
 from . import SupportsException
+from .plan import Plan
 
 
 class BaseProvider(BaseSource):

octodns/provider/plan.py

@@ -2,11 +2,10 @@
 #
 #
 
+from io import StringIO
 from logging import DEBUG, ERROR, INFO, WARN, getLogger
 from sys import stdout
 
-from io import StringIO
-
 
 class UnsafePlan(Exception):
     pass

octodns/provider/yaml.py

@@ -2,15 +2,15 @@
 #
 #
 
+import logging
 from collections import defaultdict
 from os import listdir, makedirs
 from os.path import isdir, isfile, join
-import logging
 
 from ..record import Record
-from ..yaml import safe_load, safe_dump
-from .base import BaseProvider
+from ..yaml import safe_dump, safe_load
 from . import ProviderException
+from .base import BaseProvider
 
 
 class YamlProvider(BaseProvider):

octodns/record/__init__.py

@@ -17,14 +17,14 @@ from .geo import GeoCodes, GeoValue
 from .loc import LocRecord, LocValue
 from .mx import MxRecord, MxValue
 from .naptr import NaptrRecord, NaptrValue
-from .ns import NsValue, NsRecord
-from .ptr import PtrValue, PtrRecord
+from .ns import NsRecord, NsValue
+from .ptr import PtrRecord, PtrValue
 from .rr import Rr, RrParseError
 from .spf import SpfRecord
 from .srv import SrvRecord, SrvValue
 from .sshfp import SshfpRecord, SshfpValue
 from .tlsa import TlsaRecord, TlsaValue
-from .txt import TxtValue, TxtRecord
+from .txt import TxtRecord, TxtValue
 from .urlfwd import UrlfwdRecord, UrlfwdValue
 
 # quell warnings

octodns/record/chunked.py

@@ -2,9 +2,10 @@
 #
 #
 
-from .base import ValuesMixin
 import re
 
+from .base import ValuesMixin
+
 
 class _ChunkedValuesMixin(ValuesMixin):
     CHUNK_SIZE = 255

octodns/record/dynamic.py

@@ -2,8 +2,8 @@
 #
 #
 
-from logging import getLogger
 import re
+from logging import getLogger
 
 from .change import Update
 from .geo import GeoCodes

octodns/record/geo.py

@@ -2,8 +2,8 @@
 #
 #
 
-from logging import getLogger
 import re
+from logging import getLogger
 
 from ..equality import EqualityTupleMixin
 from .base import ValuesMixin

octodns/record/srv.py

@@ -2,9 +2,10 @@
 #
 #
 
-from fqdn import FQDN
 import re
 
+from fqdn import FQDN
+
 from ..equality import EqualityTupleMixin
 from ..idna import idna_encode
 from .base import Record, ValuesMixin

octodns/source/tinydns.py

@@ -2,13 +2,13 @@
 #
 #
 
+import logging
+import re
+import textwrap
 from collections import defaultdict
 from ipaddress import ip_address
 from os import listdir
 from os.path import join
-import logging
-import re
-import textwrap
 
 from ..record import Record
 from ..zone import DuplicateRecordException, SubzoneRecordException

octodns/yaml.py

@@ -3,10 +3,9 @@
 #
 
 from natsort import natsort_keygen
-from yaml import SafeDumper, SafeLoader, load, dump
-from yaml.representer import SafeRepresenter
+from yaml import SafeDumper, SafeLoader, dump, load
 from yaml.constructor import ConstructorError
-
+from yaml.representer import SafeRepresenter
 
 _natsort_key = natsort_keygen()
 

octodns/zone.py

@@ -2,9 +2,9 @@
 #
 #
 
+import re
 from collections import defaultdict
 from logging import getLogger
-import re
 
 from .idna import idna_decode, idna_encode
 from .record import Create, Delete

setup.py

@@ -4,6 +4,7 @@ from io import StringIO
 from os import environ
 from os.path import dirname, join
 from subprocess import CalledProcessError, check_output
+
 import octodns
 
 try:

tests/helpers.py

@@ -2,9 +2,9 @@
 #
 #
 
+from logging import getLogger
 from shutil import rmtree
 from tempfile import mkdtemp
-from logging import getLogger
 
 from octodns.processor.base import BaseProcessor
 from octodns.provider.base import BaseProvider

tests/test_octodns_manager.py

@@ -4,32 +4,31 @@
 
 from os import environ
 from os.path import dirname, isfile, join
+from unittest import TestCase
+from unittest.mock import MagicMock, patch
+
+from helpers import (
+    DynamicProvider,
+    GeoProvider,
+    NoSshFpProvider,
+    PlannableProvider,
+    SimpleProvider,
+    TemporaryDirectory,
+)
 
 from octodns import __VERSION__
 from octodns.idna import IdnaDict, idna_encode
 from octodns.manager import (
-    _AggregateTarget,
     MainThreadExecutor,
     Manager,
     ManagerException,
+    _AggregateTarget,
 )
 from octodns.processor.base import BaseProcessor
 from octodns.record import Create, Delete, Record
 from octodns.yaml import safe_load
 from octodns.zone import Zone
 
-from unittest import TestCase
-from unittest.mock import MagicMock, patch
-
-from helpers import (
-    DynamicProvider,
-    GeoProvider,
-    NoSshFpProvider,
-    PlannableProvider,
-    SimpleProvider,
-    TemporaryDirectory,
-)
-
 config_dir = join(dirname(__file__), 'config')
 
 

tests/test_octodns_plan.py

@@ -6,6 +6,8 @@ from io import StringIO
 from logging import getLogger
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.provider.plan import (
     Plan,
     PlanHtml,
@@ -17,9 +19,6 @@ from octodns.provider.plan import (
 from octodns.record import Create, Delete, Record, Update
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
-
 simple = SimpleProvider()
 zone = Zone('unit.tests.', [])
 existing = Record.new(

tests/test_octodns_processor_ownership.py

@@ -4,13 +4,12 @@
 
 from unittest import TestCase
 
+from helpers import PlannableProvider
+
 from octodns.processor.ownership import OwnershipProcessor
 from octodns.record import Delete, Record
 from octodns.zone import Zone
 
-from helpers import PlannableProvider
-
-
 zone = Zone('unit.tests.', [])
 records = {}
 for record in [

tests/test_octodns_provider_yaml.py

@@ -5,22 +5,22 @@
 from os import makedirs
 from os.path import basename, dirname, isdir, isfile, join
 from unittest import TestCase
+
+from helpers import TemporaryDirectory
 from yaml import safe_load
 from yaml.constructor import ConstructorError
 
 from octodns.idna import idna_encode
-from octodns.record import NsValue, Create, Record, ValuesMixin
 from octodns.provider import ProviderException
 from octodns.provider.base import Plan
 from octodns.provider.yaml import (
-    _list_all_yaml_files,
     SplitYamlProvider,
     YamlProvider,
+    _list_all_yaml_files,
 )
+from octodns.record import Create, NsValue, Record, ValuesMixin
 from octodns.zone import SubzoneRecordException, Zone
 
-from helpers import TemporaryDirectory
-
 
 class TestYamlProvider(TestCase):
     def test_provider(self):

tests/test_octodns_record.py

@@ -6,8 +6,8 @@ from unittest import TestCase
 
 from octodns.idna import idna_encode
 from octodns.record import (
-    ARecord,
     AliasRecord,
+    ARecord,
     Create,
     Delete,
     MxValue,

tests/test_octodns_record_a.py

@@ -4,13 +4,13 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
 from octodns.record.a import ARecord
 from octodns.record.exception import ValidationError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordA(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_alias.py

@@ -4,13 +4,13 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
 from octodns.record.alias import AliasRecord
 from octodns.record.exception import ValidationError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordAlias(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_caa.py

@@ -4,14 +4,14 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
 from octodns.record.caa import CaaRecord, CaaValue
 from octodns.record.exception import ValidationError
 from octodns.record.rr import RrParseError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordCaa(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_cname.py

@@ -4,13 +4,13 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
 from octodns.record.cname import CnameRecord
 from octodns.record.exception import ValidationError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordCname(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_dname.py

@@ -4,13 +4,13 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
 from octodns.record.dname import DnameRecord
 from octodns.record.exception import ValidationError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordDname(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_dynamic.py

@@ -4,6 +4,8 @@
 
 from unittest import TestCase
 
+from helpers import DynamicProvider, SimpleProvider
+
 from octodns.idna import idna_encode
 from octodns.record import Record
 from octodns.record.a import ARecord, Ipv4Value
@@ -13,8 +15,6 @@ from octodns.record.dynamic import _Dynamic, _DynamicPool, _DynamicRule
 from octodns.record.exception import ValidationError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider, DynamicProvider
-
 
 class TestRecordDynamic(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_geo.py

@@ -4,14 +4,14 @@
 
 from unittest import TestCase
 
+from helpers import GeoProvider, SimpleProvider
+
 from octodns.record import Record
 from octodns.record.a import ARecord
-from octodns.record.geo import GeoCodes, GeoValue
 from octodns.record.exception import ValidationError
+from octodns.record.geo import GeoCodes, GeoValue
 from octodns.zone import Zone
 
-from helpers import SimpleProvider, GeoProvider
-
 
 class TestRecordGeo(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_loc.py

@@ -4,14 +4,14 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
-from octodns.record.loc import LocRecord, LocValue
 from octodns.record.exception import ValidationError
+from octodns.record.loc import LocRecord, LocValue
 from octodns.record.rr import RrParseError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordLoc(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_mx.py

@@ -4,14 +4,14 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
-from octodns.record.mx import MxRecord, MxValue
 from octodns.record.exception import ValidationError
+from octodns.record.mx import MxRecord, MxValue
 from octodns.record.rr import RrParseError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordMx(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_naptr.py

@@ -4,14 +4,14 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
-from octodns.record.naptr import NaptrRecord, NaptrValue
 from octodns.record.exception import ValidationError
+from octodns.record.naptr import NaptrRecord, NaptrValue
 from octodns.record.rr import RrParseError
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordNaptr(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_ns.py

@@ -5,8 +5,8 @@
 from unittest import TestCase
 
 from octodns.record import Record
-from octodns.record.ns import NsRecord, NsValue
 from octodns.record.exception import ValidationError
+from octodns.record.ns import NsRecord, NsValue
 from octodns.zone import Zone
 
 

tests/test_octodns_record_ptr.py

@@ -5,8 +5,8 @@
 from unittest import TestCase
 
 from octodns.record import Record
-from octodns.record.ptr import PtrRecord, PtrValue
 from octodns.record.exception import ValidationError
+from octodns.record.ptr import PtrRecord, PtrValue
 from octodns.zone import Zone
 
 

tests/test_octodns_record_spf.py

@@ -5,8 +5,8 @@
 from unittest import TestCase
 
 from octodns.record import Record
-from octodns.record.spf import SpfRecord
 from octodns.record.exception import ValidationError
+from octodns.record.spf import SpfRecord
 from octodns.zone import Zone
 
 

tests/test_octodns_record_srv.py

@@ -4,14 +4,14 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
-from octodns.record.srv import SrvRecord, SrvValue
 from octodns.record.exception import ValidationError
 from octodns.record.rr import RrParseError
+from octodns.record.srv import SrvRecord, SrvValue
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordSrv(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_sshfp.py

@@ -4,14 +4,14 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record.base import Record
 from octodns.record.exception import ValidationError
-from octodns.record.sshfp import SshfpRecord, SshfpValue
 from octodns.record.rr import RrParseError
+from octodns.record.sshfp import SshfpRecord, SshfpValue
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordSshfp(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_tlsa.py

@@ -4,14 +4,14 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
-from octodns.record.tlsa import TlsaRecord, TlsaValue
 from octodns.record.exception import ValidationError
 from octodns.record.rr import RrParseError
+from octodns.record.tlsa import TlsaRecord, TlsaValue
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordTlsa(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_record_txt.py

@@ -5,8 +5,8 @@
 from unittest import TestCase
 
 from octodns.record import Record
-from octodns.record.txt import TxtRecord
 from octodns.record.exception import ValidationError
+from octodns.record.txt import TxtRecord
 from octodns.zone import Zone
 
 

tests/test_octodns_record_urlfwd.py

@@ -4,13 +4,13 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
-from octodns.record.urlfwd import UrlfwdRecord, UrlfwdValue
 from octodns.record.exception import ValidationError
+from octodns.record.urlfwd import UrlfwdRecord, UrlfwdValue
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestRecordUrlfwd(TestCase):
     zone = Zone('unit.tests.', [])

tests/test_octodns_source_envvar.py

@@ -1,8 +1,10 @@
 from unittest import TestCase
 from unittest.mock import patch
 
-from octodns.source.envvar import EnvVarSource
-from octodns.source.envvar import EnvironmentVariableNotFoundException
+from octodns.source.envvar import (
+    EnvironmentVariableNotFoundException,
+    EnvVarSource,
+)
 from octodns.zone import Zone
 
 

tests/test_octodns_source_tinydns.py

@@ -4,12 +4,12 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.record import Record
 from octodns.source.tinydns import TinyDnsFileSource
 from octodns.zone import Zone
 
-from helpers import SimpleProvider
-
 
 class TestTinyDnsFileSource(TestCase):
     source = TinyDnsFileSource('test', './tests/zones/tinydns')

tests/test_octodns_yaml.py

@@ -4,6 +4,7 @@
 
 from io import StringIO
 from unittest import TestCase
+
 from yaml.constructor import ConstructorError
 
 from octodns.yaml import safe_dump, safe_load

tests/test_octodns_zone.py

@@ -4,10 +4,12 @@
 
 from unittest import TestCase
 
+from helpers import SimpleProvider
+
 from octodns.idna import idna_encode
 from octodns.record import (
-    ARecord,
     AaaaRecord,
+    ARecord,
     Create,
     Delete,
     NsRecord,
@@ -21,8 +23,6 @@ from octodns.zone import (
     Zone,
 )
 
-from helpers import SimpleProvider
-
 
 class TestZone(TestCase):
     def test_lowering(self):