isort: fix bad interaction between force-sort-within-sections and `…

…force-to-top`

Fixes #3630.

crates/ruff/resources/test/fixtures/isort/force_sort_within_sections.py

@@ -2,7 +2,9 @@
 from c import * # import_from_star
 import a  # import
 import c.d
+from z import z1
 import b as b1  # import_as
+import z
 
 from ..parent import *
 from .my import fn

crates/ruff/src/rules/isort/mod.rs

@@ -665,6 +665,7 @@ mod tests {
             &Settings {
                 isort: super::settings::Settings {
                     force_sort_within_sections: true,
+                    force_to_top: BTreeSet::from(["z".to_string()]),
                     ..super::settings::Settings::default()
                 },
                 src: vec![test_resource_path("fixtures/isort")],

crates/ruff/src/rules/isort/snapshots/ruff__rules__isort__tests__force_sort_within_sections.py.snap

@@ -11,15 +11,15 @@ expression: diagnostics
     row: 1
     column: 0
   end_location:
-    row: 12
+    row: 14
     column: 0
   fix:
-    content: "import a  # import\nimport b as b1  # import_as\nimport c.d\nfrom a import a1  # import_from\nfrom c import *  # import_from_star\n\nfrom ...grandparent import fn3\nfrom ..parent import *\nfrom . import my\nfrom .my import fn\nfrom .my.nested import fn2\n"
+    content: "import a  # import\nimport b as b1  # import_as\nimport c.d\nimport z\nfrom a import a1  # import_from\nfrom c import *  # import_from_star\nfrom z import z1\n\nfrom ...grandparent import fn3\nfrom ..parent import *\nfrom . import my\nfrom .my import fn\nfrom .my.nested import fn2\n"
     location:
       row: 1
       column: 0
     end_location:
-      row: 12
+      row: 14
       column: 0
   parent: ~
 

crates/ruff/src/rules/isort/snapshots/ruff__rules__isort__tests__force_sort_within_sections_force_sort_within_sections.py.snap

@@ -11,15 +11,15 @@ expression: diagnostics
     row: 1
     column: 0
   end_location:
-    row: 12
+    row: 14
     column: 0
   fix:
-    content: "import a  # import\nfrom a import a1  # import_from\nimport b as b1  # import_as\nfrom c import *  # import_from_star\nimport c.d\n\nfrom ...grandparent import fn3\nfrom ..parent import *\nfrom . import my\nfrom .my import fn\nfrom .my.nested import fn2\n"
+    content: "import z\nfrom z import z1\nimport a  # import\nfrom a import a1  # import_from\nimport b as b1  # import_as\nfrom c import *  # import_from_star\nimport c.d\n\nfrom ...grandparent import fn3\nfrom ..parent import *\nfrom . import my\nfrom .my import fn\nfrom .my.nested import fn2\n"
     location:
       row: 1
       column: 0
     end_location:
-      row: 12
+      row: 14
       column: 0
   parent: ~
 

crates/ruff/src/rules/isort/sorting.rs

@@ -138,6 +138,17 @@ pub fn cmp_import_from(
     })
 }
 
+/// Compare an import to an import-from.
+fn cmp_import_import_from(
+    import: &AliasData,
+    import_from: &ImportFromData,
+    force_to_top: &BTreeSet<String>,
+) -> Ordering {
+    cmp_force_to_top(import.name, &import_from.module_name(), force_to_top).then_with(|| {
+        natord::compare_ignore_case(import.name, import_from.module.unwrap_or_default())
+    })
+}
+
 /// Compare two [`EitherImport`] enums which may be [`Import`] or [`ImportFrom`]
 /// structs.
 pub fn cmp_either_import(
@@ -149,10 +160,10 @@ pub fn cmp_either_import(
     match (a, b) {
         (Import((alias1, _)), Import((alias2, _))) => cmp_modules(alias1, alias2, force_to_top),
         (ImportFrom((import_from, ..)), Import((alias, _))) => {
-            natord::compare_ignore_case(import_from.module.unwrap_or_default(), alias.name)
+            cmp_import_import_from(alias, import_from, force_to_top).reverse()
         }
         (Import((alias, _)), ImportFrom((import_from, ..))) => {
-            natord::compare_ignore_case(alias.name, import_from.module.unwrap_or_default())
+            cmp_import_import_from(alias, import_from, force_to_top)
         }
         (ImportFrom((import_from1, ..)), ImportFrom((import_from2, ..))) => cmp_import_from(
             import_from1,