Auto merge of #11612 - hi-rustin:rustin-patch-cargo-add, r=epage

`cargo add` check `[dependencies]` order without considering the dotted item

### What does this PR try to resolve?

Try to close #11584

`cargo check` check `[dependencies]` order without considering the dotted item.

### How should we test and review this PR?
See the unit test.

src/cargo/ops/cargo_add/mod.rs

@@ -98,7 +98,12 @@ pub fn add(workspace: &Workspace<'_>, options: &AddOptions<'_>) -> CargoResult<(
         .get_table(&dep_table)
         .map(TomlItem::as_table)
         .map_or(true, |table_option| {
-            table_option.map_or(true, |table| is_sorted(table.iter().map(|(name, _)| name)))
+            table_option.map_or(true, |table| {
+                is_sorted(table.get_values().iter_mut().map(|(key, _)| {
+                    // get_values key paths always have at least one key.
+                    key.remove(0)
+                }))
+            })
         });
     for dep in deps {
         print_action_msg(&mut options.config.shell(), &dep, &dep_table)?;

tests/testsuite/cargo_add/mod.rs

@@ -100,6 +100,7 @@ mod quiet;
 mod registry;
 mod rename;
 mod require_weak;
+mod sorted_table_with_dotted_item;
 mod target;
 mod target_cfg;
 mod unknown_inherited_feature;

tests/testsuite/cargo_add/sorted_table_with_dotted_item/in/Cargo.toml

@@ -0,0 +1,13 @@
+[workspace]
+
+[package]
+name = "cargo-list-test-fixture"
+version = "0.0.0"
+
+[dependencies]
+toml = "0.1.1"
+versioned-package = "0.1.1"
+
+[dependencies.my-build-package1]
+version = "0.1.1"
+

tests/testsuite/cargo_add/sorted_table_with_dotted_item/mod.rs

@@ -0,0 +1,25 @@
+use cargo_test_support::compare::assert_ui;
+use cargo_test_support::prelude::*;
+use cargo_test_support::Project;
+
+use crate::cargo_add::init_registry;
+use cargo_test_support::curr_dir;
+
+#[cargo_test]
+fn sorted_table_with_dotted_item() {
+    init_registry();
+    let project = Project::from_template(curr_dir!().join("in"));
+    let project_root = project.root();
+    let cwd = &project_root;
+
+    snapbox::cmd::Command::cargo_ui()
+        .arg("add")
+        .arg_line("unrelateed-crate")
+        .current_dir(cwd)
+        .assert()
+        .success()
+        .stdout_matches_path(curr_dir!().join("stdout.log"))
+        .stderr_matches_path(curr_dir!().join("stderr.log"));
+
+    assert_ui().subset_matches(curr_dir!().join("out"), &project_root);
+}

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

@@ -0,0 +1,14 @@
+[workspace]
+
+[package]
+name = "cargo-list-test-fixture"
+version = "0.0.0"
+
+[dependencies]
+toml = "0.1.1"
+unrelateed-crate = "99999.0.0"
+versioned-package = "0.1.1"
+
+[dependencies.my-build-package1]
+version = "0.1.1"
+

tests/testsuite/cargo_add/sorted_table_with_dotted_item/stderr.log

@@ -0,0 +1,2 @@
+    Updating `dummy-registry` index
+      Adding unrelateed-crate v99999.0.0 to dependencies.