fix: Don't add second ? to optional methods (#88)

Author: Liamolucko

src/parsing/parser.rs

@@ -618,7 +618,7 @@ fn parse_identifier<'a>(node: &'a Ident, context: &mut Context<'a>) -> PrintItem
     let mut items = PrintItems::new();
     items.push_str(node.sym() as &str);
 
-    if node.optional() && !node.parent().unwrap().is::<ClassProp>() {
+    if node.optional() && !node.parent().unwrap().is::<ClassProp>() && !node.parent().unwrap().is::<ClassMethod>() {
         items.push_str("?");
     }
     if let Node::VarDeclarator(node) = node.parent().unwrap() {

tests/specs/declarations/class/method/ClassMethod_All.txt

@@ -33,6 +33,9 @@ abstract class Test {
     method8(): string | number | string | number | string {
     }
 
+    method9?() {
+    }
+
     get getSet(): string {
     }
 
@@ -82,6 +85,9 @@ abstract class Test {
     {
     }
 
+    method9?() {
+    }
+
     get getSet(): string {
     }
 

tests/specs/declarations/class/private_method/PrivateMethod_All.txt

@@ -4,11 +4,17 @@ abstract class Test {
     #method1(   arg   : string  )   : test {
         return    5   ;
     }
+
+    #method2?() {
+    }
 }
 
 [expect]
 abstract class Test {
     #method1(arg: string): test {
         return 5;
     }
+
+    #method2?() {
+    }
 }