Allow group id for standard queues sqs sink (#6527)

Signed-off-by: Taylor Gray <tylgry@amazon.com>

Author: graytaylor0

data-prepper-plugins/sqs-sink/src/main/java/org/opensearch/dataprepper/plugins/sink/sqs/SqsSinkConfig.java

@@ -46,7 +46,7 @@ public class SqsSinkConfig {
     @JsonProperty("dlq")
     private PluginModel dlq;
 
-    @AssertTrue(message = "FIFO queues wth dynamic group id or dynamic deduplication id and more than one events per message is not valid OR standard queues do not support groupId or deduplication configuration")
+    @AssertTrue(message = "FIFO queues wth dynamic group id or dynamic deduplication id and more than one events per message is not valid OR standard queues do not support deduplication configuration")
     boolean isValidConfig() {
         String deDupId = getDeDuplicationId();
         String groupId = getGroupId();
@@ -64,7 +64,7 @@ boolean isValidConfig() {
                 return true;
             }
         } else {
-             return (groupId == null && deDupId == null);
+             return (deDupId == null);
         }
     }
 

data-prepper-plugins/sqs-sink/src/test/java/org/opensearch/dataprepper/plugins/sink/sqs/SqsSinkConfigTest.java

@@ -45,7 +45,7 @@ void TestDefaultConfig() {
     }
 
     @Test
-    private void TestCustomConfig() throws Exception {
+    void TestCustomConfig() throws Exception {
         AwsConfig awsConfig = mock(AwsConfig.class);
         reflectivelySetField(sqsSinkConfig, "awsConfig", awsConfig);
         assertThat(sqsSinkConfig.getAwsConfig(), equalTo(awsConfig));
@@ -83,7 +83,7 @@ void TestValidConfigs() throws Exception {
         assertTrue(sqsSinkConfig.isValidConfig());
         String testGroupId = RandomStringUtils.randomAlphabetic(10);
         reflectivelySetField(sqsSinkConfig, "groupId", testGroupId);
-        assertFalse(sqsSinkConfig.isValidConfig());
+        assertTrue(sqsSinkConfig.isValidConfig());
         reflectivelySetField(sqsSinkConfig, "groupId", null);
         String testDeDupId = RandomStringUtils.randomAlphabetic(10);
         reflectivelySetField(sqsSinkConfig, "deDuplicationId", testDeDupId);