revert(cli): 'deploy' and 'diff' silently does nothing when given unk…

…nown stack name (#16125)

Broke integration tests when `synth`ing without providing a stack name

reverts #16073

packages/aws-cdk/lib/cdk-toolkit.ts

@@ -397,7 +397,7 @@ export class CdkToolkit {
       defaultBehavior: DefaultSelection.OnlySingle,
     });
 
-    await this.validateStacks(stacks, selector.patterns);
+    await this.validateStacks(stacks);
 
     return stacks;
   }
@@ -415,7 +415,7 @@ export class CdkToolkit {
       ? allStacks.filter(art => art.validateOnSynth ?? false)
       : new StackCollection(assembly, []);
 
-    await this.validateStacks(selectedForDiff.concat(autoValidateStacks), stackNames);
+    await this.validateStacks(selectedForDiff.concat(autoValidateStacks));
 
     return selectedForDiff;
   }
@@ -435,11 +435,7 @@ export class CdkToolkit {
   /**
    * Validate the stacks for errors and warnings according to the CLI's current settings
    */
-  private async validateStacks(stacks: StackCollection, stackNames: string[]) {
-    if (stacks.stackCount == 0) {
-      throw new Error(`No stacks match the name(s) ${stackNames}`);
-    }
-
+  private async validateStacks(stacks: StackCollection) {
     stacks.processMetadataMessages({
       ignoreErrors: this.props.ignoreErrors,
       strict: this.props.strict,

packages/aws-cdk/test/cdk-toolkit.test.ts

@@ -39,14 +39,6 @@ function defaultToolkitSetup() {
 }
 
 describe('deploy', () => {
-  test('fails when no valid stack names are given', async () => {
-    // GIVEN
-    const toolkit = defaultToolkitSetup();
-
-    // WHEN
-    await expect(() => toolkit.deploy({ selector: { patterns: ['Test-Stack-D'] } })).rejects.toThrow('No stacks match the name(s) Test-Stack-D');
-  });
-
   describe('makes correct CloudFormation calls', () => {
     test('without options', async () => {
       // GIVEN

packages/aws-cdk/test/diff.test.ts

@@ -121,16 +121,6 @@ test('throws an error during diffs on stack with error metadata', async () => {
   })).rejects.toThrow(/Found errors/);
 });
 
-test('throws an error if no valid stack names given', async () => {
-  const buffer = new StringWritable();
-
-  // WHEN
-  await expect(() => toolkit.diff({
-    stackNames: ['X', 'Y', 'Z'],
-    stream: buffer,
-  })).rejects.toThrow('No stacks match the name(s) X,Y,Z');
-});
-
 class StringWritable extends Writable {
   public data: string;
   private readonly _decoder: NodeStringDecoder;