Fix the tests and fix the precommit hook

crates/ra_syntax/src/grammar/expressions/atom.rs

@@ -89,7 +89,7 @@ pub(super) fn atom_expr(p: &mut Parser, r: Restrictions) -> Option<(CompletedMar
                 WHILE_KW => while_expr(p, Some(m)),
                 L_CURLY => block_expr(p, Some(m)),
                 _ => {
-                    // test misplaced_label_err
+                    // test_err misplaced_label_err
                     // fn main() {
                     //     'loop: impl
                     // }
@@ -354,7 +354,7 @@ pub(crate) fn match_arm_list(p: &mut Parser) {
 // fn foo() {
 //     match () {
 //         _ => (),
-//         _ if Test>{field: 0} => (),
+//         _ if Test > Test{field: 0} => (),
 //         X | Y if Z => (),
 //         | X | Y if Z => (),
 //         | X => (),

crates/ra_syntax/src/grammar/items.rs

@@ -89,7 +89,7 @@ pub(super) fn maybe_item(p: &mut Parser, flavor: ItemFlavor) -> MaybeItem {
     // modifiers
     has_mods |= p.eat(CONST_KW);
 
-    // test unsafe_block_in_mod
+    // test_err unsafe_block_in_mod
     // fn foo(){} unsafe { } fn bar(){}
     if p.at(UNSAFE_KW) && p.nth(1) != L_CURLY {
         p.eat(UNSAFE_KW);

crates/ra_syntax/src/grammar/items/traits.rs

@@ -116,7 +116,7 @@ fn choose_type_params_over_qpath(p: &Parser) -> bool {
         && (p.nth(2) == R_ANGLE || p.nth(2) == COMMA || p.nth(2) == COLON || p.nth(2) == EQ)
 }
 
-// test impl_type
+// test_err impl_type
 // impl Type {}
 // impl Trait1 for T {}
 // impl impl NotType {}

crates/ra_syntax/src/grammar/items/use_item.rs

@@ -74,7 +74,7 @@ fn use_tree(p: &mut Parser) {
                     //  other::path as some_other_name,
                     //  different::path as different_name,
                     //  yet::another::path,
-                    //  running::out::of::synonyms::for::different::*
+                    //  running::out::of::synonyms::for_::different::*
                     // };
                     opt_alias(p);
                 }

crates/ra_syntax/src/grammar/type_args.rs

@@ -26,7 +26,7 @@ pub(super) fn opt_type_arg_list(p: &mut Parser, colon_colon_required: bool) {
 }
 
 // test type_arg
-// type A = B<'static, i32, Item=u64>
+// type A = B<'static, i32, Item=u64>;
 fn type_arg(p: &mut Parser) {
     let m = p.start();
     match p.current() {

crates/ra_syntax/src/grammar/types.rs

@@ -97,7 +97,7 @@ fn pointer_type(p: &mut Parser) {
         // type C = *mut ();
         MUT_KW | CONST_KW => p.bump(),
         _ => {
-            // test pointer_type_no_mutability
+            // test_err pointer_type_no_mutability
             // type T = *();
             p.error(
                 "expected mut or const in raw pointer type \
@@ -132,7 +132,7 @@ fn array_or_slice_type(p: &mut Parser) {
             p.expect(R_BRACK);
             ARRAY_TYPE
         }
-        // test array_type_missing_semi
+        // test_err array_type_missing_semi
         // type T = [() 92];
         _ => {
             p.error("expected `;` or `]`");
@@ -175,7 +175,7 @@ fn fn_pointer_type(p: &mut Parser) {
     if p.at(EXTERN_KW) {
         abi(p);
     }
-    // test fn_pointer_type_missing_fn
+    // test_err fn_pointer_type_missing_fn
     // type F = unsafe ();
     if !p.eat(FN_KW) {
         m.abandon(p);

crates/ra_syntax/tests/data/parser/err/0028_macro_2.0.rs

@@ -13,4 +13,3 @@ fn test_use_tree_merge() {
         );
     }
 }
-

crates/ra_syntax/tests/data/parser/err/0028_macro_2.0.txt

@@ -1,4 +1,4 @@
-SOURCE_FILE@[0; 350)
+SOURCE_FILE@[0; 349)
   MACRO_CALL@[0; 41)
     PATH@[0; 5)
       PATH_SEGMENT@[0; 5)
@@ -314,4 +314,4 @@ SOURCE_FILE@[0; 350)
           R_CURLY@[345; 346)
       WHITESPACE@[346; 347)
       R_CURLY@[347; 348)
-  WHITESPACE@[348; 350)
+  WHITESPACE@[348; 349)

crates/ra_syntax/tests/data/parser/inline/0058_type_arg.rs

@@ -1 +0,0 @@
-type A = B<'static, i32, Item=u64>

crates/ra_syntax/tests/data/parser/inline/0063_lambda_expr.txt

@@ -1,91 +0,0 @@
-SOURCE_FILE@[0; 74)
-  FN_DEF@[0; 74)
-    FN_KW@[0; 2)
-    NAME@[2; 6)
-      WHITESPACE@[2; 3)
-      IDENT@[3; 6) "foo"
-    PARAM_LIST@[6; 9)
-      L_PAREN@[6; 7)
-      R_PAREN@[7; 8)
-      WHITESPACE@[8; 9)
-    BLOCK_EXPR@[9; 74)
-      L_CURLY@[9; 10)
-      EXPR_STMT@[10; 26)
-        LAMBDA_EXPR@[10; 20)
-          PARAM_LIST@[10; 18)
-            WHITESPACE@[10; 15)
-            PIPE@[15; 16)
-            PIPE@[16; 17)
-            WHITESPACE@[17; 18)
-          TUPLE_EXPR@[18; 20)
-            L_PAREN@[18; 19)
-            R_PAREN@[19; 20)
-        SEMI@[20; 21)
-        WHITESPACE@[21; 26)
-      EXPR_STMT@[26; 48)
-        LAMBDA_EXPR@[26; 42)
-          PARAM_LIST@[26; 29)
-            PIPE@[26; 27)
-            PIPE@[27; 28)
-            WHITESPACE@[28; 29)
-          THIN_ARROW@[29; 31)
-          PATH_TYPE@[31; 36)
-            PATH@[31; 36)
-              PATH_SEGMENT@[31; 36)
-                NAME_REF@[31; 36)
-                  WHITESPACE@[31; 32)
-                  IDENT@[32; 35) "i32"
-                  WHITESPACE@[35; 36)
-          BLOCK_EXPR@[36; 42)
-            L_CURLY@[36; 37)
-            LITERAL@[37; 41)
-              WHITESPACE@[37; 38)
-              INT_NUMBER@[38; 40) "92"
-              WHITESPACE@[40; 41)
-            R_CURLY@[41; 42)
-        SEMI@[42; 43)
-        WHITESPACE@[43; 48)
-      EXPR_STMT@[48; 59)
-        LAMBDA_EXPR@[48; 53)
-          PARAM_LIST@[48; 52)
-            PIPE@[48; 49)
-            PARAM@[49; 50)
-              BIND_PAT@[49; 50)
-                NAME@[49; 50)
-                  IDENT@[49; 50) "x"
-            PIPE@[50; 51)
-            WHITESPACE@[51; 52)
-          PATH_EXPR@[52; 53)
-            PATH@[52; 53)
-              PATH_SEGMENT@[52; 53)
-                NAME_REF@[52; 53)
-                  IDENT@[52; 53) "x"
-        SEMI@[53; 54)
-        WHITESPACE@[54; 59)
-      EXPR_STMT@[59; 72)
-        LAMBDA_EXPR@[59; 70)
-          PARAM_LIST@[59; 69)
-            PIPE@[59; 60)
-            PARAM@[60; 66)
-              BIND_PAT@[60; 61)
-                NAME@[60; 61)
-                  IDENT@[60; 61) "x"
-              COLON@[61; 62)
-              PATH_TYPE@[62; 66)
-                PATH@[62; 66)
-                  PATH_SEGMENT@[62; 66)
-                    NAME_REF@[62; 66)
-                      WHITESPACE@[62; 63)
-                      IDENT@[63; 66) "i32"
-            COMMA@[66; 67)
-            PIPE@[67; 68)
-            WHITESPACE@[68; 69)
-          PATH_EXPR@[69; 70)
-            PATH@[69; 70)
-              PATH_SEGMENT@[69; 70)
-                NAME_REF@[69; 70)
-                  IDENT@[69; 70) "x"
-        SEMI@[70; 71)
-        WHITESPACE@[71; 72)
-      R_CURLY@[72; 73)
-      WHITESPACE@[73; 74)

crates/ra_syntax/tests/data/parser/inline/0069_match_arm.txt

@@ -1,148 +0,0 @@
-SOURCE_FILE@[0; 161)
-  FN_DEF@[0; 160)
-    FN_KW@[0; 2)
-    WHITESPACE@[2; 3)
-    NAME@[3; 6)
-      IDENT@[3; 6) "foo"
-    PARAM_LIST@[6; 8)
-      L_PAREN@[6; 7)
-      R_PAREN@[7; 8)
-    WHITESPACE@[8; 9)
-    BLOCK@[9; 160)
-      L_CURLY@[9; 10)
-      WHITESPACE@[10; 15)
-      EXPR_STMT@[15; 158)
-        MATCH_EXPR@[15; 157)
-          MATCH_KW@[15; 20)
-          WHITESPACE@[20; 21)
-          TUPLE_EXPR@[21; 23)
-            L_PAREN@[21; 22)
-            R_PAREN@[22; 23)
-          WHITESPACE@[23; 24)
-          MATCH_ARM_LIST@[24; 157)
-            L_CURLY@[24; 25)
-            WHITESPACE@[25; 34)
-            MATCH_ARM@[34; 41)
-              PLACEHOLDER_PAT@[34; 35)
-                UNDERSCORE@[34; 35)
-              WHITESPACE@[35; 36)
-              FAT_ARROW@[36; 38)
-              WHITESPACE@[38; 39)
-              TUPLE_EXPR@[39; 41)
-                L_PAREN@[39; 40)
-                R_PAREN@[40; 41)
-            COMMA@[41; 42)
-            WHITESPACE@[42; 51)
-            MATCH_ARM@[51; 77)
-              PLACEHOLDER_PAT@[51; 52)
-                UNDERSCORE@[51; 52)
-              WHITESPACE@[52; 53)
-              IF_KW@[53; 55)
-              WHITESPACE@[55; 56)
-              BIN_EXPR@[56; 71)
-                PATH_EXPR@[56; 60)
-                  PATH@[56; 60)
-                    PATH_SEGMENT@[56; 60)
-                      NAME_REF@[56; 60)
-                        IDENT@[56; 60) "Test"
-                R_ANGLE@[60; 61)
-                BLOCK_EXPR@[61; 71)
-                  BLOCK@[61; 71)
-                    L_CURLY@[61; 62)
-                    EXPR_STMT@[62; 67)
-                      PATH_EXPR@[62; 67)
-                        PATH@[62; 67)
-                          PATH_SEGMENT@[62; 67)
-                            NAME_REF@[62; 67)
-                              IDENT@[62; 67) "field"
-                              err: `expected SEMI`
-                              err: `expected expression`
-                    EXPR_STMT@[67; 68)
-                      ERROR@[67; 68)
-                        COLON@[67; 68)
-                        err: `expected SEMI`
-                    WHITESPACE@[68; 69)
-                    LITERAL@[69; 70)
-                      INT_NUMBER@[69; 70) "0"
-                    R_CURLY@[70; 71)
-              WHITESPACE@[71; 72)
-              FAT_ARROW@[72; 74)
-              WHITESPACE@[74; 75)
-              TUPLE_EXPR@[75; 77)
-                L_PAREN@[75; 76)
-                R_PAREN@[76; 77)
-            COMMA@[77; 78)
-            WHITESPACE@[78; 87)
-            MATCH_ARM@[87; 103)
-              BIND_PAT@[87; 88)
-                NAME@[87; 88)
-                  IDENT@[87; 88) "X"
-              WHITESPACE@[88; 89)
-              PIPE@[89; 90)
-              WHITESPACE@[90; 91)
-              BIND_PAT@[91; 92)
-                NAME@[91; 92)
-                  IDENT@[91; 92) "Y"
-              WHITESPACE@[92; 93)
-              IF_KW@[93; 95)
-              WHITESPACE@[95; 96)
-              PATH_EXPR@[96; 97)
-                PATH@[96; 97)
-                  PATH_SEGMENT@[96; 97)
-                    NAME_REF@[96; 97)
-                      IDENT@[96; 97) "Z"
-              WHITESPACE@[97; 98)
-              FAT_ARROW@[98; 100)
-              WHITESPACE@[100; 101)
-              TUPLE_EXPR@[101; 103)
-                L_PAREN@[101; 102)
-                R_PAREN@[102; 103)
-            COMMA@[103; 104)
-            WHITESPACE@[104; 113)
-            MATCH_ARM@[113; 131)
-              PIPE@[113; 114)
-              WHITESPACE@[114; 115)
-              BIND_PAT@[115; 116)
-                NAME@[115; 116)
-                  IDENT@[115; 116) "X"
-              WHITESPACE@[116; 117)
-              PIPE@[117; 118)
-              WHITESPACE@[118; 119)
-              BIND_PAT@[119; 120)
-                NAME@[119; 120)
-                  IDENT@[119; 120) "Y"
-              WHITESPACE@[120; 121)
-              IF_KW@[121; 123)
-              WHITESPACE@[123; 124)
-              PATH_EXPR@[124; 125)
-                PATH@[124; 125)
-                  PATH_SEGMENT@[124; 125)
-                    NAME_REF@[124; 125)
-                      IDENT@[124; 125) "Z"
-              WHITESPACE@[125; 126)
-              FAT_ARROW@[126; 128)
-              WHITESPACE@[128; 129)
-              TUPLE_EXPR@[129; 131)
-                L_PAREN@[129; 130)
-                R_PAREN@[130; 131)
-            COMMA@[131; 132)
-            WHITESPACE@[132; 141)
-            MATCH_ARM@[141; 150)
-              PIPE@[141; 142)
-              WHITESPACE@[142; 143)
-              BIND_PAT@[143; 144)
-                NAME@[143; 144)
-                  IDENT@[143; 144) "X"
-              WHITESPACE@[144; 145)
-              FAT_ARROW@[145; 147)
-              WHITESPACE@[147; 148)
-              TUPLE_EXPR@[148; 150)
-                L_PAREN@[148; 149)
-                R_PAREN@[149; 150)
-            COMMA@[150; 151)
-            WHITESPACE@[151; 156)
-            R_CURLY@[156; 157)
-        SEMI@[157; 158)
-      WHITESPACE@[158; 159)
-      R_CURLY@[159; 160)
-  WHITESPACE@[160; 161)

crates/ra_syntax/tests/data/parser/inline/ok/0012_type_item_where_clause.txt

@@ -21,7 +21,6 @@ SOURCE_FILE@[0; 31)
             PATH_SEGMENT@[20; 24)
               NAME_REF@[20; 24)
                 IDENT@[20; 24) "Copy"
-                err: `expected COMMA`
     WHITESPACE@[24; 25)
     EQ@[25; 26)
     WHITESPACE@[26; 27)

crates/ra_syntax/tests/data/parser/inline/ok/0039_type_arg.rs

@@ -0,0 +1 @@
+type A = B<'static, i32, Item=u64>;