Use TextRange::contains_inclusive in fixes check

Author: popzxc

crates/ide/src/diagnostics.rs

@@ -257,8 +257,7 @@ mod tests {
 
         assert_eq_text!(&after, &actual);
         assert!(
-            fix.fix_trigger_range.start() <= file_position.offset
-                && fix.fix_trigger_range.end() >= file_position.offset,
+            fix.fix_trigger_range.contains_inclusive(file_position.offset),
             "diagnostic fix range {:?} does not touch cursor position {:?}",
             fix.fix_trigger_range,
             file_position.offset
@@ -288,8 +287,7 @@ mod tests {
 
         assert_eq_text!(&after, &actual);
         assert!(
-            fix.fix_trigger_range.start() <= file_position.offset
-                && fix.fix_trigger_range.end() >= file_position.offset,
+            fix.fix_trigger_range.contains_inclusive(file_position.offset),
             "diagnostic fix range {:?} does not touch cursor position {:?}",
             fix.fix_trigger_range,
             file_position.offset

crates/ide/src/diagnostics/fixes.rs

@@ -111,7 +111,8 @@ impl DiagnosticWithFix for IncorrectCase {
         let offset = name_node.syntax().text_range().start();
         let file_position = FilePosition { file_id, offset };
 
-        let rename_changes = rename_with_semantics(sema, file_position, &self.suggested_text)?;
+        let rename_changes =
+            rename_with_semantics(sema, file_position, &self.suggested_text).ok()?;
 
         let label = format!("Rename to {}", self.suggested_text);
         Some(Fix::new(&label, rename_changes.info, rename_changes.range))