internal: move inline parser tests to parser crate

Author: matklad

Cargo.lock

@@ -16,3 +16,5 @@ limit = { path = "../limit", version = "0.0.0" }
 
 [dev-dependencies]
 expect-test = "1.2"
+sourcegen = { path = "../sourcegen" }
+

crates/parser/Cargo.toml

@@ -463,13 +463,6 @@ fn method_call_expr(p: &mut Parser, lhs: CompletedMarker) -> CompletedMarker {
 //     x.0.bar;
 //     x.0();
 // }
-
-// test_err bad_tuple_index_expr
-// fn foo() {
-//     x.0.;
-//     x.1i32;
-//     x.0x01;
-// }
 fn field_expr(p: &mut Parser, lhs: CompletedMarker) -> CompletedMarker {
     assert!(p.at(T![.]));
     let m = lhs.precede(p);

crates/parser/src/grammar/expressions.rs

@@ -1,3 +1,5 @@
+mod sourcegen_inline_tests;
+
 use std::{
     fmt::Write,
     fs,
@@ -9,15 +11,15 @@ use expect_test::expect_file;
 use crate::LexedStr;
 
 #[test]
-fn lex_valid() {
+fn lex_ok() {
     for case in TestCase::list("lexer/ok") {
         let actual = lex(&case.text);
         expect_file![case.txt].assert_eq(&actual)
     }
 }
 
 #[test]
-fn lex_invalid() {
+fn lex_err() {
     for case in TestCase::list("lexer/err") {
         let actual = lex(&case.text);
         expect_file![case.txt].assert_eq(&actual)
@@ -40,7 +42,7 @@ fn lex(text: &str) -> String {
 }
 
 #[test]
-fn parse_valid() {
+fn parse_ok() {
     for case in TestCase::list("parser/ok") {
         let (actual, errors) = parse(&case.text);
         assert!(!errors, "errors in an OK file {}:\n{}", case.rs.display(), actual);
@@ -49,14 +51,32 @@ fn parse_valid() {
 }
 
 #[test]
-fn parse_invalid() {
+fn parse_inline_ok() {
+    for case in TestCase::list("parser/inline/ok") {
+        let (actual, errors) = parse(&case.text);
+        assert!(!errors, "errors in an OK file {}:\n{}", case.rs.display(), actual);
+        expect_file![case.txt].assert_eq(&actual);
+    }
+}
+
+#[test]
+fn parse_err() {
     for case in TestCase::list("parser/err") {
         let (actual, errors) = parse(&case.text);
         assert!(errors, "no errors in an ERR file {}:\n{}", case.rs.display(), actual);
         expect_file![case.txt].assert_eq(&actual)
     }
 }
 
+#[test]
+fn parse_inline_err() {
+    for case in TestCase::list("parser/inline/err") {
+        let (actual, errors) = parse(&case.text);
+        assert!(errors, "no errors in an ERR file {}:\n{}", case.rs.display(), actual);
+        expect_file![case.txt].assert_eq(&actual)
+    }
+}
+
 fn parse(text: &str) -> (String, bool) {
     let lexed = LexedStr::new(text);
     let input = lexed.to_input();

crates/parser/src/tests.rs

@@ -2,21 +2,20 @@
 //! them into tests.
 
 use std::{
+    collections::HashMap,
     fs, iter,
     path::{Path, PathBuf},
 };
 
-use rustc_hash::FxHashMap;
-
 #[test]
 fn sourcegen_parser_tests() {
     let grammar_dir = sourcegen::project_root().join(Path::new("crates/parser/src/grammar"));
     let tests = tests_from_dir(&grammar_dir);
 
-    install_tests(&tests.ok, "crates/syntax/test_data/parser/inline/ok");
-    install_tests(&tests.err, "crates/syntax/test_data/parser/inline/err");
+    install_tests(&tests.ok, "crates/parser/test_data/parser/inline/ok");
+    install_tests(&tests.err, "crates/parser/test_data/parser/inline/err");
 
-    fn install_tests(tests: &FxHashMap<String, Test>, into: &str) {
+    fn install_tests(tests: &HashMap<String, Test>, into: &str) {
         let tests_dir = sourcegen::project_root().join(into);
         if !tests_dir.is_dir() {
             fs::create_dir_all(&tests_dir).unwrap();
@@ -51,8 +50,8 @@ struct Test {
 
 #[derive(Default, Debug)]
 struct Tests {
-    ok: FxHashMap<String, Test>,
-    err: FxHashMap<String, Test>,
+    ok: HashMap<String, Test>,
+    err: HashMap<String, Test>,
 }
 
 fn collect_tests(s: &str) -> Vec<Test> {
@@ -102,8 +101,8 @@ fn tests_from_dir(dir: &Path) -> Tests {
     }
 }
 
-fn existing_tests(dir: &Path, ok: bool) -> FxHashMap<String, (PathBuf, Test)> {
-    let mut res = FxHashMap::default();
+fn existing_tests(dir: &Path, ok: bool) -> HashMap<String, (PathBuf, Test)> {
+    let mut res = HashMap::default();
     for file in fs::read_dir(dir).unwrap() {
         let file = file.unwrap();
         let path = file.path();

crates/syntax/src/tests/sourcegen_tests.rs renamed to crates/parser/src/tests/sourcegen_inline_tests.rs

@@ -0,0 +1,27 @@
+SOURCE_FILE
+  TYPE_ALIAS
+    TYPE_KW "type"
+    WHITESPACE " "
+    NAME
+      IDENT "T"
+    WHITESPACE " "
+    EQ "="
+    WHITESPACE " "
+    SLICE_TYPE
+      L_BRACK "["
+      TUPLE_TYPE
+        L_PAREN "("
+        R_PAREN ")"
+  WHITESPACE " "
+  ERROR
+    INT_NUMBER "92"
+  ERROR
+    R_BRACK "]"
+  ERROR
+    SEMICOLON ";"
+  WHITESPACE "\n"
+error 12: expected `;` or `]`
+error 12: expected SEMICOLON
+error 13: expected an item
+error 15: expected an item
+error 16: expected an item