fix application leak tooltip (#3117)

2 files changed, 15 insertions, 8 deletions

diff --git a/server/sonar-web/src/main/js/apps/overview/components/ApplicationLeakPeriodLegend.tsx b/server/sonar-web/src/main/js/apps/overview/components/ApplicationLeakPeriodLegend.tsx
index 55d8c5082ce..6045dd6871d 100644
--- a/server/sonar-web/src/main/js/apps/overview/components/ApplicationLeakPeriodLegend.tsx
+++ b/server/sonar-web/src/main/js/apps/overview/components/ApplicationLeakPeriodLegend.tsx
@@ -50,7 +50,7 @@ export default class ApplicationLeakPeriodLegend extends React.Component<Props,
   }
 
   fetchLeaks = (visible: boolean) => {
-    if (visible && this.state.leaks) {
+    if (visible && !this.state.leaks) {
       getApplicationLeak(this.props.component).then(
         leaks => {
           if (this.mounted) {
diff --git a/server/sonar-web/src/main/js/apps/overview/components/__tests__/ApplicationLeakPeriodLegend-test.tsx b/server/sonar-web/src/main/js/apps/overview/components/__tests__/ApplicationLeakPeriodLegend-test.tsx
index ee95828e890..42b2caeb249 100644
--- a/server/sonar-web/src/main/js/apps/overview/components/__tests__/ApplicationLeakPeriodLegend-test.tsx
+++ b/server/sonar-web/src/main/js/apps/overview/components/__tests__/ApplicationLeakPeriodLegend-test.tsx
@@ -20,15 +20,22 @@
 import * as React from 'react';
 import { shallow } from 'enzyme';
 import ApplicationLeakPeriodLegend from '../ApplicationLeakPeriodLegend';
+import { waitAndUpdate } from '../../../../helpers/testUtils';
 
-it('renders', () => {
-  const wrapper = shallow(<ApplicationLeakPeriodLegend component="foo" />);
-  expect(wrapper).toMatchSnapshot();
-  wrapper.setState({
-    leaks: [
+jest.mock('../../../../api/application', () => ({
+  getApplicationLeak: jest.fn(() =>
+    Promise.resolve([
       { date: '2017-01-01T11:39:03+0100', project: 'foo', projectName: 'Foo' },
       { date: '2017-02-01T11:39:03+0100', project: 'bar', projectName: 'Bar' }
-    ]
-  });
+    ])
+  )
+}));
+
+it('renders', async () => {
+  const wrapper = shallow(<ApplicationLeakPeriodLegend component="foo" />);
+  expect(wrapper).toMatchSnapshot();
+
+  wrapper.find('Tooltip').prop<Function>('onVisibleChange')(true);
+  await waitAndUpdate(wrapper);
   expect(wrapper).toMatchSnapshot();
 });