fix(add): Don't select yanked versions when normalizing names

Fixes #14893

Author: epage

src/cargo/sources/registry/mod.rs

@@ -856,7 +856,13 @@ impl<'gctx> Source for RegistrySource<'gctx> {
                     }
                     any_pending |= self
                         .index
-                        .query_inner(name_permutation, &req, &mut *self.ops, f)?
+                        .query_inner(name_permutation, &req, &mut *self.ops, &mut |s| {
+                            if !s.is_yanked() {
+                                f(s);
+                            } else if kind == QueryKind::Alternatives {
+                                f(s);
+                            }
+                        })?
                         .is_pending();
                 }
             }

tests/testsuite/cargo_add/normalize_name_registry_yanked/mod.rs

@@ -23,7 +23,7 @@ fn case() {
         .arg_line("linked_hash_map")
         .current_dir(cwd)
         .assert()
-        .failure()
+        .success()
         .stdout_eq(str![""])
         .stderr_eq(file!["stderr.term.svg"]);
 

tests/testsuite/cargo_add/normalize_name_registry_yanked/out/Cargo.toml

@@ -4,3 +4,6 @@
 name = "cargo-list-test-fixture"
 version = "0.0.0"
 edition = "2015"
+
+[dependencies]
+linked-hash-map = "0.5.4"