Fixing the failing test cases for tagging related changes. (#80)

Author: adityatomer

aws-redshiftserverless-workgroup/src/test/java/software/amazon/redshiftserverless/workgroup/UpdateHandlerTest.java

@@ -82,26 +82,6 @@ public void handleRequest_SimpleSuccess() {
         assertThat(response.getErrorCode()).isNull();
     }
 
-    @Test
-    public void handleRequest_NoReadTagsPermissions() {
-        final UpdateHandler handler = new UpdateHandler();
-
-        final ResourceModel responseResourceModel = updateResponseResourceModel();
-
-        final ResourceHandlerRequest<ResourceModel> request = ResourceHandlerRequest.<ResourceModel>builder()
-                .previousResourceState(createRequestResourceModel())
-                .desiredResourceState(updateRequestResourceModel())
-                .build();
-
-        when(proxyClient.client().listTagsForResource(any(ListTagsForResourceRequest.class))).thenThrow(ValidationException.builder().build());
-        when(proxyClient.client().getWorkgroup(any(GetWorkgroupRequest.class))).thenReturn(getReadResponseSdk());
-
-        final ProgressEvent<ResourceModel, CallbackContext> response = handler.handleRequest(proxy, request, new CallbackContext(), proxyClient, logger);
-
-        assertThat(response).isNotNull();
-        assertThat(response.getStatus()).isEqualTo(OperationStatus.FAILED);
-    }
-
     @Test
     public void handleRequest_RestoreFromSnapshot_Success() {
         final UpdateHandler handler = new UpdateHandler();
@@ -116,6 +96,7 @@ public void handleRequest_RestoreFromSnapshot_Success() {
                 .desiredResourceState(requestModel)
                 .build();
 
+        when(proxyClient.client().listTagsForResource(any(ListTagsForResourceRequest.class))).thenReturn(ListTagsForResourceResponse.builder().build());
         when(proxyClient.client().updateWorkgroup(any(UpdateWorkgroupRequest.class))).thenReturn(updateResponseSdk());
         when(proxyClient.client().getWorkgroup(any(GetWorkgroupRequest.class))).thenReturn(getReadResponseSdk());
         when(proxyClient.client().restoreFromSnapshot(any(RestoreFromSnapshotRequest.class))).thenReturn(
@@ -145,8 +126,7 @@ public void handleRequest_RestoreFromSnapshot_Failed() {
                 .previousResourceState(createRequestResourceModel())
                 .desiredResourceState(requestModel)
                 .build();
-
-        when(proxyClient.client().listTagsForResource(any(ListTagsForResourceRequest.class))).thenReturn(ListTagsForResourceResponse.builder().build());
+        
         when(proxyClient.client().updateWorkgroup(any(UpdateWorkgroupRequest.class))).thenReturn(updateResponseSdk());
         when(proxyClient.client().getWorkgroup(any(GetWorkgroupRequest.class))).thenReturn(getReadResponseSdk());
         when(proxyClient.client().restoreFromSnapshot(any(RestoreFromSnapshotRequest.class))).thenThrow(ValidationException.builder().build());