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

### What does this PR try to resolve?

Fixes #14893

In writing a test for this, I had a hard time finding the other tests as
they mixed
- Having an `add_` prefix
- Calling this fuzzy vs normalized

So I made the test names consistent

### How should we test and review this PR?

### Additional information

Author: weihanglo

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/mod.rs

@@ -2,9 +2,7 @@ mod add_basic;
 mod add_multiple;
 mod add_no_vendored_package_with_alter_registry;
 mod add_no_vendored_package_with_vendor;
-mod add_normalized_name_external;
 mod add_toolchain;
-mod add_workspace_non_fuzzy;
 mod build;
 mod build_prefer_existing_version;
 mod change_rename_target;
@@ -14,7 +12,6 @@ mod deprecated_default_features;
 mod deprecated_section;
 mod detect_workspace_inherit;
 mod detect_workspace_inherit_features;
-mod detect_workspace_inherit_fuzzy;
 mod detect_workspace_inherit_optional;
 mod detect_workspace_inherit_path_base;
 mod detect_workspace_inherit_public;
@@ -28,7 +25,6 @@ mod features;
 mod features_activated_over_limit;
 mod features_deactivated_over_limit;
 mod features_empty;
-mod features_fuzzy;
 mod features_multiple_occurrences;
 mod features_preserve;
 mod features_spaced_values;
@@ -42,7 +38,6 @@ mod git_inferred_name;
 mod git_inferred_name_multiple;
 mod git_multiple_names;
 mod git_multiple_packages_features;
-mod git_normalized_name;
 mod git_registry;
 mod git_rev;
 mod git_tag;
@@ -75,6 +70,13 @@ mod no_args;
 mod no_default_features;
 mod no_optional;
 mod no_public;
+mod normalize_name_git;
+mod normalize_name_path;
+mod normalize_name_path_existing;
+mod normalize_name_registry;
+mod normalize_name_registry_existing;
+mod normalize_name_registry_yanked;
+mod normalize_name_workspace_dep;
 mod offline_empty_cache;
 mod optional;
 mod overwrite_default_features;
@@ -118,7 +120,6 @@ mod path_base_unstable;
 mod path_dev;
 mod path_inferred_name;
 mod path_inferred_name_conflicts_full_feature;
-mod path_normalized_name;
 mod preserve_dep_std_table;
 mod preserve_features_sorted;
 mod preserve_features_table;
@@ -149,3 +150,4 @@ mod vers;
 mod workspace_name;
 mod workspace_path;
 mod workspace_path_dev;
+mod yanked;

tests/testsuite/cargo_add/add_normalized_name_external/in renamed to tests/testsuite/cargo_add/normalize_name_git/in

@@ -0,0 +1 @@
+../add-basic.in

tests/testsuite/cargo_add/git_normalized_name/mod.rs renamed to tests/testsuite/cargo_add/normalize_name_git/mod.rs

@@ -0,0 +1,31 @@
+use cargo_test_support::compare::assert_ui;
+use cargo_test_support::current_dir;
+use cargo_test_support::file;
+use cargo_test_support::prelude::*;
+use cargo_test_support::str;
+use cargo_test_support::Project;
+
+#[cargo_test]
+fn case() {
+    cargo_test_support::registry::init();
+    cargo_test_support::registry::Package::new("linked-hash-map", "0.5.0").publish();
+    cargo_test_support::registry::Package::new("linked-hash-map", "0.5.4").publish();
+    cargo_test_support::registry::Package::new("linked-hash-map", "0.6.0")
+        .yanked(true)
+        .publish();
+
+    let project = Project::from_template(current_dir!().join("in"));
+    let project_root = project.root();
+    let cwd = &project_root;
+
+    snapbox::cmd::Command::cargo_ui()
+        .arg("add")
+        .arg_line("linked_hash_map")
+        .current_dir(cwd)
+        .assert()
+        .success()
+        .stdout_eq(str![""])
+        .stderr_eq(file!["stderr.term.svg"]);
+
+    assert_ui().subset_matches(current_dir!().join("out"), &project_root);
+}

tests/testsuite/cargo_add/git_normalized_name/out/Cargo.toml renamed to tests/testsuite/cargo_add/normalize_name_git/out/Cargo.toml

@@ -0,0 +1,9 @@
+[workspace]
+
+[package]
+name = "cargo-list-test-fixture"
+version = "0.0.0"
+edition = "2015"
+
+[dependencies]
+linked-hash-map = "0.5.4"

tests/testsuite/cargo_add/git_normalized_name/stderr.term.svg renamed to tests/testsuite/cargo_add/normalize_name_git/stderr.term.svg

@@ -0,0 +1 @@
+../add-basic.in

tests/testsuite/cargo_add/path_normalized_name/in/dependency/Cargo.toml renamed to tests/testsuite/cargo_add/normalize_name_path/in/dependency/Cargo.toml

@@ -0,0 +1,31 @@
+use cargo_test_support::compare::assert_ui;
+use cargo_test_support::current_dir;
+use cargo_test_support::file;
+use cargo_test_support::prelude::*;
+use cargo_test_support::str;
+use cargo_test_support::Project;
+
+#[cargo_test]
+fn case() {
+    cargo_test_support::registry::init();
+    cargo_test_support::registry::Package::new("linked-hash-map", "0.5.0").publish();
+    cargo_test_support::registry::Package::new("linked-hash-map", "0.5.4").publish();
+    cargo_test_support::registry::Package::new("linked-hash-map", "0.6.0")
+        .yanked(true)
+        .publish();
+
+    let project = Project::from_template(current_dir!().join("in"));
+    let project_root = project.root();
+    let cwd = &project_root;
+
+    snapbox::cmd::Command::cargo_ui()
+        .arg("add")
+        .arg_line("linked-hash-map")
+        .current_dir(cwd)
+        .assert()
+        .success()
+        .stdout_eq(str![""])
+        .stderr_eq(file!["stderr.term.svg"]);
+
+    assert_ui().subset_matches(current_dir!().join("out"), &project_root);
+}

tests/testsuite/cargo_add/add_workspace_non_fuzzy/in/fuzzy_name/src/lib.rs renamed to tests/testsuite/cargo_add/normalize_name_path/in/dependency/src/lib.rs

@@ -0,0 +1,9 @@
+[workspace]
+
+[package]
+name = "cargo-list-test-fixture"
+version = "0.0.0"
+edition = "2015"
+
+[dependencies]
+linked-hash-map = "0.5.4"