fix duplicate definition test

scopegraphs/examples/records/ast.rs

@@ -1,5 +1,3 @@
-use std::collections::HashMap;
-
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub enum Type {
     StructRef(String),
@@ -9,14 +7,14 @@ pub enum Type {
 #[derive(Debug)]
 pub struct RecordDef {
     pub name: String,
-    pub fields: HashMap<String, Type>,
+    pub fields: Vec<(String, Type)>,
 }
 
 #[derive(Debug, Clone)]
 pub enum Expr {
     StructInit {
         name: String,
-        fields: HashMap<String, Expr>,
+        fields: Vec<(String, Expr)>,
     },
     #[allow(unused)]
     Add(Box<Expr>, Box<Expr>),
@@ -30,7 +28,7 @@ pub enum Expr {
         in_expr: Box<Expr>,
     },
     LetRec {
-        values: HashMap<String, Expr>,
+        values: Vec<(String, Expr)>,
         in_expr: Box<Expr>,
     },
 }

scopegraphs/examples/records/main.rs

@@ -392,7 +392,7 @@ mod tests {
     }
 
     #[test]
-    #[should_panic = "duplicate definition"]
+    #[should_panic = "variable did not resolve uniquely: OnlyElementError::Multiple {..}"]
     fn test_letrec_shadow() {
         test_example(
             "record A {} main = letrec a = new A {}; a = 42; in a;",

scopegraphs/examples/records/parse.rs

@@ -1,4 +1,3 @@
-use std::collections::HashMap;
 use winnow::ascii::multispace0;
 use winnow::combinator::{alt, delimited, opt, preceded, repeat, separated, terminated};
 use winnow::error::{ParserError, StrContext};
@@ -57,7 +56,7 @@ fn parse_field_def(input: &mut &'_ str) -> PResult<(String, Type)> {
     .parse_next(input)
 }
 
-fn parse_field_defs(input: &mut &'_ str) -> PResult<HashMap<String, Type>> {
+fn parse_field_defs(input: &mut &'_ str) -> PResult<Vec<(String, Type)>> {
     terminated(separated(0.., ws(parse_field_def), ws(",")), opt(ws(","))).parse_next(input)
 }
 
@@ -72,7 +71,7 @@ fn parse_field(input: &mut &'_ str) -> PResult<(String, Expr)> {
     .parse_next(input)
 }
 
-fn parse_fields(input: &mut &'_ str) -> PResult<HashMap<String, Expr>> {
+fn parse_fields(input: &mut &'_ str) -> PResult<Vec<(String, Expr)>> {
     terminated(separated(0.., ws(parse_field), ws(",")), opt(ws(","))).parse_next(input)
 }
 
@@ -97,7 +96,7 @@ fn parse_value(input: &mut &'_ str) -> PResult<(String, Expr)> {
     .parse_next(input)
 }
 
-fn parse_values(input: &mut &'_ str) -> PResult<HashMap<String, Expr>> {
+fn parse_values(input: &mut &'_ str) -> PResult<Vec<(String, Expr)>> {
     repeat(0.., parse_value).parse_next(input)
 }