fix(apig): Adding rest api name for API resource

gradientedge · Dec 22, 2021 · 7f88469 · 7f88469
1 parent 43b2984
commit 7f88469
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/src/lib/manager/apiManager.ts b/src/lib/manager/apiManager.ts
@@ -65,6 +65,7 @@ export class ApiManager {
         types: props.endpointConfiguration?.types || [apig.EndpointType.REGIONAL],
         vpcEndpoints: props.endpointConfiguration?.vpcEndpoints,
       },
+      restApiName: `${props.restApiName}-${scope.props.stage}`,
       handler: lambdaFunction,
       defaultCorsPreflightOptions: {
         allowOrigins: props.defaultCorsPreflightOptions?.allowOrigins || apig.Cors.ALL_ORIGINS,

diff --git a/src/lib/manager/appConfigManager.ts b/src/lib/manager/appConfigManager.ts
@@ -68,7 +68,7 @@ export class AppConfigManager {
   public createApplication(id: string, scope: CommonConstruct, props: AppConfigProps): appconfig.CfnApplication {
     if (!props) throw `AppConfig props undefined`
 
-    const application = new appconfig.CfnApplication(scope, `${id}Application`, {
+    const application = new appconfig.CfnApplication(scope, `${id}`, {
       name: `${props.application.name}-${scope.props.stage}`,
       description: props.application.description,
       tags: props.application.tags,
@@ -96,7 +96,7 @@ export class AppConfigManager {
   ): appconfig.CfnEnvironment {
     if (!props) throw `AppConfig props undefined`
 
-    const environment = new appconfig.CfnEnvironment(scope, `${id}Environment`, {
+    const environment = new appconfig.CfnEnvironment(scope, `${id}`, {
       applicationId: applicationId,
       name: scope.props.stage,
       description: props.environment.description,
@@ -127,7 +127,7 @@ export class AppConfigManager {
   ): appconfig.CfnConfigurationProfile {
     if (!props) throw `AppConfig props undefined`
 
-    const profile = new appconfig.CfnConfigurationProfile(scope, `${id}ConfigurationProfile`, {
+    const profile = new appconfig.CfnConfigurationProfile(scope, `${id}`, {
       applicationId: applicationId,
       locationUri: props.configurationProfile.locationUri || 'hosted',
       name: `${props.configurationProfile.name}-${scope.props.stage}`,

diff --git a/src/test/construct/graphQLApiLambda.test.ts b/src/test/construct/graphQLApiLambda.test.ts
@@ -64,6 +64,7 @@ class TestGraphQLApiConstruct extends GraphQLApiLambda {
     this.props.graphQLApiSource = new lambda.AssetCode('src/test/common/nodejs/lib')
     this.props.graphqlRestApi = {
       deploy: true,
+      restApiName: 'test-lambda-rest-api',
       deployOptions: {
         description: `${this.id} - ${this.props.stage} stage`,
         stageName: this.props.stage,
@@ -146,7 +147,8 @@ describe('TestGraphQLApiLambdaConstruct', () => {
       EndpointConfiguration: {
         Types: ['REGIONAL'],
       },
-      Name: 'test-graphql-lambda-rest-api',
+      FailOnWarnings: false,
+      Name: 'test-lambda-rest-api-test',
     })
   })
 })