dotCMS/core#16591 Error when login and logout as limited user (#1028)

dotCMS · May 20, 2019 · 8e46e57 · 8e46e57
1 parent 28f93d3
commit 8e46e57
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/projects/dotcms-js/src/lib/core/dotcms-events.service.spec.ts b/projects/dotcms-js/src/lib/core/dotcms-events.service.spec.ts
@@ -33,6 +33,8 @@ class DotEventsSocketMock {
     isConnected(): boolean {
         return this.connected;
     }
+
+    destroy(): void {}
 }
 
 
@@ -125,4 +127,23 @@ describe('DotcmsEventsService', () => {
 
         expect(count).toBe(2);
     });
+
+    it('should destroy socket', () => {
+        spyOn(socket, 'destroy').and.callThrough();
+
+        dotcmsEventsService.destroy();
+
+        expect(socket.destroy).toHaveBeenCalled();
+    });
+
+    it('should destroy socket and connect', () => {
+        spyOn(socket, 'destroy').and.callThrough();
+        spyOn(socket, 'connect').and.callThrough();
+
+        dotcmsEventsService.destroy();
+        expect(socket.destroy).toHaveBeenCalled();
+
+        dotcmsEventsService.start();
+        expect(socket.connect).toHaveBeenCalledTimes(1);
+    });
 });
diff --git a/projects/dotcms-js/src/lib/core/dotcms-events.service.ts b/projects/dotcms-js/src/lib/core/dotcms-events.service.ts
@@ -20,7 +20,6 @@ export class DotcmsEventsService {
      */
     destroy(): void {
         this.dotEventsSocket.destroy();
-        this.dotEventsSocket = null;
     }
 
     /**