Improve test cases

ng2-components/ng2-alfresco-upload/src/directives/file-draggable.directive.spec.ts

@@ -0,0 +1,110 @@
+/*!
+ * @license
+ * Copyright 2016 Alfresco Software, Ltd.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { describe, expect, beforeEach, it } from '@angular/core/testing';
+import { FileDraggableDirective } from '../directives/file-draggable.directive';
+
+describe('AlfrescoDirectiveFileDraggable', () => {
+
+    let component;
+
+    beforeEach( () => {
+        component = new FileDraggableDirective();
+    });
+
+    it('should emit onFolderEntityDropped event when a folder is dragged with Chrome' , (done) => {
+
+        let itemEntity = {
+            fullPath: '/folder-fake',
+            isDirectory: true,
+            isFile: false,
+            name: 'folder-fake'
+        };
+        let fakeEvent = {
+            dataTransfer: {
+                items: [{
+                    webkitGetAsEntry: () => {
+                        return itemEntity;
+                    }
+                }]
+            },
+            stopPropagation: jasmine.createSpy('stopPropagation'),
+            preventDefault: jasmine.createSpy('preventDefault')
+        };
+
+        component.onFolderEntityDropped.subscribe(files => {
+            expect(files).toEqual(itemEntity);
+            expect(component.getInputFocus()).toBe(false);
+            done();
+        });
+
+        component._onDropFiles(fakeEvent);
+    });
+
+    it('should emit onFilesDropped event when a file is dragged not with Chrome' , (done) => {
+        let file = {name: 'fake-name-1', size: 10, webkitRelativePath: 'fake-folder1/fake-name-1.json'};
+        let fakeEvent = {
+            dataTransfer: {
+                files: [file]
+            },
+            stopPropagation: jasmine.createSpy('stopPropagation'),
+            preventDefault: jasmine.createSpy('preventDefault')
+        };
+
+        component.onFilesDropped.subscribe(files => {
+            expect(files).toEqual([file]);
+            expect(component.getInputFocus()).toBe(false);
+            done();
+        });
+
+        component._onDropFiles(fakeEvent);
+    });
+
+    it('should emit onFilesDropped event when a file is dragged with Chrome', (done) => {
+        let file = {name: 'fake-name-2', size: 10, webkitRelativePath: 'fake-folder1/fake-name-2.json'};
+        let fakeEvent = {
+            dataTransfer: {
+                items: [''],
+                files: [file]
+            },
+            stopPropagation: jasmine.createSpy('stopPropagation'),
+            preventDefault: jasmine.createSpy('preventDefault')
+        };
+
+        component.onFilesDropped.subscribe(files => {
+            expect(files).toEqual([file]);
+            expect(component.getInputFocus()).toBe(false);
+            done();
+        });
+
+        component._onDropFiles(fakeEvent);
+    });
+
+    it('should take the focus when the drag enter is called', () => {
+        let mockEvent = new Event('dragstart');
+        spyOn(mockEvent, 'preventDefault');
+
+        expect(component.getInputFocus()).toBe(false);
+        component._onDragEnter(mockEvent);
+        expect(component.getInputFocus()).toBe(true);
+    });
+
+
+
+
+
+});

ng2-components/ng2-alfresco-upload/src/directives/file-draggable.directive.ts

@@ -15,7 +15,7 @@
  * limitations under the License.
  */
 
-import { Directive, ElementRef, EventEmitter, Output } from '@angular/core';
+import { Directive, EventEmitter, Output } from '@angular/core';
 
 /**
  * [file-draggable]
@@ -53,8 +53,7 @@ export class FileDraggableDirective {
 
     private _inputFocusClass: boolean = false;
 
-    constructor(public el: ElementRef) {
-        console.log('FileDraggableComponent constructor', el);
+    constructor() {
     }
 
     /**
@@ -68,12 +67,13 @@ export class FileDraggableDirective {
         let items = $event.dataTransfer.items;
         if (items) {
             for (let i = 0; i < items.length; i++) {
-                let item = items[i].webkitGetAsEntry();
-                if (item) {
-                    this._traverseFileTree(item);
+                if (typeof items[i].webkitGetAsEntry !== 'undefined') {
+                    let item = items[i].webkitGetAsEntry();
+                    if (item) {
+                        this._traverseFileTree(item);
+                    }
                 } else {
-                    let dt = $event.dataTransfer;
-                    let files = dt.files;
+                    let files = $event.dataTransfer.files;
                     this.onFilesDropped.emit(files);
                 }
             }
@@ -144,4 +144,13 @@ export class FileDraggableDirective {
         $event.stopPropagation();
         $event.preventDefault();
     }
+
+
+    /**
+     * Return the value of input focus class
+     * @returns {boolean}
+     */
+    getInputFocus () {
+        return this._inputFocusClass;
+    }
 }