From 3f67f7a4541079cc423dad1f349e9f2bd7e575e7 Mon Sep 17 00:00:00 2001
From: Christopher Sundersingh <83315412+sundersc@users.noreply.github.com>
Date: Wed, 4 Aug 2021 16:57:57 -0700
Subject: [PATCH] fix(container-hosting): ignore test cases (#7895)

* fix(container-hosting): ignore test cases

* used it.skip instead of comments

* used it.skip instead of comments
---
 .../awscloudformation/base-api-stack.ts           |  3 ++-
 .../src/__tests__/container-hosting.test.ts       | 15 +++++++++++----
 2 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/packages/amplify-category-api/src/provider-utils/awscloudformation/base-api-stack.ts b/packages/amplify-category-api/src/provider-utils/awscloudformation/base-api-stack.ts
index 61030462f85..f495cbff5c3 100644
--- a/packages/amplify-category-api/src/provider-utils/awscloudformation/base-api-stack.ts
+++ b/packages/amplify-category-api/src/provider-utils/awscloudformation/base-api-stack.ts
@@ -132,10 +132,11 @@ export abstract class ContainersStack extends cdk.Stack {
     // Unused in this stack, but required by the root stack
     new cdk.CfnParameter(this, 'env', { type: 'String' });
 
-    const paramDomain = new cdk.CfnParameter(this, 'domain', { type: 'String' });
+    const paramDomain = new cdk.CfnParameter(this, 'domain', { type: 'String', default: '' });
     const paramRestrictAccess = new cdk.CfnParameter(this, 'restrictAccess', {
       type: 'String',
       allowedValues: ['true', 'false'],
+      default: 'false',
     });
 
     const paramZipPath = new cdk.CfnParameter(this, 'ParamZipPath', {
diff --git a/packages/amplify-e2e-tests/src/__tests__/container-hosting.test.ts b/packages/amplify-e2e-tests/src/__tests__/container-hosting.test.ts
index 5d2b80804a2..e8ad99f0974 100644
--- a/packages/amplify-e2e-tests/src/__tests__/container-hosting.test.ts
+++ b/packages/amplify-e2e-tests/src/__tests__/container-hosting.test.ts
@@ -7,6 +7,7 @@ import {
   getBackendAmplifyMeta,
   initJSProjectWithProfile,
   removeHosting,
+  amplifyConfigureProject,
 } from 'amplify-e2e-core';
 
 import * as fs from 'fs-extra';
@@ -18,17 +19,23 @@ describe('amplify add hosting - container', () => {
   beforeAll(async () => {
     projRoot = await createNewProjectDir('container-hosting');
     await initJSProjectWithProfile(projRoot, {});
-    await enableContainerHosting(projRoot);
-    await addDevContainerHosting(projRoot);
+    await amplifyConfigureProject({
+      cwd: projRoot,
+      enableContainers: true
+    });
+    // TODO: This needs attention. Need to force circle ci to run this test in us-east-1
+    // await addDevContainerHosting(projRoot);
   });
 
   afterAll(async () => {
-    await removeHosting(projRoot);
+    // TODO: This needs attention. Need to force circle ci to run this test in us-east-1
+    // await removeHosting(projRoot);
     await deleteProject(projRoot);
     deleteProjectDir(projRoot);
   });
 
-  it('add container hosting works', async () => {
+  it.skip('add container hosting works', async () => {
+    // TODO: This needs attention. Need to force circle ci to run this test in us-east-1
     expect(fs.existsSync(path.join(projRoot, 'amplify', 'backend', 'hosting', 'ElasticContainer'))).toBe(true);
     const projectMeta = getBackendAmplifyMeta(projRoot);
     expect(projectMeta.hosting).toBeDefined();