mirrors/RustPython-Parser
1
0
Fork 0
mirror of https://github.com/RustPython/Parser.git synced 2025-07-08 05:35:22 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Allow trailing commas in MappingPattern

Browse source
This commit is contained in:
Charlie Marsh 2023-02-22 10:01:30 -05:00
parent 2a8aa6f308
commit b61f4d7b69
4 changed files with 983 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
parser/python.lalrpop
View file

@ -801,7 +801,7 @@ MappingPattern: ast::PatternKind = {
rest: None,
};
},
<location:@L> "{" <e:OneOrMore<MatchMappingEntry>> "}" <end_location:@R> => {
<location:@L> "{" <e:OneOrMore<MatchMappingEntry>> ","? "}" <end_location:@R> => {
let (keys, patterns) = e
.into_iter()
.unzip();
@ -811,14 +811,14 @@ MappingPattern: ast::PatternKind = {
rest: None,
};
},
<location:@L> "{" "**" <rest:Identifier> "}" <end_location:@R> => {
<location:@L> "{" "**" <rest:Identifier> ","? "}" <end_location:@R> => {
return ast::PatternKind::MatchMapping {
keys: vec![],
patterns: vec![],
rest: Some(rest),
};
},
<location:@L> "{" <e:OneOrMore<MatchMappingEntry>> "," "**" <rest:Identifier> "}" <end_location:@R> => {
<location:@L> "{" <e:OneOrMore<MatchMappingEntry>> "," "**" <rest:Identifier> ","? "}" <end_location:@R> => {
let (keys, patterns) = e
.into_iter()
.unzip();

32
parser/src/parser.rs
View file

@ -497,7 +497,7 @@ match match:
}
#[test]
fn test_match_complex() {
fn test_patma() {
let source = r#"# Cases sampled from Lib/test/test_patma.py
# case test_patma_098
@ -666,4 +666,34 @@ match w := x,:
let parse_ast = parse_program(source, "<test>").unwrap();
insta::assert_debug_snapshot!(parse_ast);
}
#[test]
fn test_match() {
let parse_ast = parse_program(
r#"
match {"test": 1}:
case {
**rest,
}:
print(rest)
match {"label": "test"}:
case {
"label": str() | None as label,
}:
print(label)
match x:
case [0, 1,]:
y = 0
match x:
case (0, 1,):
y = 0
match x:
case (0,):
y = 0
"#,
"<test>",
)
.unwrap();
insta::assert_debug_snapshot!(parse_ast);
}
}

949
parser/src/snapshots/rustpython_parser__parser__tests__match.snap Normal file
View file

@ -0,0 +1,949 @@
---
source: compiler/parser/src/parser.rs
expression: parse_ast
---
[
Located {
location: Location {
row: 2,
column: 0,
},
end_location: Some(
Location {
row: 7,
column: 0,
},
),
custom: (),
node: Match {
subject: Located {
location: Location {
row: 2,
column: 6,
},
end_location: Some(
Location {
row: 2,
column: 17,
},
),
custom: (),
node: Dict {
keys: [
Some(
Located {
location: Location {
row: 2,
column: 7,
},
end_location: Some(
Location {
row: 2,
column: 13,
},
),
custom: (),
node: Constant {
value: Str(
"test",
),
kind: None,
},
},
),
],
values: [
Located {
location: Location {
row: 2,
column: 15,
},
end_location: Some(
Location {
row: 2,
column: 16,
},
),
custom: (),
node: Constant {
value: Int(
1,
),
kind: None,
},
},
],
},
},
cases: [
MatchCase {
pattern: Located {
location: Location {
row: 3,
column: 9,
},
end_location: Some(
Location {
row: 5,
column: 5,
},
),
custom: (),
node: MatchMapping {
keys: [],
patterns: [],
rest: Some(
"rest",
),
},
},
guard: None,
body: [
Located {
location: Location {
row: 6,
column: 8,
},
end_location: Some(
Location {
row: 6,
column: 19,
},
),
custom: (),
node: Expr {
value: Located {
location: Location {
row: 6,
column: 8,
},
end_location: Some(
Location {
row: 6,
column: 19,
},
),
custom: (),
node: Call {
func: Located {
location: Location {
row: 6,
column: 8,
},
end_location: Some(
Location {
row: 6,
column: 13,
},
),
custom: (),
node: Name {
id: "print",
ctx: Load,
},
},
args: [
Located {
location: Location {
row: 6,
column: 14,
},
end_location: Some(
Location {
row: 6,
column: 18,
},
),
custom: (),
node: Name {
id: "rest",
ctx: Load,
},
},
],
keywords: [],
},
},
},
},
],
},
],
},
},
Located {
location: Location {
row: 7,
column: 0,
},
end_location: Some(
Location {
row: 12,
column: 0,
},
),
custom: (),
node: Match {
subject: Located {
location: Location {
row: 7,
column: 6,
},
end_location: Some(
Location {
row: 7,
column: 23,
},
),
custom: (),
node: Dict {
keys: [
Some(
Located {
location: Location {
row: 7,
column: 7,
},
end_location: Some(
Location {
row: 7,
column: 14,
},
),
custom: (),
node: Constant {
value: Str(
"label",
),
kind: None,
},
},
),
],
values: [
Located {
location: Location {
row: 7,
column: 16,
},
end_location: Some(
Location {
row: 7,
column: 22,
},
),
custom: (),
node: Constant {
value: Str(
"test",
),
kind: None,
},
},
],
},
},
cases: [
MatchCase {
pattern: Located {
location: Location {
row: 8,
column: 9,
},
end_location: Some(
Location {
row: 10,
column: 5,
},
),
custom: (),
node: MatchMapping {
keys: [
Located {
location: Location {
row: 9,
column: 8,
},
end_location: Some(
Location {
row: 9,
column: 15,
},
),
custom: (),
node: Constant {
value: Str(
"label",
),
kind: None,
},
},
],
patterns: [
Located {
location: Location {
row: 9,
column: 17,
},
end_location: Some(
Location {
row: 9,
column: 38,
},
),
custom: (),
node: MatchAs {
pattern: Some(
Located {
location: Location {
row: 9,
column: 17,
},
end_location: Some(
Location {
row: 9,
column: 29,
},
),
custom: (),
node: MatchOr {
patterns: [
Located {
location: Location {
row: 9,
column: 17,
},
end_location: Some(
Location {
row: 9,
column: 22,
},
),
custom: (),
node: MatchClass {
cls: Located {
location: Location {
row: 9,
column: 17,
},
end_location: Some(
Location {
row: 9,
column: 20,
},
),
custom: (),
node: Name {
id: "str",
ctx: Load,
},
},
patterns: [],
kwd_attrs: [],
kwd_patterns: [],
},
},
Located {
location: Location {
row: 9,
column: 25,
},
end_location: Some(
Location {
row: 9,
column: 29,
},
),
custom: (),
node: MatchSingleton {
value: None,
},
},
],
},
},
),
name: Some(
"label",
),
},
},
],
rest: None,
},
},
guard: None,
body: [
Located {
location: Location {
row: 11,
column: 8,
},
end_location: Some(
Location {
row: 11,
column: 20,
},
),
custom: (),
node: Expr {
value: Located {
location: Location {
row: 11,
column: 8,
},
end_location: Some(
Location {
row: 11,
column: 20,
},
),
custom: (),
node: Call {
func: Located {
location: Location {
row: 11,
column: 8,
},
end_location: Some(
Location {
row: 11,
column: 13,
},
),
custom: (),
node: Name {
id: "print",
ctx: Load,
},
},
args: [
Located {
location: Location {
row: 11,
column: 14,
},
end_location: Some(
Location {
row: 11,
column: 19,
},
),
custom: (),
node: Name {
id: "label",
ctx: Load,
},
},
],
keywords: [],
},
},
},
},
],
},
],
},
},
Located {
location: Location {
row: 12,
column: 0,
},
end_location: Some(
Location {
row: 15,
column: 0,
},
),
custom: (),
node: Match {
subject: Located {
location: Location {
row: 12,
column: 6,
},
end_location: Some(
Location {
row: 12,
column: 7,
},
),
custom: (),
node: Name {
id: "x",
ctx: Load,
},
},
cases: [
MatchCase {
pattern: Located {
location: Location {
row: 13,
column: 9,
},
end_location: Some(
Location {
row: 13,
column: 16,
},
),
custom: (),
node: MatchSequence {
patterns: [
Located {
location: Location {
row: 13,
column: 10,
},
end_location: Some(
Location {
row: 13,
column: 11,
},
),
custom: (),
node: MatchValue {
value: Located {
location: Location {
row: 13,
column: 10,
},
end_location: Some(
Location {
row: 13,
column: 11,
},
),
custom: (),
node: Constant {
value: Int(
0,
),
kind: None,
},
},
},
},
Located {
location: Location {
row: 13,
column: 13,
},
end_location: Some(
Location {
row: 13,
column: 14,
},
),
custom: (),
node: MatchValue {
value: Located {
location: Location {
row: 13,
column: 13,
},
end_location: Some(
Location {
row: 13,
column: 14,
},
),
custom: (),
node: Constant {
value: Int(
1,
),
kind: None,
},
},
},
},
],
},
},
guard: None,
body: [
Located {
location: Location {
row: 14,
column: 8,
},
end_location: Some(
Location {
row: 14,
column: 13,
},
),
custom: (),
node: Assign {
targets: [
Located {
location: Location {
row: 14,
column: 8,
},
end_location: Some(
Location {
row: 14,
column: 9,
},
),
custom: (),
node: Name {
id: "y",
ctx: Store,
},
},
],
value: Located {
location: Location {
row: 14,
column: 12,
},
end_location: Some(
Location {
row: 14,
column: 13,
},
),
custom: (),
node: Constant {
value: Int(
0,
),
kind: None,
},
},
type_comment: None,
},
},
],
},
],
},
},
Located {
location: Location {
row: 15,
column: 0,
},
end_location: Some(
Location {
row: 18,
column: 0,
},
),
custom: (),
node: Match {
subject: Located {
location: Location {
row: 15,
column: 6,
},
end_location: Some(
Location {
row: 15,
column: 7,
},
),
custom: (),
node: Name {
id: "x",
ctx: Load,
},
},
cases: [
MatchCase {
pattern: Located {
location: Location {
row: 16,
column: 9,
},
end_location: Some(
Location {
row: 16,
column: 16,
},
),
custom: (),
node: MatchSequence {
patterns: [
Located {
location: Location {
row: 16,
column: 10,
},
end_location: Some(
Location {
row: 16,
column: 11,
},
),
custom: (),
node: MatchValue {
value: Located {
location: Location {
row: 16,
column: 10,
},
end_location: Some(
Location {
row: 16,
column: 11,
},
),
custom: (),
node: Constant {
value: Int(
0,
),
kind: None,
},
},
},
},
Located {
location: Location {
row: 16,
column: 13,
},
end_location: Some(
Location {
row: 16,
column: 14,
},
),
custom: (),
node: MatchValue {
value: Located {
location: Location {
row: 16,
column: 13,
},
end_location: Some(
Location {
row: 16,
column: 14,
},
),
custom: (),
node: Constant {
value: Int(
1,
),
kind: None,
},
},
},
},
],
},
},
guard: None,
body: [
Located {
location: Location {
row: 17,
column: 8,
},
end_location: Some(
Location {
row: 17,
column: 13,
},
),
custom: (),
node: Assign {
targets: [
Located {
location: Location {
row: 17,
column: 8,
},
end_location: Some(
Location {
row: 17,
column: 9,
},
),
custom: (),
node: Name {
id: "y",
ctx: Store,
},
},
],
value: Located {
location: Location {
row: 17,
column: 12,
},
end_location: Some(
Location {
row: 17,
column: 13,
},
),
custom: (),
node: Constant {
value: Int(
0,
),
kind: None,
},
},
type_comment: None,
},
},
],
},
],
},
},
Located {
location: Location {
row: 18,
column: 0,
},
end_location: Some(
Location {
row: 21,
column: 0,
},
),
custom: (),
node: Match {
subject: Located {
location: Location {
row: 18,
column: 6,
},
end_location: Some(
Location {
row: 18,
column: 7,
},
),
custom: (),
node: Name {
id: "x",
ctx: Load,
},
},
cases: [
MatchCase {
pattern: Located {
location: Location {
row: 19,
column: 9,
},
end_location: Some(
Location {
row: 19,
column: 13,
},
),
custom: (),
node: MatchSequence {
patterns: [
Located {
location: Location {
row: 19,
column: 10,
},
end_location: Some(
Location {
row: 19,
column: 11,
},
),
custom: (),
node: MatchValue {
value: Located {
location: Location {
row: 19,
column: 10,
},
end_location: Some(
Location {
row: 19,
column: 11,
},
),
custom: (),
node: Constant {
value: Int(
0,
),
kind: None,
},
},
},
},
],
},
},
guard: None,
body: [
Located {
location: Location {
row: 20,
column: 8,
},
end_location: Some(
Location {
row: 20,
column: 13,
},
),
custom: (),
node: Assign {
targets: [
Located {
location: Location {
row: 20,
column: 8,
},
end_location: Some(
Location {
row: 20,
column: 9,
},
),
custom: (),
node: Name {
id: "y",
ctx: Store,
},
},
],
value: Located {
location: Location {
row: 20,
column: 12,
},
end_location: Some(
Location {
row: 20,
column: 13,
},
),
custom: (),
node: Constant {
value: Int(
0,
),
kind: None,
},
},
type_comment: None,
},
},
],
},
],
},
},
]

0
parser/src/snapshots/rustpython_parser__parser__tests__match_complex.snap → parser/src/snapshots/rustpython_parser__parser__tests__patma.snap
View file