pr comments

Author: adstep

crates/ide/src/runnables.rs

@@ -208,7 +208,9 @@ pub(crate) fn runnables(db: &RootDatabase, file_id: FileId) -> Vec<Runnable> {
 // |===
 pub(crate) fn selected_test(db: &RootDatabase, position: FilePosition) -> Option<Runnable> {
     let sema = Semantics::new(db);
-    let syntax = sema.parse(position.file_id).syntax().clone();
+    let parsed = sema.parse(position.file_id);
+    let syntax = parsed.syntax();
+
     return find_selected_test(&sema, &syntax, position);
 }
 
@@ -219,11 +221,11 @@ fn find_selected_test(
 ) -> Option<Runnable> {
     syntax
         .token_at_offset(position.offset)
-        .find(|_| true)
+        .next()
         .and_then(|token| token.parent())
         .and_then(|node| {
             node.ancestors().find_map(|ancestor| {
-                let fn_def = ast::Fn::cast(ancestor.clone())?;
+                let fn_def = ast::Fn::cast(ancestor)?;
                 test_related_attribute(&fn_def)?;
                 as_test_runnable(sema, &fn_def)
             })
@@ -249,7 +251,8 @@ pub(crate) fn related_tests(
 ) -> Vec<Runnable> {
     let sema = Semantics::new(db);
     let mut res: FxHashSet<Runnable> = FxHashSet::default();
-    let syntax = sema.parse(position.file_id).syntax().clone();
+    let parsed = sema.parse(position.file_id);
+    let syntax = parsed.syntax();
 
     find_related_tests(&sema, &syntax, position, search_scope, &mut res);