From f212e66542d14e1bea89ac0c2d7028279953e7d4 Mon Sep 17 00:00:00 2001 From: Ryan Pelaez Date: Wed, 14 Jun 2023 18:18:59 -0700 Subject: [PATCH] Updated exception wording for region validation in StreamArn to be more consistent with other error messages --- .../kinesis/multilang/config/KinesisClientLibConfigurator.java | 2 +- .../amazon/kinesis/multilang/MultiLangDaemonConfigTest.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/amazon-kinesis-client-multilang/src/main/java/software/amazon/kinesis/multilang/config/KinesisClientLibConfigurator.java b/amazon-kinesis-client-multilang/src/main/java/software/amazon/kinesis/multilang/config/KinesisClientLibConfigurator.java index 9a9a0537..42fdc295 100644 --- a/amazon-kinesis-client-multilang/src/main/java/software/amazon/kinesis/multilang/config/KinesisClientLibConfigurator.java +++ b/amazon-kinesis-client-multilang/src/main/java/software/amazon/kinesis/multilang/config/KinesisClientLibConfigurator.java @@ -88,7 +88,7 @@ public class KinesisClientLibConfigurator { //Parse out the region from the Arn and set (and/or override existing value for region) Region regionObj = Region.of(streamArnObj.region().get()); if(Region.regions().stream().filter(x -> x.id().equalsIgnoreCase(regionObj.id())).count() == 0){ - throw new IllegalArgumentException(String.format("%s is not a valid region", regionObj.id())); + throw new IllegalArgumentException(String.format("StreamArn has unsupported region of '%s'.", regionObj.id())); } configuration.setRegionName(regionObj); } else { diff --git a/amazon-kinesis-client-multilang/src/test/java/software/amazon/kinesis/multilang/MultiLangDaemonConfigTest.java b/amazon-kinesis-client-multilang/src/test/java/software/amazon/kinesis/multilang/MultiLangDaemonConfigTest.java index 80715d1d..188fcd30 100644 --- a/amazon-kinesis-client-multilang/src/test/java/software/amazon/kinesis/multilang/MultiLangDaemonConfigTest.java +++ b/amazon-kinesis-client-multilang/src/test/java/software/amazon/kinesis/multilang/MultiLangDaemonConfigTest.java @@ -98,7 +98,7 @@ public class MultiLangDaemonConfigTest { @Test public void testConstructorFailsBecauseStreamArnHasInvalidRegion() throws Exception { setup("", "arn:aws:kinesis:us-east-1000:ACCOUNT_ID:stream/streamName"); - assertConstructorThrowsException(IllegalArgumentException.class, "us-east-1000 is not a valid region"); + assertConstructorThrowsException(IllegalArgumentException.class, "StreamArn has unsupported region of 'us-east-1000'."); } @Test