Merge #532

532: fix go to parent module r=matklad a=matklad



Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

crates/ra_ide_api/src/navigation_target.rs

@@ -89,6 +89,25 @@ impl NavigationTarget {
         Ok(res)
     }
 
+    pub(crate) fn from_module_to_decl(
+        db: &RootDatabase,
+        module: hir::Module,
+    ) -> Cancelable<NavigationTarget> {
+        let name = module
+            .name(db)?
+            .map(|it| it.to_string().into())
+            .unwrap_or_default();
+        if let Some((file_id, source)) = module.declaration_source(db)? {
+            return Ok(NavigationTarget::from_syntax(
+                file_id,
+                name,
+                None,
+                source.syntax(),
+            ));
+        }
+        NavigationTarget::from_module(db, module)
+    }
+
     // TODO once Def::Item is gone, this should be able to always return a NavigationTarget
     pub(crate) fn from_def(db: &RootDatabase, def: Def) -> Cancelable<Option<NavigationTarget>> {
         let res = match def {

crates/ra_ide_api/src/parent_module.rs

@@ -12,7 +12,7 @@ pub(crate) fn parent_module(
         None => return Ok(Vec::new()),
         Some(it) => it,
     };
-    let nav = NavigationTarget::from_module(db, module)?;
+    let nav = NavigationTarget::from_module_to_decl(db, module)?;
     Ok(vec![nav])
 }
 
@@ -31,7 +31,7 @@ mod tests {
             ",
         );
         let nav = analysis.parent_module(pos).unwrap().pop().unwrap();
-        nav.assert_match("foo SOURCE_FILE FileId(2) [0; 10)");
+        nav.assert_match("foo MODULE FileId(1) [0; 8)");
     }
 
     #[test]