Skip to content

Commit

Permalink
Revert "fix pipeline unit tests"
Browse files Browse the repository at this point in the history
This reverts commit 19532af.
  • Loading branch information
Niranjan Jayakar committed Feb 9, 2021
1 parent b180258 commit 09d87d0
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions packages/@aws-cdk/aws-codepipeline/test/pipeline.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import * as kms from '@aws-cdk/aws-kms';
import * as s3 from '@aws-cdk/aws-s3';
import * as cdk from '@aws-cdk/core';
import * as cxapi from '@aws-cdk/cx-api';
import { testLegacyBehavior } from 'cdk-build-tools/lib/feature-flag';
import * as codepipeline from '../lib';
import { FakeBuildAction } from './fake-build-action';
import { FakeSourceAction } from './fake-source-action';
Expand Down Expand Up @@ -68,8 +67,9 @@ describe('', () => {

});

// only support legacy behavior. see https://github.com/aws/aws-cdk/issues/12921
testLegacyBehavior('allows passing an Alias in place of the KMS Key in the replication Bucket', cdk.App, (app) => {
test('allows passing an Alias in place of the KMS Key in the replication Bucket', () => {
const app = new cdk.App();

const replicationRegion = 'us-west-1';
const replicationStack = new cdk.Stack(app, 'ReplicationStack', {
env: { region: replicationRegion, account: '123456789012' },
Expand Down Expand Up @@ -175,8 +175,8 @@ describe('', () => {

});

// only support legacy behavior. see https://github.com/aws/aws-cdk/issues/12921
testLegacyBehavior('generates ArtifactStores with the alias ARN as the KeyID', cdk.App, (app) => {
test('generates ArtifactStores with the alias ARN as the KeyID', () => {
const app = new cdk.App();
const replicationRegion = 'us-west-1';

const pipelineRegion = 'us-west-2';
Expand Down Expand Up @@ -298,8 +298,7 @@ describe('', () => {

});

// only support legacy behavior. see https://github.com/aws/aws-cdk/issues/12921
testLegacyBehavior('generates the support stack containing the replication Bucket without the need to bootstrap in that environment', cdk.App, () => {
test('generates the support stack containing the replication Bucket without the need to bootstrap in that environment', () => {
const app = new cdk.App({
treeMetadata: false, // we can't set the context otherwise, because App will have a child
});
Expand Down

0 comments on commit 09d87d0

Please sign in to comment.