fix: Connector "Type" cannot be Serverless resource types

samtranslator/model/connector/connector.py

@@ -89,6 +89,17 @@ def _is_valid_resource_reference(obj: Dict[str, Any]) -> bool:
     return id_provided != non_id_provided
 
 
+def get_underlying_cfn_resource_type(resource_type: str) -> str:
+    sam_to_cfn_types = {
+        "AWS::Serverless::Function": "AWS::Lambda::Function",
+        "AWS::Serverless::StateMachine": "AWS::StepFunctions::StateMachine",
+        "AWS::Serverless::Api": "AWS::ApiGateway::RestApi",
+        "AWS::Serverless::SimpleTable": "AWS::DynamoDB::Table",
+        "AWS::Serverless::HttpApi": "AWS::ApiGatewayV2::Api",
+    }
+    return sam_to_cfn_types.get(resource_type, resource_type)
+
+
 def get_resource_reference(
     obj: Dict[str, Any], resource_resolver: ResourceResolver, connecting_obj: Dict[str, Any]
 ) -> ConnectorResourceReference:
@@ -98,14 +109,14 @@ def get_resource_reference(
         )
 
     logical_id = obj.get("Id")
-
     # Must provide Id (with optional Qualifier) or a supported combination of other properties
     # If Id is not provided, all values must come from overrides.
     if not logical_id:
         resource_type = obj.get("Type")
+
         if not _is_nonblank_str(resource_type):
             raise ConnectorResourceError("'Type' is missing or not a string.")
-        resource_type = str(resource_type)
+        resource_type = get_underlying_cfn_resource_type(str(resource_type))
 
         return ConnectorResourceReference(
             logical_id=None,

tests/translator/input/connector_with_non_id_source_and_destination.yaml

@@ -0,0 +1,110 @@
+Transform: AWS::Serverless-2016-10-31
+
+Resources:
+  MyRole:
+    Type: AWS::IAM::Role
+    Properties:
+      AssumeRolePolicyDocument:
+        Statement:
+        - Effect: Allow
+          Action: sts:AssumeRole
+          Principal:
+            Service: lambda.amazonaws.com
+      ManagedPolicyArns:
+      - arn:{AWS::Partition}:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole
+
+  SamFunction:
+    Type: AWS::Serverless::Function
+    Properties:
+      Runtime: nodejs14.x
+      Handler: index.handler
+      Role: !GetAtt MyRole.Arn
+      InlineCode: |
+        const AWS = require('aws-sdk');
+        exports.handler = async (event) => {
+          console.log(JSON.stringify(event));
+        };
+
+  SamTable:
+    Type: AWS::Serverless::SimpleTable
+    Properties:
+      PrimaryKey:
+        Name: NoteId
+        Type: String
+
+  SamStateMachine:
+    Type: AWS::Serverless::StateMachine
+    Properties:
+      # Express state machine support sync execution
+      # which allows us to get the error message quickly in trigger function.
+      Type: EXPRESS
+      Role: !GetAtt MyRole.Arn
+      Definition:
+        StartAt: MyLambdaState
+        States:
+          MyLambdaState:
+            Type: Task
+            Resource: !GetAtt SamFunction.Arn
+            End: true
+
+  SamApi:
+    Type: AWS::Serverless::Api
+    Properties:
+      StageName: Prod
+      DefinitionUri: s3://sam-demo-bucket/webpage_swagger.json
+      Description: my description
+
+  SamHttpApi:
+    Type: AWS::Serverless::HttpApi
+    Properties:
+      StageName: Prod
+
+  Connector1:
+    Type: AWS::Serverless::Connector
+    Properties:
+      Source:
+        Type: AWS::Serverless::Function
+        RoleName: MyRole
+      Destination:
+        Type: AWS::Serverless::SimpleTable
+        Arn: !GetAtt SamTable.Arn
+      Permissions:
+      - Read
+
+  Connector2:
+    Type: AWS::Serverless::Connector
+    Properties:
+      Source:
+        Type: AWS::Serverless::Api
+        ResourceId: !Ref SamApi
+        Qualifier: Prod/GET/foobar
+      Destination:
+        Type: AWS::Serverless::Function
+        Arn: !GetAtt SamFunction.Arn
+      Permissions:
+      - Write
+
+  Connector3:
+    Type: AWS::Serverless::Connector
+    Properties:
+      Source:
+        Type: AWS::Serverless::StateMachine
+        RoleName: MyRole
+      Destination:
+        Type: AWS::Serverless::Function
+        Arn: !GetAtt SamFunction.Arn
+      Permissions:
+      - Write
+
+  Connector4:
+    Type: AWS::Serverless::Connector
+    Properties:
+      Source:
+        Type: AWS::Serverless::HttpApi
+        ResourceId: !Ref SamHttpApi
+        Qualifier: Prod/GET/foobar
+      Destination:
+        Type: AWS::Serverless::Function
+        Arn: !GetAtt SamFunction.Arn
+      Permissions:
+      - Write