Skip to content

Commit

Permalink
code review feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
rshen91 committed Sep 19, 2024
1 parent b0fb938 commit 460084b
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ describe('Dashboard App', () => {
render(<DashboardApp redirectTo={jest.fn()} history={mockHistory} />);

await waitFor(() => {
expect(expandPanelSpyInRouter).not.toHaveBeenCalled();
// this value should be undefined by default
expect(mockDashboard.expandedPanelId.getValue()).toBe(undefined);
// history should not be called
Expand Down Expand Up @@ -87,6 +88,7 @@ describe('Dashboard App', () => {

// simulate minimizing a panel
mockDashboard.expandedPanelId.next(undefined);

await waitFor(() => {
expect(mockDashboard.expandedPanelId.getValue()).toBe(undefined);
expect(historySpy).toHaveBeenCalledTimes(1);
Expand Down

0 comments on commit 460084b

Please sign in to comment.