Lint using isort

test/conftest.py

@@ -7,7 +7,8 @@ import os
 import platform
 import pwd
 import shutil
-from subprocess import Popen, PIPE
+from subprocess import PIPE, Popen
+
 import py
 import pytest
 
@@ -34,6 +35,12 @@ def pylint_version():
     return '2.17.0'
 
 
+@pytest.fixture(scope='session')
+def isort_version():
+    """Version of isort supported"""
+    return '5.12.0'
+
+
 @pytest.fixture(scope='session')
 def flake8_version():
     """Version of flake8 supported"""

test/test_alt.py

@@ -1,6 +1,7 @@
 """Test alt"""
 import os
 import string
+
 import py
 import pytest
 import utils

test/test_alt_copy.py

@@ -1,6 +1,7 @@
 """Test yadm.alt-copy"""
 
 import os
+
 import pytest
 
 

test/test_assert_private_dirs.py

@@ -2,6 +2,7 @@
 
 import os
 import re
+
 import pytest
 
 pytestmark = pytest.mark.usefixtures('ds1_copy')

test/test_clone.py

@@ -2,6 +2,7 @@
 
 import os
 import re
+
 import pytest
 
 BOOTSTRAP_CODE = 123

test/test_config.py

@@ -1,6 +1,7 @@
 """Test config"""
 
 import os
+
 import pytest
 
 TEST_SECTION = 'test'

test/test_encryption.py

@@ -3,6 +3,7 @@
 import os
 import shlex
 import time
+
 import pytest
 
 KEY_FILE = 'test/test_key'

test/test_enter.py

@@ -1,6 +1,7 @@
 """Test enter"""
 
 import os
+
 import pytest
 
 

test/test_git.py

@@ -1,6 +1,7 @@
 """Test git"""
 
 import re
+
 import pytest
 
 

test/test_list.py

@@ -1,6 +1,7 @@
 """Test list"""
 
 import os
+
 import pytest
 
 

test/test_perms.py

@@ -1,6 +1,7 @@
 """Test perms"""
 
 import os
+
 import pytest
 
 

test/test_syntax.py

@@ -1,6 +1,7 @@
 """Syntax checks"""
 
 import os
+
 import pytest
 
 
@@ -34,6 +35,16 @@ def test_pylint(pytestconfig, runner, pylint_version):
     assert run.success
 
 
+def test_isort(pytestconfig, runner, isort_version):
+    """Passes isort"""
+    if not pytestconfig.getoption("--force-linters"):
+        run = runner(command=['isort', '--version'], report=False)
+        if isort_version not in run.out:
+            pytest.skip('Unsupported isort version')
+    run = runner(command=['isort', '-c', 'test'])
+    assert run.success
+
+
 def test_flake8(pytestconfig, runner, flake8_version):
     """Passes flake8"""
     if not pytestconfig.getoption("--force-linters"):

test/test_unit_copy_perms.py

@@ -1,5 +1,6 @@
 """Unit tests: copy_perms"""
 import os
+
 import pytest
 
 OCTAL = '7654'

test/test_unit_remove_stale_links.py

@@ -1,5 +1,6 @@
 """Unit tests: remove_stale_links"""
 import os
+
 import pytest
 
 

test/test_unit_template_j2.py

@@ -1,5 +1,6 @@
 """Unit tests: template_j2cli & template_envtpl"""
 import os
+
 import pytest
 
 FILE_MODE = 0o754

test/test_unit_x_program.py

@@ -1,6 +1,7 @@
 """Unit tests: yadm.[git,gpg]-program"""
 
 import os
+
 import pytest
 
 

test/test_upgrade.py

@@ -1,6 +1,7 @@
 """Test upgrade"""
 
 import os
+
 import pytest
 
 

test/test_version.py

@@ -1,6 +1,7 @@
 """Test version"""
 
 import re
+
 import pytest
 
 

test/utils.py

@@ -3,8 +3,8 @@
 This module holds values/functions common to multiple tests.
 """
 
-import re
 import os
+import re
 
 ALT_FILE1 = 'test_alt'
 ALT_FILE2 = 'test alt/test alt'