Include argument parentheses in range (#5)

# Conflicts:
#	parser/src/parser.rs
#	parser/src/python.rs
#	parser/src/snapshots/rustpython_parser__parser__tests__decorator_ranges.snap

Author: zanieb

parser/src/function.rs

@@ -136,7 +136,6 @@ mod tests {
     use super::*;
     use crate::{ast, parser::ParseErrorType, Parse};
 
-    #[cfg(feature = "all-nodes-with-ranges")]
     macro_rules! function_and_lambda {
         ($($name:ident: $code:expr,)*) => {
             $(
@@ -149,6 +148,12 @@ mod tests {
         }
     }
 
+    #[cfg(feature = "all-nodes-with-ranges")]
+    function_and_lambda! {
+        test_function_no_args_with_ranges: "def f(): pass",
+        test_function_pos_args_with_ranges: "def f(a, b, c): pass",
+    }
+
     #[cfg(feature = "all-nodes-with-ranges")]
     function_and_lambda! {
         test_function_no_args: "def f(): pass",

parser/src/parser.rs

@@ -1221,26 +1221,6 @@ def args_to_tuple(*args: *Ts) -> Tuple[*Ts]: ...
         insta::assert_debug_snapshot!(parse_ast);
     }
 
-    #[test]
-    #[cfg(not(feature = "all-nodes-with-ranges"))]
-    fn decorator_ranges() {
-        let parse_ast = parse_program(
-            r#"
-@my_decorator
-def test():
-    pass
-    
-@class_decorator
-class Abcd:
-    pass
-"#
-            .trim(),
-            "<test>",
-        )
-        .unwrap();
-        insta::assert_debug_snapshot!(parse_ast);
-    }
-
     #[test]
     fn test_parse_constant() {
         use num_traits::ToPrimitive;
@@ -1257,4 +1237,24 @@ class Abcd:
         let i = ast::Identifier::parse_without_path("test").unwrap();
         assert_eq!(i.as_str(), "test");
     }
+
+    #[test]
+    #[cfg(not(feature = "all-nodes-with-ranges"))]
+    fn decorator_ranges() {
+        let parse_ast = parse_program(
+            r#"
+@my_decorator
+def test():
+    pass
+
+@class_decorator
+class Abcd:
+    pass
+"#
+                .trim(),
+            "<test>",
+        )
+            .unwrap();
+        insta::assert_debug_snapshot!(parse_ast);
+    }
 }

parser/src/python.lalrpop

@@ -1001,8 +1001,14 @@ TypeAliasStatement: ast::Stmt = {
 Parameters: ast::Arguments = {
     <location:@L> "(" <a: (ParameterList<TypedParameter, StarTypedParameter, DoubleStarTypedParameter>)?> ")" <end_location:@R> =>? {
         a.as_ref().map(validate_arguments).transpose()?;
+
+        let range = optional_range(location, end_location);
         let args = a
-            .unwrap_or_else(|| ast::Arguments::empty(optional_range(location, end_location)));
+            .map(|mut arguments| {
+                arguments.range = range;
+                arguments
+            })
+            .unwrap_or_else(|| ast::Arguments::empty(range));
 
         Ok(args)
     }