imports: sort imports everywhere in Spack (#24695)

* fix remaining flake8 errors

* imports: sort imports everywhere in Spack

We enabled import order checking in #23947, but fixing things manually drives
people crazy. This used `spack style --fix --all` from #24071 to automatically
sort everything in Spack so PR submitters won't have to deal with it.

This should go in after #24071, as it assumes we're using `isort`, not
`flake8-import-order` to order things. `isort` seems to be more flexible and
allows `llnl` mports to be in their own group before `spack` ones, so this
seems like a good switch.

1 files changed, 14 insertions, 8 deletions

diff --git a/lib/spack/spack/test/variant.py b/lib/spack/spack/test/variant.py
index 501d897321..efbe8501a4 100644
--- a/lib/spack/spack/test/variant.py
+++ b/lib/spack/spack/test/variant.py
@@ -4,18 +4,24 @@
 # SPDX-License-Identifier: (Apache-2.0 OR MIT)
 
 
-import pytest
 import numbers
 
-from spack.variant import Variant, SingleValuedVariant, VariantMap
-from spack.variant import MultiValuedVariant, BoolValuedVariant
-from spack.variant import UnsatisfiableVariantSpecError
-from spack.variant import InconsistentValidationError
-from spack.variant import MultipleValuesInExclusiveVariantError
-from spack.variant import InvalidVariantValueError, DuplicateVariantError
-from spack.variant import disjoint_sets
+import pytest
 
 import spack.error
+from spack.variant import (
+    BoolValuedVariant,
+    DuplicateVariantError,
+    InconsistentValidationError,
+    InvalidVariantValueError,
+    MultipleValuesInExclusiveVariantError,
+    MultiValuedVariant,
+    SingleValuedVariant,
+    UnsatisfiableVariantSpecError,
+    Variant,
+    VariantMap,
+    disjoint_sets,
+)
 
 
 class TestMultiValuedVariant(object):