Column/Table Drag & Drop (#10702)

* able to drag objects from left pane

* changed scheme from file

* base functionality working

* extended functionality to tables

* added string literals and formatting

* cleanup

* cleanup

* added table/column parsing

* removed bad logic

* updated to use metadata

* cleanup and added sql carbon edit tags

* moved changes from vs into sql code base

* refactoring drag and drop

* cleanup

* cleanup

* cleanup

* added unit tests

* pr changes

* moved treeMock file

* fixed small bug

src/sql/workbench/services/objectExplorer/browser/dragAndDropController.ts

@@ -10,7 +10,8 @@ import { ITree, IDragAndDrop, IDragOverReaction, DRAG_OVER_ACCEPT_BUBBLE_DOWN, D
 import { DragMouseEvent } from 'vs/base/browser/mouseEvent';
 import { TreeUpdateUtils } from 'sql/workbench/services/objectExplorer/browser/treeUpdateUtils';
 import { UNSAVED_GROUP_ID } from 'sql/platform/connection/common/constants';
-import { IDragAndDropData } from 'vs/base/browser/dnd';
+import { DataTransfers, IDragAndDropData } from 'vs/base/browser/dnd';
+import { TreeNode } from 'sql/workbench/services/objectExplorer/common/treeNode';
 
 /**
  * Implements drag and drop for the server tree
@@ -27,24 +28,40 @@ export class ServerTreeDragAndDrop implements IDragAndDrop {
 	 * Returns null, otherwise.
 	 */
 	public getDragURI(tree: ITree, element: any): string {
-		if (element instanceof ConnectionProfile) {
-			return (<ConnectionProfile>element).id;
+		if (element) {
+			if (element instanceof ConnectionProfile) {
+				return (<ConnectionProfile>element).id;
+			} else if (element instanceof ConnectionProfileGroup) {
+				return (<ConnectionProfileGroup>element).id;
+			} else if (element.nodeTypeId === 'Table' || element.nodeTypeId === 'Column') {
+				return (<TreeNode>element).id;
+			}
+			else {
+				return undefined;
+			}
 		}
-		else if (element instanceof ConnectionProfileGroup) {
-			return (<ConnectionProfileGroup>element).id;
+		else {
+			return undefined;
 		}
-		return null;
 	}
 
 	/**
 	 * Returns a label(name) to display when dragging the element.
 	 */
 	public getDragLabel(tree: ITree, elements: any[]): string {
-		if (elements[0] instanceof ConnectionProfile) {
-			return (<ConnectionProfile>elements[0]).serverName;
-		} else if (elements[0] instanceof ConnectionProfileGroup) {
-			return (<ConnectionProfileGroup>elements[0]).name;
-		} else {
+		if (elements) {
+			if (elements[0] instanceof ConnectionProfile) {
+				return (<ConnectionProfile>elements[0]).serverName;
+			} else if (elements[0] instanceof ConnectionProfileGroup) {
+				return (<ConnectionProfileGroup>elements[0]).name;
+			} else if (elements[0].label) {
+				return elements[0].label;
+			}
+			else {
+				return undefined;
+			}
+		}
+		else {
 			return undefined;
 		}
 	}
@@ -52,8 +69,15 @@ export class ServerTreeDragAndDrop implements IDragAndDrop {
 	/**
 	 * Called when the drag operation starts.
 	 */
-	public onDragStart(tree: ITree, data: IDragAndDropData, originalEvent: DragMouseEvent): void {
+	public onDragStart(tree: ITree, dragAndDropData: IDragAndDropData, originalEvent: DragMouseEvent): void {
 		TreeUpdateUtils.isInDragAndDrop = true;
+		const data = dragAndDropData.getData();
+		const element = data[0];
+		if (element.nodeTypeId === 'Column' || element.nodeTypeId === 'Table') {
+			const schema = element.metadata.schema;
+			const name = element.metadata.name;
+			originalEvent.dataTransfer.setData(DataTransfers.RESOURCES, JSON.stringify([`${element.nodeTypeId}:${element.id}?${schema ? schema + '.' + name : name}`]));
+		}
 		return;
 	}
 
@@ -78,9 +102,8 @@ export class ServerTreeDragAndDrop implements IDragAndDrop {
 				// to avoid creating a circular structure.
 				canDragOver = source.id !== targetElement.id && !source.isAncestorOf(targetElement);
 			}
-
 		} else {
-			canDragOver = false;
+			canDragOver = true;
 		}
 
 		if (canDragOver) {

src/sql/workbench/services/objectExplorer/test/browser/dragAndDropController.test.ts

@@ -0,0 +1,111 @@
+/*---------------------------------------------------------------------------------------------
+ *  Copyright (c) Microsoft Corporation. All rights reserved.
+ *  Licensed under the Source EULA. See License.txt in the project root for license information.
+ *--------------------------------------------------------------------------------------------*/
+
+import { ConnectionProfile } from 'sql/platform/connection/common/connectionProfile';
+import { ConnectionProfileGroup } from 'sql/platform/connection/common/connectionProfileGroup';
+import { ConnectionManagementService } from 'sql/workbench/services/connection/browser/connectionManagementService';
+import { TestCapabilitiesService } from 'sql/platform/capabilities/test/common/testCapabilitiesService';
+import { IStorageService } from 'vs/platform/storage/common/storage';
+import { TestStorageService } from 'vs/workbench/test/common/workbenchTestServices';
+import { TestInstantiationService } from 'vs/platform/instantiation/test/common/instantiationServiceMock';
+import { ServerTreeDragAndDrop } from 'sql/workbench/services/objectExplorer/browser/dragAndDropController';
+import { TestTree } from 'sql/workbench/test/treeMock';
+import { ConnectionProviderProperties } from 'sql/platform/capabilities/common/capabilitiesService';
+import { IConnectionProfile } from 'sql/platform/connection/common/interfaces';
+import { mssqlProviderName } from 'sql/platform/connection/common/constants';
+import { TreeNode } from 'sql/workbench/services/objectExplorer/common/treeNode';
+
+
+
+import * as TypeMoq from 'typemoq';
+import * as assert from 'assert';
+
+
+
+suite('SQL Drag And Drop Controller tests', () => {
+	const testTree = new TestTree();
+	let serverTreeDragAndDrop: ServerTreeDragAndDrop;
+	let msSQLCapabilities: ConnectionProviderProperties;
+	let capabilitiesService: TestCapabilitiesService;
+
+	let iConnectionProfileId: IConnectionProfile = {
+		connectionName: 'new name',
+		serverName: 'new server',
+		databaseName: 'database',
+		userName: 'user',
+		password: 'password',
+		authenticationType: '',
+		savePassword: true,
+		groupFullName: 'g2/g2-2',
+		groupId: 'group id',
+		getOptionsKey: undefined!,
+		matches: undefined!,
+		providerName: mssqlProviderName,
+		options: {},
+		saveProfile: true,
+		id: 'd936bb32-422b-49c3-963f-ae9532d63dc5'
+	};
+
+	let connectionProfileId = new ConnectionProfile(capabilitiesService, iConnectionProfileId);
+	let connectionProfileArray = [connectionProfileId];
+	let connectionProfileGroupId = new ConnectionProfileGroup('name', undefined, 'd936bb32-422b-49c3-963f-ae9532d63dc5', 'color', 'description');
+	let connectionProfileGroupArray = [connectionProfileGroupId];
+	let treeNode = new TreeNode('Column', 'label', undefined, undefined, undefined, undefined, undefined, undefined, undefined, undefined);
+	let treeNodeArray = [treeNode];
+
+	setup(() => {
+		let instantiationService = new TestInstantiationService();
+		instantiationService.stub(IStorageService, new TestStorageService());
+		let mockConnectionManagementService = TypeMoq.Mock.ofType(ConnectionManagementService, TypeMoq.MockBehavior.Strict,
+			undefined, //connection store
+			undefined, // connectionstatusmanager
+			undefined, // connectiondialog service
+			instantiationService, // instantiation service
+			undefined, // editor service
+			undefined, // telemetryservice
+			undefined, // configuration service
+			new TestCapabilitiesService(), // capabilities service
+		);
+		serverTreeDragAndDrop = new ServerTreeDragAndDrop(mockConnectionManagementService.object);
+
+		capabilitiesService = new TestCapabilitiesService();
+		capabilitiesService.capabilities[mssqlProviderName] = { connection: msSQLCapabilities };
+	});
+
+
+	test('create new serverTreeDragAndDrop object should create serverTreeDragAndDrop object successfully', async () => {
+
+		assert.equal(serverTreeDragAndDrop !== null || serverTreeDragAndDrop !== undefined, true);
+	});
+
+	test('able to get DragURI', async () => {
+		let uri = serverTreeDragAndDrop.getDragURI(testTree, connectionProfileId);
+		assert.equal(connectionProfileId.id, uri);
+
+		let uriGroup = serverTreeDragAndDrop.getDragURI(testTree, connectionProfileGroupId);
+		assert.equal(connectionProfileGroupId.id, uriGroup);
+
+		let uriUndefined = serverTreeDragAndDrop.getDragURI(testTree, null);
+		assert.equal(null, uriUndefined);
+
+	});
+
+	test('able to get DragLabel', async () => {
+		let label = serverTreeDragAndDrop.getDragLabel(testTree, connectionProfileArray);
+		assert.equal(connectionProfileArray[0].serverName, label);
+
+		let labelGroup = serverTreeDragAndDrop.getDragLabel(testTree, connectionProfileGroupArray);
+		assert.equal(connectionProfileGroupArray[0].name, labelGroup);
+
+		let labelTreeNode = serverTreeDragAndDrop.getDragLabel(testTree, treeNodeArray);
+		assert.equal(treeNodeArray[0].label, labelTreeNode);
+
+		let labelUndefined = serverTreeDragAndDrop.getDragLabel(testTree, null);
+		assert.equal(undefined, labelUndefined);
+
+	});
+
+
+});