Add calledExpression label to FunctionCallExpr convenience initializer

Otherwise initializing a `FunctionCallExpr` from a string is ambiguous whether it should be parsed or call the string

Author: ahoppen

Sources/SwiftSyntaxBuilder/ConvenienceInitializers/FunctionCallExprConvenienceInitializers.swift

@@ -18,7 +18,7 @@ extension FunctionCallExpr {
   /// instead of having to wrap them in a `TupleExprElementList`.
   /// The presence of the parenthesis will be inferred based on the presence of arguments and the trailing closure.
   public init(
-    _ calledExpression: ExpressibleAsExprBuildable,
+    calledExpression: ExpressibleAsExprBuildable,
     trailingClosure: ExpressibleAsClosureExpr? = nil,
     additionalTrailingClosures: MultipleTrailingClosureElementList? = nil,
     @TupleExprElementListBuilder argumentList: () -> ExpressibleAsTupleExprElementList = { [] }

Tests/SwiftSyntaxBuilderTest/DoStmtTests.swift

@@ -5,14 +5,14 @@ final class DoStmtTests: XCTestCase {
   func testDoStmt() {
     let buildable = DoStmt(
       body: CodeBlock(statementsBuilder: {
-        CodeBlockItem(item: TryExpr(expression: FunctionCallExpr(MemberAccessExpr(base: "a", name: "b"))))
+        CodeBlockItem(item: TryExpr(expression: FunctionCallExpr(calledExpression: MemberAccessExpr(base: "a", name: "b"))))
       }),
       catchClauses: [
         CatchClause(CatchItemList {
           CatchItem(pattern: "Error1")
           CatchItem(pattern: "Error2")
         }) {
-          FunctionCallExpr("print") {
+          FunctionCallExpr(calledExpression: "print") {
             TupleExprElement(expression: StringLiteralExpr("Known error"))
           }
         },
@@ -23,7 +23,7 @@ final class DoStmtTests: XCTestCase {
           ThrowStmt(expression: MemberAccessExpr(base: "Error4", name: "error3"))
         },
         CatchClause {
-          FunctionCallExpr("print") {
+          FunctionCallExpr(calledExpression: "print") {
             TupleExprElement(expression: "error")
           }
         }

Tests/SwiftSyntaxBuilderTest/ExpressibleBuildablesTests.swift

@@ -45,7 +45,7 @@ final class ExpressibleBuildablesTests: XCTestCase {
 
   func testExprExpressibleAsCodeBlockItem() {
     let myCodeBlock = CodeBlock(leftBrace: .leftBrace.withLeadingTrivia([])) {
-      FunctionCallExpr("print") { TupleExprElement(expression: StringLiteralExpr("Hello world")) }
+      FunctionCallExpr(calledExpression: "print") { TupleExprElement(expression: StringLiteralExpr("Hello world")) }
       IntegerLiteralExpr(42)
       "someIdentifier"
     }

Tests/SwiftSyntaxBuilderTest/FunctionTests.swift

@@ -26,7 +26,7 @@ final class FunctionTests: XCTestCase {
         body: ifCodeBlock)
 
       ReturnStmt(expression: SequenceExpr {
-        FunctionCallExpr("fibonacci") {
+        FunctionCallExpr(calledExpression: "fibonacci") {
           SequenceExpr {
             IntegerLiteralExpr(digits: "n")
             BinaryOperatorExpr("-")
@@ -36,7 +36,7 @@ final class FunctionTests: XCTestCase {
 
         BinaryOperatorExpr("+")
 
-        FunctionCallExpr(MemberAccessExpr(base: "self", name: "fibonacci")) {
+        FunctionCallExpr(calledExpression: MemberAccessExpr(base: "self", name: "fibonacci")) {
           SequenceExpr {
             IntegerLiteralExpr(digits: "n")
             BinaryOperatorExpr("-")
@@ -58,7 +58,7 @@ final class FunctionTests: XCTestCase {
   }
 
   func testArguments() {
-    let buildable = FunctionCallExpr("test") {
+    let buildable = FunctionCallExpr(calledExpression: "test") {
       for param in (1...5) {
         TupleExprElement(label: param.isMultiple(of: 2) ? "p\(param)" : nil, expression: "value\(param)")
       }
@@ -74,7 +74,7 @@ final class FunctionTests: XCTestCase {
   }
 
   func testParensEmittedForArgumentAndTrailingClosure() {
-    let buildable = FunctionCallExpr("test", trailingClosure: ClosureExpr()) {
+    let buildable = FunctionCallExpr(calledExpression: "test", trailingClosure: ClosureExpr()) {
       TupleExprElement(expression: "42")
     }
     let syntax = buildable.buildSyntax()
@@ -83,11 +83,11 @@ final class FunctionTests: XCTestCase {
 
   func testParensOmittedForNoArgumentsAndTrailingClosure() {
     let closure = ClosureExpr(statementsBuilder: {
-      FunctionCallExpr("f") {
+      FunctionCallExpr(calledExpression: "f") {
         TupleExprElement(expression: "a")
       }
     })
-    let buildable = FunctionCallExpr("test", trailingClosure: closure)
+    let buildable = FunctionCallExpr(calledExpression: "test", trailingClosure: closure)
     let syntax = buildable.buildSyntax()
     XCTAssertEqual(
       syntax.description,

Tests/SwiftSyntaxBuilderTest/IfStmtTests.swift

@@ -19,11 +19,11 @@ final class IfStmtTests: XCTestCase {
     // Use the convenience initializer from IfStmtConvenienceInitializers
     // with an else branch expressed by a second trailing closure.
     let buildable = IfStmt(conditions: ExprList([BooleanLiteralExpr(true)])) {
-      FunctionCallExpr("print") {
+      FunctionCallExpr(calledExpression: "print") {
         TupleExprElement(expression: StringLiteralExpr("Hello from the if-branch!"))
       }
     } elseBody: {
-      FunctionCallExpr("print") {
+      FunctionCallExpr(calledExpression: "print") {
         TupleExprElement(expression: StringLiteralExpr("Hello from the else-branch!"))
       }
     }