Merge branch...

Merge branch '9036-broken-master-karma-failure-should-show-hide-popover-on-mouseenter-and-mouseleave' into 'master'

Resolve "[Broken master] Karma failure -  Should Show+Hide Popover on mouseenter and mouseleave"

Closes #9036

See merge request gitlab-org/gitlab-ee!9021

spec/javascripts/user_popovers_spec.js

@@ -30,7 +30,9 @@ describe('User Popovers', () => {
   });
 
   it('Should Show+Hide Popover on mouseenter and mouseleave', done => {
-    triggerEvent('mouseenter', document.querySelector(selector));
+    const targetLink = document.querySelector(selector);
+    const { userId } = targetLink.dataset;
+    triggerEvent('mouseenter', targetLink);
 
     setTimeout(() => {
       const shownPopover = document.querySelector('.popover');
@@ -38,9 +40,9 @@ describe('User Popovers', () => {
       expect(shownPopover).not.toBeNull();
 
       expect(shownPopover.innerHTML).toContain(dummyUser.name);
-      expect(UsersCache.retrieveById).toHaveBeenCalledWith('58');
+      expect(UsersCache.retrieveById).toHaveBeenCalledWith(userId.toString());
 
-      triggerEvent('mouseleave', document.querySelector(selector));
+      triggerEvent('mouseleave', targetLink);
 
       setTimeout(() => {
         // After Mouse leave it should be hidden now
@@ -51,13 +53,15 @@ describe('User Popovers', () => {
   });
 
   it('Should Not show a popover on short mouse over', done => {
-    triggerEvent('mouseenter', document.querySelector(selector));
+    const targetLink = document.querySelector(selector);
+    const { userId } = targetLink.dataset;
+    triggerEvent('mouseenter', targetLink);
 
     setTimeout(() => {
       expect(document.querySelector('.popover')).toBeNull();
-      expect(UsersCache.retrieveById).not.toHaveBeenCalledWith('1');
+      expect(UsersCache.retrieveById).not.toHaveBeenCalledWith(userId.toString());
 
-      triggerEvent('mouseleave', document.querySelector(selector));
+      triggerEvent('mouseleave', targetLink);
 
       done();
     });