Update refresh for dashboard scene

[vitPinchuk]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.67%. Comparing base (9f6b696) to head (8c37a63).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #346      +/-   ##
==========================================
+ Coverage   98.55%   98.67%   +0.12%     
==========================================
  Files          58       58              
  Lines         830      832       +2     
  Branches      150      138      -12     
==========================================
+ Hits          818      821       +3     
+ Misses         12       11       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[asimonok]

In tests we should check if useDashboardRefresh is called instead of checking implementation of the hook. Please update tests

[mikhail-vl]

@vitPinchuk Please fix tests after upgrading to Grafana 11.3.0:

    console.error
      Warning: Cannot update a component (`Unknown`) while rendering a different component (`mockConstructor`). To locate the bad setState() call inside `mockConstructor`, follow the stack trace as described in https://reactjs.org/link/setstate-in-render
          at mockConstructor
          at value (/Users/mikhailvolkov/GitHub/business-charts/node_modules/@volkovlabs/components/src/components/AutosizeCodeEditor/AutosizeCodeEditor.tsx:48:3)
          at div
          at value (/Users/mikhailvolkov/GitHub/business-charts/src/components/EchartsEditor/EchartsEditor.tsx:26:50)

       98 |
       99 |     jest.mocked(CodeEditor).mockImplementationOnce(({ onEditorDidMount }: any) => {
    > 100 |       onEditorDidMount(editor);
          |       ^
      101 |       return null;
      102 |     });
      103 |

Console

    console.error
      Warning: An update to EchartsPanel inside a test was not wrapped in act(...).
      
      When testing, code that causes React state updates should be wrapped into act(...):
      
      act(() => {
        /* fire events that update state */
      });
      /* assert on the output */
      
      This ensures that you're testing the behavior the user would see in the browser. Learn more at https://reactjs.org/link/wrap-tests-with-act
          at options (/Users/mikhailvolkov/GitHub/business-charts/src/components/EchartsPanel/EchartsPanel.tsx:31:49)

      293 |         );
      294 |       } catch (err) {
    > 295 |         setError(err instanceof Error ? err : new Error(`${err}`));
          |         ^
      296 |       }
      297 |     };
      298 |

[asimonok]

Please fix act warnings

[vitPinchuk]

ter upgrading to Grafana 11.3.0:

done

[mikhail-vl]

Looks good!

[mikhail-vl]

@asimonok Could you please check and approve?

[asimonok]

Looks good!