fix(parser): Error on duplicate table created as dotted

Fixes #509
toml-rs · Feb 6, 2023 · fc59238 · fc59238
1 parent e006c77
commit fc59238
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/crates/toml_edit/src/parser/state.rs b/crates/toml_edit/src/parser/state.rs
@@ -117,6 +117,8 @@ impl ParseState {
 
         self.current_table_position += 1;
         self.current_table.decor = decor;
+        self.current_table.set_implicit(false);
+        self.current_table.set_dotted(false);
         self.current_table.set_position(self.current_table_position);
         self.current_table.span = Some(span);
         self.current_is_array = true;
@@ -152,6 +154,8 @@ impl ParseState {
 
         self.current_table_position += 1;
         self.current_table.decor = decor;
+        self.current_table.set_implicit(false);
+        self.current_table.set_dotted(false);
         self.current_table.set_position(self.current_table_position);
         self.current_table.span = Some(span);
         self.current_is_array = false;

diff --git a/crates/toml_edit/tests/testsuite/invalid.rs b/crates/toml_edit/tests/testsuite/invalid.rs
@@ -45,7 +45,7 @@ expected `]`
 
 #[test]
 fn duplicate_table_after_dotted_key_issue_509() {
-    "
+    let err = "
 [dependencies.foo]
 version = \"0.16\"
 
@@ -56,7 +56,17 @@ libc = \"0.2\"
 rand = \"0.3.14\"
 "
     .parse::<toml_edit::Document>()
-    .unwrap();
+    .unwrap_err();
+    snapbox::assert_eq(
+        r#"TOML parse error at line 8, column 1
+  |
+8 | [dependencies]
+  | ^
+invalid table header
+duplicate key `dependencies` in document root
+"#,
+        err.to_string(),
+    );
 }
 
 #[test]