revokeDestinationDefinitionFromWorkspace

Author: git-phu

airbyte-server/src/main/java/io/airbyte/server/apis/ConfigurationApi.java

@@ -576,7 +576,10 @@ public PrivateDestinationDefinitionRead grantDestinationDefinitionToWorkspace(
 
   @Override
   public void revokeDestinationDefinitionFromWorkspace(final DestinationDefinitionIdWithWorkspaceId destinationDefinitionIdWithWorkspaceId) {
-
+    execute(() -> {
+      destinationDefinitionsHandler.revokeDestinationDefinitionFromWorkspace(destinationDefinitionIdWithWorkspaceId);
+      return null;
+    });
   }
 
   // DESTINATION SPECIFICATION

airbyte-server/src/main/java/io/airbyte/server/handlers/DestinationDefinitionsHandler.java

@@ -234,4 +234,11 @@ public PrivateDestinationDefinitionRead grantDestinationDefinitionToWorkspace(
         .granted(true);
   }
 
+  public void revokeDestinationDefinitionFromWorkspace(final DestinationDefinitionIdWithWorkspaceId destinationDefinitionIdWithWorkspaceId)
+      throws IOException {
+    configRepository.deleteActorDefinitionWorkspaceGrant(
+        destinationDefinitionIdWithWorkspaceId.getDestinationDefinitionId(),
+        destinationDefinitionIdWithWorkspaceId.getWorkspaceId());
+  }
+
 }

airbyte-server/src/test/java/io/airbyte/server/handlers/DestinationDefinitionsHandlerTest.java

@@ -296,6 +296,17 @@ void testGrantDestinationDefinitionToWorkspace() throws JsonValidationException,
         workspaceId);
   }
 
+  @Test
+  @DisplayName("revokeDestinationDefinitionFromWorkspace should correctly delete a workspace grant")
+  void testRevokeDestinationDefinitionFromWorkspace() throws IOException {
+    destinationDefinitionsHandler.revokeDestinationDefinitionFromWorkspace(new DestinationDefinitionIdWithWorkspaceId()
+        .destinationDefinitionId(destinationDefinition.getDestinationDefinitionId())
+        .workspaceId(workspaceId));
+    verify(configRepository).deleteActorDefinitionWorkspaceGrant(
+        destinationDefinition.getDestinationDefinitionId(),
+        workspaceId);
+  }
+
   @Nested
   @DisplayName("listLatest")
   class listLatest {