test(lsp): make lsp_completions_auto_import more robust (#18482)

A completely unrelated change in
https://github.com/denoland/deno/pull/18286 threw of this test.

These changes make it more robust and easier to update in such cases.

cli/tests/integration/lsp_tests.rs

@@ -4402,14 +4402,12 @@ fn lsp_completions_auto_import() {
     json!({ "triggerKind": 1 }),
   );
   assert!(!list.is_incomplete);
-  if !list.items.iter().any(|item| item.label == "foo") {
+  let item = list.items.iter().find(|item| item.label == "foo");
+  if item.is_none() {
     panic!("completions items missing 'foo' symbol");
   }
 
-  // the request here is one of the items in `list`
+  let req = json!({
-  let res = client.write_request(
-    "completionItem/resolve",
-    json!({
     "label": "foo",
     "kind": 6,
     "sortText": "￿16",
@@ -4434,8 +4432,10 @@ fn lsp_completions_auto_import() {
         "useCodeSnippet": false
       }
     }
-    }),
+  });
-  );
+  assert_eq!(serde_json::to_value(item.unwrap()).unwrap(), req);
+
+  let res = client.write_request("completionItem/resolve", req);
   assert_eq!(
     res,
     json!({