Fix IdentifierUtils unit tests (#442)

Author: tvp-1

src/main/java/software/amazon/cloudformation/resource/IdentifierUtils.java

@@ -23,8 +23,8 @@
 
 public class IdentifierUtils {
 
+    public static final int GUID_LENGTH = 12;
     private static final int GENERATED_PHYSICALID_MAXLEN = 40;
-    private static final int GUID_LENGTH = 12;
     private static final int MIN_PHYSICAL_RESOURCE_ID_LENGTH = 15;
     private static final int MIN_PREFERRED_LENGTH = 17;
     private static final Splitter STACKID_SPLITTER = Splitter.on('/');

src/test/java/software/amazon/cloudformation/resource/IdentifierUtilsTest.java

@@ -18,6 +18,8 @@
 import org.junit.jupiter.api.Test;
 
 public class IdentifierUtilsTest {
+    private static final String guidPattern = "[a-zA-Z0-9]{" + IdentifierUtils.GUID_LENGTH + "}$";
+
     @Test
     public void generateResourceIdentifier_withDefaultLength() {
         String result = IdentifierUtils.generateResourceIdentifier("my-resource", "123456");
@@ -88,7 +90,7 @@ public void generateResourceIdentifier_withStackNameBothFitMaxLen() {
             "arn:aws:cloudformation:us-east-1:123456789012:stack/my-stack-name/084c0bd1-082b-11eb-afdc-0a2fadfa68a5",
             "my-resource", "123456", 255);
         assertThat(result.length()).isLessThanOrEqualTo(44);
-        assertThat(result).isEqualTo("my-stack-name-my-resource-hDoP0dahAFjd");
+        assertThat(result).matches("^my-stack-name-my-resource-" + IdentifierUtilsTest.guidPattern);
     }
 
     @Test
@@ -97,15 +99,15 @@ public void generateResourceIdentifier_withLongStackNameAndShotLogicalId() {
             "arn:aws:cloudformation:us-east-1:123456789012:stack/my-very-very-very-very-very-very-long-custom-stack-name/084c0bd1-082b-11eb-afdc-0a2fadfa68a5",
             "abc", "123456", 36);
         assertThat(result.length()).isLessThanOrEqualTo(36);
-        assertThat(result).isEqualTo("my-very-very-very-v-abc-hDoP0dahAFjd");
+        assertThat(result).matches("^my-very-very-very-v-abc-" + IdentifierUtilsTest.guidPattern);
     }
 
     @Test
     public void generateResourceIdentifier_withShortStackNameAndLongLogicalId() {
         String result = IdentifierUtils.generateResourceIdentifier("abc",
             "my-very-very-very-very-very-very-long-custom-logical-id", "123456", 36);
         assertThat(result.length()).isLessThanOrEqualTo(36);
-        assertThat(result).isEqualTo("abc-my-very-very-very-v-hDoP0dahAFjd");
+        assertThat(result).matches("^abc-my-very-very-very-v-" + IdentifierUtilsTest.guidPattern);
     }
 
     @Test
@@ -114,7 +116,7 @@ public void generateResourceIdentifier_withLongStackNameAndLongLogicalId() {
             "arn:aws:cloudformation:us-east-1:123456789012:stack/my-very-very-very-very-very-very-long-custom-stack-name/084c0bd1-082b-11eb-afdc-0a2fadfa68a5",
             "my-very-very-very-very-very-very-long-custom-logical-id", "123456", 36);
         assertThat(result.length()).isEqualTo(36);
-        assertThat(result).isEqualTo("my-very-ver-my-very-ver-hDoP0dahAFjd");
+        assertThat(result).matches("^my-very-ver-my-very-ver-" + IdentifierUtilsTest.guidPattern);
     }
 
     @Test