crates/ide/src/diagnostics.rs

   1 //! Collects diagnostics & fixits  for a single file.
   2 //!
   3 //! The tricky bit here is that diagnostics are produced by hir in terms of
   4 //! macro-expanded files, but we need to present them to the users in terms of
   5 //! original files. So we need to map the ranges.
   6
   7 mod fixes;
   8 mod field_shorthand;
   9
  10 use std::cell::RefCell;
  11
  12 use hir::{
  13     diagnostics::{Diagnostic as _, DiagnosticCode, DiagnosticSinkBuilder},
  14     Semantics,
  15 };
  16 use ide_db::base_db::SourceDatabase;
  17 use ide_db::RootDatabase;
  18 use itertools::Itertools;
  19 use rustc_hash::FxHashSet;
  20 use syntax::{
  21     ast::{self, AstNode},
  22     SyntaxNode, TextRange, T,
  23 };
  24 use text_edit::TextEdit;
  25
  26 use crate::{FileId, Label, SourceChange, SourceFileEdit};
  27
  28 use self::fixes::DiagnosticWithFix;
  29
  30 #[derive(Debug)]
  31 pub struct Diagnostic {
  32     // pub name: Option<String>,
  33     pub message: String,
  34     pub range: TextRange,
  35     pub severity: Severity,
  36     pub fix: Option<Fix>,
  37     pub unused: bool,
  38     pub code: Option<DiagnosticCode>,
  39 }
  40
  41 impl Diagnostic {
  42     fn error(range: TextRange, message: String) -> Self {
  43         Self { message, range, severity: Severity::Error, fix: None, unused: false, code: None }
  44     }
  45
  46     fn hint(range: TextRange, message: String) -> Self {
  47         Self {
  48             message,
  49             range,
  50             severity: Severity::WeakWarning,
  51             fix: None,
  52             unused: false,
  53             code: None,
  54         }
  55     }
  56
  57     fn with_fix(self, fix: Option<Fix>) -> Self {
  58         Self { fix, ..self }
  59     }
  60
  61     fn with_unused(self, unused: bool) -> Self {
  62         Self { unused, ..self }
  63     }
  64
  65     fn with_code(self, code: Option<DiagnosticCode>) -> Self {
  66         Self { code, ..self }
  67     }
  68 }
  69
  70 #[derive(Debug)]
  71 pub struct Fix {
  72     pub label: Label,
  73     pub source_change: SourceChange,
  74     /// Allows to trigger the fix only when the caret is in the range given
  75     pub fix_trigger_range: TextRange,
  76 }
  77
  78 impl Fix {
  79     fn new(label: &str, source_change: SourceChange, fix_trigger_range: TextRange) -> Self {
  80         let label = Label::new(label);
  81         Self { label, source_change, fix_trigger_range }
  82     }
  83 }
  84
  85 #[derive(Debug, Copy, Clone)]
  86 pub enum Severity {
  87     Error,
  88     WeakWarning,
  89 }
  90
  91 #[derive(Default, Debug, Clone)]
  92 pub struct DiagnosticsConfig {
  93     pub disable_experimental: bool,
  94     pub disabled: FxHashSet<String>,
  95 }
  96
  97 pub(crate) fn diagnostics(
  98     db: &RootDatabase,
  99     config: &DiagnosticsConfig,
 100     file_id: FileId,
 101 ) -> Vec<Diagnostic> {
 102     let _p = profile::span("diagnostics");
 103     let sema = Semantics::new(db);
 104     let parse = db.parse(file_id);
 105     let mut res = Vec::new();
 106
 107     // [#34344] Only take first 128 errors to prevent slowing down editor/ide, the number 128 is chosen arbitrarily.
 108     res.extend(
 109         parse
 110             .errors()
 111             .iter()
 112             .take(128)
 113             .map(|err| Diagnostic::error(err.range(), format!("Syntax Error: {}", err))),
 114     );
 115
 116     for node in parse.tree().syntax().descendants() {
 117         check_unnecessary_braces_in_use_statement(&mut res, file_id, &node);
 118         field_shorthand::check(&mut res, file_id, &node);
 119     }
 120     let res = RefCell::new(res);
 121     let sink_builder = DiagnosticSinkBuilder::new()
 122         .on::<hir::diagnostics::UnresolvedModule, _>(|d| {
 123             res.borrow_mut().push(diagnostic_with_fix(d, &sema));
 124         })
 125         .on::<hir::diagnostics::MissingFields, _>(|d| {
 126             res.borrow_mut().push(diagnostic_with_fix(d, &sema));
 127         })
 128         .on::<hir::diagnostics::MissingOkInTailExpr, _>(|d| {
 129             res.borrow_mut().push(diagnostic_with_fix(d, &sema));
 130         })
 131         .on::<hir::diagnostics::NoSuchField, _>(|d| {
 132             res.borrow_mut().push(diagnostic_with_fix(d, &sema));
 133         })
 134         .on::<hir::diagnostics::RemoveThisSemicolon, _>(|d| {
 135             res.borrow_mut().push(diagnostic_with_fix(d, &sema));
 136         })
 137         .on::<hir::diagnostics::IncorrectCase, _>(|d| {
 138             res.borrow_mut().push(warning_with_fix(d, &sema));
 139         })
 140         .on::<hir::diagnostics::InactiveCode, _>(|d| {
 141             // If there's inactive code somewhere in a macro, don't propagate to the call-site.
 142             if d.display_source().file_id.expansion_info(db).is_some() {
 143                 return;
 144             }
 145
 146             // Override severity and mark as unused.
 147             res.borrow_mut().push(
 148                 Diagnostic::hint(sema.diagnostics_display_range(d).range, d.message())
 149                     .with_unused(true)
 150                     .with_code(Some(d.code())),
 151             );
 152         })
 153         .on::<hir::diagnostics::UnresolvedProcMacro, _>(|d| {
 154             // Use more accurate position if available.
 155             let display_range =
 156                 d.precise_location.unwrap_or_else(|| sema.diagnostics_display_range(d).range);
 157
 158             // FIXME: it would be nice to tell the user whether proc macros are currently disabled
 159             res.borrow_mut()
 160                 .push(Diagnostic::hint(display_range, d.message()).with_code(Some(d.code())));
 161         })
 162         // Only collect experimental diagnostics when they're enabled.
 163         .filter(|diag| !(diag.is_experimental() && config.disable_experimental))
 164         .filter(|diag| !config.disabled.contains(diag.code().as_str()));
 165
 166     // Finalize the `DiagnosticSink` building process.
 167     let mut sink = sink_builder
 168         // Diagnostics not handled above get no fix and default treatment.
 169         .build(|d| {
 170             res.borrow_mut().push(
 171                 Diagnostic::error(sema.diagnostics_display_range(d).range, d.message())
 172                     .with_code(Some(d.code())),
 173             );
 174         });
 175
 176     if let Some(m) = sema.to_module_def(file_id) {
 177         m.diagnostics(db, &mut sink);
 178     };
 179     drop(sink);
 180     res.into_inner()
 181 }
 182
 183 fn diagnostic_with_fix<D: DiagnosticWithFix>(d: &D, sema: &Semantics<RootDatabase>) -> Diagnostic {
 184     Diagnostic::error(sema.diagnostics_display_range(d).range, d.message())
 185         .with_fix(d.fix(&sema))
 186         .with_code(Some(d.code()))
 187 }
 188
 189 fn warning_with_fix<D: DiagnosticWithFix>(d: &D, sema: &Semantics<RootDatabase>) -> Diagnostic {
 190     Diagnostic::hint(sema.diagnostics_display_range(d).range, d.message())
 191         .with_fix(d.fix(&sema))
 192         .with_code(Some(d.code()))
 193 }
 194
 195 fn check_unnecessary_braces_in_use_statement(
 196     acc: &mut Vec<Diagnostic>,
 197     file_id: FileId,
 198     node: &SyntaxNode,
 199 ) -> Option<()> {
 200     let use_tree_list = ast::UseTreeList::cast(node.clone())?;
 201     if let Some((single_use_tree,)) = use_tree_list.use_trees().collect_tuple() {
 202         // If there is a comment inside the bracketed `use`,
 203         // assume it is a commented out module path and don't show diagnostic.
 204         if use_tree_list.has_inner_comment() {
 205             return Some(());
 206         }
 207
 208         let use_range = use_tree_list.syntax().text_range();
 209         let edit =
 210             text_edit_for_remove_unnecessary_braces_with_self_in_use_statement(&single_use_tree)
 211                 .unwrap_or_else(|| {
 212                     let to_replace = single_use_tree.syntax().text().to_string();
 213                     let mut edit_builder = TextEdit::builder();
 214                     edit_builder.delete(use_range);
 215                     edit_builder.insert(use_range.start(), to_replace);
 216                     edit_builder.finish()
 217                 });
 218
 219         acc.push(
 220             Diagnostic::hint(use_range, "Unnecessary braces in use statement".to_string())
 221                 .with_fix(Some(Fix::new(
 222                     "Remove unnecessary braces",
 223                     SourceFileEdit { file_id, edit }.into(),
 224                     use_range,
 225                 ))),
 226         );
 227     }
 228
 229     Some(())
 230 }
 231
 232 fn text_edit_for_remove_unnecessary_braces_with_self_in_use_statement(
 233     single_use_tree: &ast::UseTree,
 234 ) -> Option<TextEdit> {
 235     let use_tree_list_node = single_use_tree.syntax().parent()?;
 236     if single_use_tree.path()?.segment()?.syntax().first_child_or_token()?.kind() == T![self] {
 237         let start = use_tree_list_node.prev_sibling_or_token()?.text_range().start();
 238         let end = use_tree_list_node.text_range().end();
 239         return Some(TextEdit::delete(TextRange::new(start, end)));
 240     }
 241     None
 242 }
 243
 244 #[cfg(test)]
 245 mod tests {
 246     use expect_test::{expect, Expect};
 247     use stdx::trim_indent;
 248     use test_utils::assert_eq_text;
 249
 250     use crate::{fixture, DiagnosticsConfig};
 251
 252     /// Takes a multi-file input fixture with annotated cursor positions,
 253     /// and checks that:
 254     ///  * a diagnostic is produced
 255     ///  * this diagnostic fix trigger range touches the input cursor position
 256     ///  * that the contents of the file containing the cursor match `after` after the diagnostic fix is applied
 257     pub(crate) fn check_fix(ra_fixture_before: &str, ra_fixture_after: &str) {
 258         let after = trim_indent(ra_fixture_after);
 259
 260         let (analysis, file_position) = fixture::position(ra_fixture_before);
 261         let diagnostic = analysis
 262             .diagnostics(&DiagnosticsConfig::default(), file_position.file_id)
 263             .unwrap()
 264             .pop()
 265             .unwrap();
 266         let fix = diagnostic.fix.unwrap();
 267         let actual = {
 268             let file_id = fix.source_change.source_file_edits.first().unwrap().file_id;
 269             let mut actual = analysis.file_text(file_id).unwrap().to_string();
 270
 271             // Go from the last one to the first one, so that ranges won't be affected by previous edits.
 272             // FIXME: https://github.com/rust-analyzer/rust-analyzer/issues/4901#issuecomment-644675309
 273             for edit in fix.source_change.source_file_edits.iter().rev() {
 274                 edit.edit.apply(&mut actual);
 275             }
 276             actual
 277         };
 278
 279         assert_eq_text!(&after, &actual);
 280         assert!(
 281             fix.fix_trigger_range.contains_inclusive(file_position.offset),
 282             "diagnostic fix range {:?} does not touch cursor position {:?}",
 283             fix.fix_trigger_range,
 284             file_position.offset
 285         );
 286     }
 287
 288     /// Takes a multi-file input fixture with annotated cursor position and checks that no diagnostics
 289     /// apply to the file containing the cursor.
 290     pub(crate) fn check_no_diagnostics(ra_fixture: &str) {
 291         let (analysis, files) = fixture::files(ra_fixture);
 292         let diagnostics = files
 293             .into_iter()
 294             .flat_map(|file_id| {
 295                 analysis.diagnostics(&DiagnosticsConfig::default(), file_id).unwrap()
 296             })
 297             .collect::<Vec<_>>();
 298         assert_eq!(diagnostics.len(), 0, "unexpected diagnostics:\n{:#?}", diagnostics);
 299     }
 300
 301     fn check_expect(ra_fixture: &str, expect: Expect) {
 302         let (analysis, file_id) = fixture::file(ra_fixture);
 303         let diagnostics = analysis.diagnostics(&DiagnosticsConfig::default(), file_id).unwrap();
 304         expect.assert_debug_eq(&diagnostics)
 305     }
 306
 307     #[test]
 308     fn test_wrap_return_type() {
 309         check_fix(
 310             r#"
 311 //- /main.rs crate:main deps:core
 312 use core::result::Result::{self, Ok, Err};
 313
 314 fn div(x: i32, y: i32) -> Result<i32, ()> {
 315     if y == 0 {
 316         return Err(());
 317     }
 318     x / y<|>
 319 }
 320 //- /core/lib.rs crate:core
 321 pub mod result {
 322     pub enum Result<T, E> { Ok(T), Err(E) }
 323 }
 324 "#,
 325             r#"
 326 use core::result::Result::{self, Ok, Err};
 327
 328 fn div(x: i32, y: i32) -> Result<i32, ()> {
 329     if y == 0 {
 330         return Err(());
 331     }
 332     Ok(x / y)
 333 }
 334 "#,
 335         );
 336     }
 337
 338     #[test]
 339     fn test_wrap_return_type_handles_generic_functions() {
 340         check_fix(
 341             r#"
 342 //- /main.rs crate:main deps:core
 343 use core::result::Result::{self, Ok, Err};
 344
 345 fn div<T>(x: T) -> Result<T, i32> {
 346     if x == 0 {
 347         return Err(7);
 348     }
 349     <|>x
 350 }
 351 //- /core/lib.rs crate:core
 352 pub mod result {
 353     pub enum Result<T, E> { Ok(T), Err(E) }
 354 }
 355 "#,
 356             r#"
 357 use core::result::Result::{self, Ok, Err};
 358
 359 fn div<T>(x: T) -> Result<T, i32> {
 360     if x == 0 {
 361         return Err(7);
 362     }
 363     Ok(x)
 364 }
 365 "#,
 366         );
 367     }
 368
 369     #[test]
 370     fn test_wrap_return_type_handles_type_aliases() {
 371         check_fix(
 372             r#"
 373 //- /main.rs crate:main deps:core
 374 use core::result::Result::{self, Ok, Err};
 375
 376 type MyResult<T> = Result<T, ()>;
 377
 378 fn div(x: i32, y: i32) -> MyResult<i32> {
 379     if y == 0 {
 380         return Err(());
 381     }
 382     x <|>/ y
 383 }
 384 //- /core/lib.rs crate:core
 385 pub mod result {
 386     pub enum Result<T, E> { Ok(T), Err(E) }
 387 }
 388 "#,
 389             r#"
 390 use core::result::Result::{self, Ok, Err};
 391
 392 type MyResult<T> = Result<T, ()>;
 393
 394 fn div(x: i32, y: i32) -> MyResult<i32> {
 395     if y == 0 {
 396         return Err(());
 397     }
 398     Ok(x / y)
 399 }
 400 "#,
 401         );
 402     }
 403
 404     #[test]
 405     fn test_wrap_return_type_not_applicable_when_expr_type_does_not_match_ok_type() {
 406         check_no_diagnostics(
 407             r#"
 408 //- /main.rs crate:main deps:core
 409 use core::result::Result::{self, Ok, Err};
 410
 411 fn foo() -> Result<(), i32> { 0 }
 412
 413 //- /core/lib.rs crate:core
 414 pub mod result {
 415     pub enum Result<T, E> { Ok(T), Err(E) }
 416 }
 417 "#,
 418         );
 419     }
 420
 421     #[test]
 422     fn test_wrap_return_type_not_applicable_when_return_type_is_not_result() {
 423         check_no_diagnostics(
 424             r#"
 425 //- /main.rs crate:main deps:core
 426 use core::result::Result::{self, Ok, Err};
 427
 428 enum SomeOtherEnum { Ok(i32), Err(String) }
 429
 430 fn foo() -> SomeOtherEnum { 0 }
 431
 432 //- /core/lib.rs crate:core
 433 pub mod result {
 434     pub enum Result<T, E> { Ok(T), Err(E) }
 435 }
 436 "#,
 437         );
 438     }
 439
 440     #[test]
 441     fn test_fill_struct_fields_empty() {
 442         check_fix(
 443             r#"
 444 struct TestStruct { one: i32, two: i64 }
 445
 446 fn test_fn() {
 447     let s = TestStruct {<|>};
 448 }
 449 "#,
 450             r#"
 451 struct TestStruct { one: i32, two: i64 }
 452
 453 fn test_fn() {
 454     let s = TestStruct { one: (), two: ()};
 455 }
 456 "#,
 457         );
 458     }
 459
 460     #[test]
 461     fn test_fill_struct_fields_self() {
 462         check_fix(
 463             r#"
 464 struct TestStruct { one: i32 }
 465
 466 impl TestStruct {
 467     fn test_fn() { let s = Self {<|>}; }
 468 }
 469 "#,
 470             r#"
 471 struct TestStruct { one: i32 }
 472
 473 impl TestStruct {
 474     fn test_fn() { let s = Self { one: ()}; }
 475 }
 476 "#,
 477         );
 478     }
 479
 480     #[test]
 481     fn test_fill_struct_fields_enum() {
 482         check_fix(
 483             r#"
 484 enum Expr {
 485     Bin { lhs: Box<Expr>, rhs: Box<Expr> }
 486 }
 487
 488 impl Expr {
 489     fn new_bin(lhs: Box<Expr>, rhs: Box<Expr>) -> Expr {
 490         Expr::Bin {<|> }
 491     }
 492 }
 493 "#,
 494             r#"
 495 enum Expr {
 496     Bin { lhs: Box<Expr>, rhs: Box<Expr> }
 497 }
 498
 499 impl Expr {
 500     fn new_bin(lhs: Box<Expr>, rhs: Box<Expr>) -> Expr {
 501         Expr::Bin { lhs: (), rhs: () }
 502     }
 503 }
 504 "#,
 505         );
 506     }
 507
 508     #[test]
 509     fn test_fill_struct_fields_partial() {
 510         check_fix(
 511             r#"
 512 struct TestStruct { one: i32, two: i64 }
 513
 514 fn test_fn() {
 515     let s = TestStruct{ two: 2<|> };
 516 }
 517 "#,
 518             r"
 519 struct TestStruct { one: i32, two: i64 }
 520
 521 fn test_fn() {
 522     let s = TestStruct{ two: 2, one: () };
 523 }
 524 ",
 525         );
 526     }
 527
 528     #[test]
 529     fn test_fill_struct_fields_no_diagnostic() {
 530         check_no_diagnostics(
 531             r"
 532             struct TestStruct { one: i32, two: i64 }
 533
 534             fn test_fn() {
 535                 let one = 1;
 536                 let s = TestStruct{ one, two: 2 };
 537             }
 538         ",
 539         );
 540     }
 541
 542     #[test]
 543     fn test_fill_struct_fields_no_diagnostic_on_spread() {
 544         check_no_diagnostics(
 545             r"
 546             struct TestStruct { one: i32, two: i64 }
 547
 548             fn test_fn() {
 549                 let one = 1;
 550                 let s = TestStruct{ ..a };
 551             }
 552         ",
 553         );
 554     }
 555
 556     #[test]
 557     fn test_unresolved_module_diagnostic() {
 558         check_expect(
 559             r#"mod foo;"#,
 560             expect![[r#"
 561                 [
 562                     Diagnostic {
 563                         message: "unresolved module",
 564                         range: 0..8,
 565                         severity: Error,
 566                         fix: Some(
 567                             Fix {
 568                                 label: "Create module",
 569                                 source_change: SourceChange {
 570                                     source_file_edits: [],
 571                                     file_system_edits: [
 572                                         CreateFile {
 573                                             dst: AnchoredPathBuf {
 574                                                 anchor: FileId(
 575                                                     0,
 576                                                 ),
 577                                                 path: "foo.rs",
 578                                             },
 579                                             initial_contents: "",
 580                                         },
 581                                     ],
 582                                     is_snippet: false,
 583                                 },
 584                                 fix_trigger_range: 0..8,
 585                             },
 586                         ),
 587                         unused: false,
 588                         code: Some(
 589                             DiagnosticCode(
 590                                 "unresolved-module",
 591                             ),
 592                         ),
 593                     },
 594                 ]
 595             "#]],
 596         );
 597     }
 598
 599     #[test]
 600     fn range_mapping_out_of_macros() {
 601         // FIXME: this is very wrong, but somewhat tricky to fix.
 602         check_fix(
 603             r#"
 604 fn some() {}
 605 fn items() {}
 606 fn here() {}
 607
 608 macro_rules! id { ($($tt:tt)*) => { $($tt)*}; }
 609
 610 fn main() {
 611     let _x = id![Foo { a: <|>42 }];
 612 }
 613
 614 pub struct Foo { pub a: i32, pub b: i32 }
 615 "#,
 616             r#"
 617 fn some(, b: ()) {}
 618 fn items() {}
 619 fn here() {}
 620
 621 macro_rules! id { ($($tt:tt)*) => { $($tt)*}; }
 622
 623 fn main() {
 624     let _x = id![Foo { a: 42 }];
 625 }
 626
 627 pub struct Foo { pub a: i32, pub b: i32 }
 628 "#,
 629         );
 630     }
 631
 632     #[test]
 633     fn test_check_unnecessary_braces_in_use_statement() {
 634         check_no_diagnostics(
 635             r#"
 636 use a;
 637 use a::{c, d::e};
 638
 639 mod a {
 640     mod c {}
 641     mod d {
 642         mod e {}
 643     }
 644 }
 645 "#,
 646         );
 647         check_no_diagnostics(
 648             r#"
 649 use a;
 650 use a::{
 651     c,
 652     // d::e
 653 };
 654
 655 mod a {
 656     mod c {}
 657     mod d {
 658         mod e {}
 659     }
 660 }
 661 "#,
 662         );
 663         check_fix(
 664             r"
 665             mod b {}
 666             use {<|>b};
 667             ",
 668             r"
 669             mod b {}
 670             use b;
 671             ",
 672         );
 673         check_fix(
 674             r"
 675             mod b {}
 676             use {b<|>};
 677             ",
 678             r"
 679             mod b {}
 680             use b;
 681             ",
 682         );
 683         check_fix(
 684             r"
 685             mod a { mod c {} }
 686             use a::{c<|>};
 687             ",
 688             r"
 689             mod a { mod c {} }
 690             use a::c;
 691             ",
 692         );
 693         check_fix(
 694             r"
 695             mod a {}
 696             use a::{self<|>};
 697             ",
 698             r"
 699             mod a {}
 700             use a;
 701             ",
 702         );
 703         check_fix(
 704             r"
 705             mod a { mod c {} mod d { mod e {} } }
 706             use a::{c, d::{e<|>}};
 707             ",
 708             r"
 709             mod a { mod c {} mod d { mod e {} } }
 710             use a::{c, d::e};
 711             ",
 712         );
 713     }
 714
 715     #[test]
 716     fn test_add_field_from_usage() {
 717         check_fix(
 718             r"
 719 fn main() {
 720     Foo { bar: 3, baz<|>: false};
 721 }
 722 struct Foo {
 723     bar: i32
 724 }
 725 ",
 726             r"
 727 fn main() {
 728     Foo { bar: 3, baz: false};
 729 }
 730 struct Foo {
 731     bar: i32,
 732     baz: bool
 733 }
 734 ",
 735         )
 736     }
 737
 738     #[test]
 739     fn test_add_field_in_other_file_from_usage() {
 740         check_fix(
 741             r#"
 742 //- /main.rs
 743 mod foo;
 744
 745 fn main() {
 746     foo::Foo { bar: 3, <|>baz: false};
 747 }
 748 //- /foo.rs
 749 struct Foo {
 750     bar: i32
 751 }
 752 "#,
 753             r#"
 754 struct Foo {
 755     bar: i32,
 756     pub(crate) baz: bool
 757 }
 758 "#,
 759         )
 760     }
 761
 762     #[test]
 763     fn test_disabled_diagnostics() {
 764         let mut config = DiagnosticsConfig::default();
 765         config.disabled.insert("unresolved-module".into());
 766
 767         let (analysis, file_id) = fixture::file(r#"mod foo;"#);
 768
 769         let diagnostics = analysis.diagnostics(&config, file_id).unwrap();
 770         assert!(diagnostics.is_empty());
 771
 772         let diagnostics = analysis.diagnostics(&DiagnosticsConfig::default(), file_id).unwrap();
 773         assert!(!diagnostics.is_empty());
 774     }
 775
 776     #[test]
 777     fn test_rename_incorrect_case() {
 778         check_fix(
 779             r#"
 780 pub struct test_struct<|> { one: i32 }
 781
 782 pub fn some_fn(val: test_struct) -> test_struct {
 783     test_struct { one: val.one + 1 }
 784 }
 785 "#,
 786             r#"
 787 pub struct TestStruct { one: i32 }
 788
 789 pub fn some_fn(val: TestStruct) -> TestStruct {
 790     TestStruct { one: val.one + 1 }
 791 }
 792 "#,
 793         );
 794
 795         check_fix(
 796             r#"
 797 pub fn some_fn(NonSnakeCase<|>: u8) -> u8 {
 798     NonSnakeCase
 799 }
 800 "#,
 801             r#"
 802 pub fn some_fn(non_snake_case: u8) -> u8 {
 803     non_snake_case
 804 }
 805 "#,
 806         );
 807
 808         check_fix(
 809             r#"
 810 pub fn SomeFn<|>(val: u8) -> u8 {
 811     if val != 0 { SomeFn(val - 1) } else { val }
 812 }
 813 "#,
 814             r#"
 815 pub fn some_fn(val: u8) -> u8 {
 816     if val != 0 { some_fn(val - 1) } else { val }
 817 }
 818 "#,
 819         );
 820
 821         check_fix(
 822             r#"
 823 fn some_fn() {
 824     let whatAWeird_Formatting<|> = 10;
 825     another_func(whatAWeird_Formatting);
 826 }
 827 "#,
 828             r#"
 829 fn some_fn() {
 830     let what_a_weird_formatting = 10;
 831     another_func(what_a_weird_formatting);
 832 }
 833 "#,
 834         );
 835     }
 836
 837     #[test]
 838     fn test_uppercase_const_no_diagnostics() {
 839         check_no_diagnostics(
 840             r#"
 841 fn foo() {
 842     const ANOTHER_ITEM<|>: &str = "some_item";
 843 }
 844 "#,
 845         );
 846     }
 847
 848     #[test]
 849     fn test_rename_incorrect_case_struct_method() {
 850         check_fix(
 851             r#"
 852 pub struct TestStruct;
 853
 854 impl TestStruct {
 855     pub fn SomeFn<|>() -> TestStruct {
 856         TestStruct
 857     }
 858 }
 859 "#,
 860             r#"
 861 pub struct TestStruct;
 862
 863 impl TestStruct {
 864     pub fn some_fn() -> TestStruct {
 865         TestStruct
 866     }
 867 }
 868 "#,
 869         );
 870     }
 871
 872     #[test]
 873     fn test_single_incorrect_case_diagnostic_in_function_name_issue_6970() {
 874         let input = r#"fn FOO<|>() {}"#;
 875         let expected = r#"fn foo() {}"#;
 876
 877         let (analysis, file_position) = fixture::position(input);
 878         let diagnostics =
 879             analysis.diagnostics(&DiagnosticsConfig::default(), file_position.file_id).unwrap();
 880         assert_eq!(diagnostics.len(), 1);
 881
 882         check_fix(input, expected);
 883     }
 884 }