@scality/core-ui 0.162.0 → 0.163.0

package/src/lib/components/selectv2/selectv2.test.tsx

@@ -1,15 +1,10 @@
-import { screen, render as testingRender } from '@testing-library/react';
+import { act, screen, render as testingRender, waitFor } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import React, { useState, useRef } from 'react';
-import { QueryClient, QueryClientProvider } from 'react-query';
 import { Option, Select, SelectRef } from '../selectv2/Selectv2.component';
 
 const render = (args) => {
-  return testingRender(
-    <QueryClientProvider client={new QueryClient()}>
-      {args}
-    </QueryClientProvider>,
-  );
+  return testingRender(args);
 };
 
 const generateOptionsData = (n: number) =>
@@ -74,27 +69,29 @@ describe('SelectV2', () => {
     expect(() => render(<Option value="Option 1" />)).toThrowError();
   });
 
-  it('should open/close on click', () => {
+  it('should open/close on click', async () => {
     render(<SelectWrapper />);
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
     const select = selectors.select();
     expect(select).toBeInTheDocument();
     let options = selectors.options();
     expect(options).toHaveLength(0);
 
     // should open on click
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     simpleOptions.forEach((opt) => {
       const option = selectors.option(opt.props.label);
       expect(option).toBeInTheDocument();
     });
 
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     options = selectors.options();
     expect(options).toHaveLength(0);
   });
 
-  it('should open/close with keyboard', () => {
+  it('should open/close with keyboard', async () => {
     render(<SelectWrapper />);
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
     const select = selectors.select();
     expect(select).toBeInTheDocument();
     const options = selectors.options();
@@ -102,7 +99,7 @@ describe('SelectV2', () => {
 
     // should open on Enter
     userEvent.tab();
-    userEvent.keyboard('{Enter}');
+    await act(() => userEvent.keyboard('{Enter}'));
     simpleOptions.forEach((opt) => {
       const option = selectors.option(opt.props.label);
       expect(option).toBeInTheDocument();
@@ -121,13 +118,14 @@ describe('SelectV2', () => {
     });
   });
 
-  it('should display custom placeholder', () => {
+  it('should display custom placeholder', async () => {
     const placeholder = 'My placeholder...';
     render(<SelectWrapper placeholder={placeholder} />);
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
     expect(screen.getByText(placeholder)).toBeInTheDocument();
   });
 
-  it('should be disabled', () => {
+  it('should be disabled', async () => {
     render(
       <SelectWrapper value="1" disabled={true}>
         {simpleOptions}
@@ -139,27 +137,29 @@ describe('SelectV2', () => {
     // use input instead of select because select will still trigger the open/close action
     // despite select container not being clickable and input being disabled
     const input = selectors.input();
-    userEvent.click(input);
+    await act(() => userEvent.click(input));
     const options = selectors.options();
     expect(options).toHaveLength(0);
   });
 
-  it('should display no option', () => {
+  it('should display no option', async () => {
     render(
       <SelectWrapper>
         <></>
       </SelectWrapper>,
     );
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
     const select = selectors.select();
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     const noOptions = selectors.noOptions();
     expect(noOptions).toBeInTheDocument();
   });
 
-  it('should filter and highlight on search', () => {
+  it('should filter and highlight on search', async () => {
     render(<SelectWrapper>{optionsWithScrollSearchBar} </SelectWrapper>);
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
     const select = selectors.select(true);
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     const input = selectors.input();
 
     userEvent.type(input, '2');
@@ -169,25 +169,26 @@ describe('SelectV2', () => {
     expect(searchedText).toHaveTextContent('2');
   });
 
-  it('should unfocus the search input when the select is closed', () => {
+  it('should unfocus the search input when the select is closed', async () => {
     render(<SelectWrapper>{optionsWithScrollSearchBar} </SelectWrapper>);
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
     const select = selectors.select(true);
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     let input = selectors.input();
     expect(input).toHaveFocus();
     const option = selectors.option(/Item 1/);
-    userEvent.click(option);
+    await act(() => userEvent.click(option));
     input = selectors.input();
     expect(input).not.toHaveFocus();
   });
 
-  it('should be possible to use searchbar when option is selected', () => {
+  it('should be possible to use searchbar when option is selected', async () => {
     render(
       <SelectWrapper value="1">{optionsWithScrollSearchBar}</SelectWrapper>,
     );
     expect(screen.getByText(/Item 1/)).toBeVisible();
     const select = selectors.select(true);
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     const input = selectors.input();
     userEvent.type(input, '2');
     expect(screen.queryByText(/Item 1/)).not.toBeInTheDocument();
@@ -195,47 +196,48 @@ describe('SelectV2', () => {
     expect(options).toHaveLength(1);
   });
 
-  it('should select/unselect option with keyboard', () => {
+  it('should select/unselect option with keyboard', async () => {
     render(<SelectWrapper />);
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
     const select = selectors.select();
     userEvent.tab();
-    userEvent.keyboard('{ArrowDown}');
+    act(() => userEvent.keyboard('{ArrowDown}'));
 
     // should select first option
-    userEvent.keyboard('{Enter}');
+    await act(() => userEvent.keyboard('{Enter}'));
     expect(select).toHaveTextContent('Item 0');
 
     // should select second option
     userEvent.tab();
-    userEvent.keyboard('{ArrowDown}');
-    userEvent.keyboard('{ArrowDown}');
+    await act(() => userEvent.keyboard('{ArrowDown}'));
+    await act(() => userEvent.keyboard('{ArrowDown}'));
 
-    userEvent.keyboard('{Enter}');
+    await act(() => userEvent.keyboard('{Enter}'));
     expect(select).toHaveTextContent('Item 1');
   });
 
-  it('should scroll to selected value when opening select', () => {
+  it('should scroll to selected value when opening select', async () => {
     render(
       <SelectWrapper value={optionsWithScrollSearchBar[9].props.value}>
         {optionsWithScrollSearchBar}
       </SelectWrapper>,
     );
     const select = selectors.select(true);
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     const option = selectors.option(/Item 9/);
     expect(screen.queryByRole('option', { name: /Item 1/i })).toBeNull();
     expect(option).toBeVisible();
   });
 
-  it('should be able to reset the value', () => {
+  it('should be able to reset the value', async () => {
     render(<SelectReset>{simpleOptions}</SelectReset>);
     const button = screen.getByText(/reset/);
-    userEvent.click(button);
+    await act(() => userEvent.click(button));
     const select = selectors.select();
     expect(select).toHaveTextContent('Select...');
   });
 
-  it('should not be possible to select an option if it is disabled', () => {
+  it('should not be possible to select an option if it is disabled', async () => {
     render(
       <SelectWrapper>
         <Option value="1" disabled>
@@ -245,15 +247,15 @@ describe('SelectV2', () => {
       </SelectWrapper>,
     );
     const select = selectors.select();
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     const option = selectors.option(/Item 1/);
 
-    userEvent.click(option);
+    await act(() => userEvent.click(option));
     const option2 = selectors.option(/Item 2/);
     expect(option2).toBeVisible();
   });
 
-  it('should display a tooltip if the option is disabled with a reason', () => {
+  it('should display a tooltip if the option is disabled with a reason', async () => {
     render(
       <SelectWrapper>
         <Option value="1" disabled disabledReason="This option is disabled">
@@ -262,10 +264,10 @@ describe('SelectV2', () => {
       </SelectWrapper>,
     );
     const select = selectors.select();
-    userEvent.click(select);
+    await act(() => userEvent.click(select));
     const option = selectors.option(/Item 1/);
     expect(option).toHaveAttribute('aria-disabled', 'true');
-    userEvent.hover(option);
+    await act(() => userEvent.hover(option));
     const tooltip = screen.getByText(/This option is disabled/);
     expect(tooltip).toBeInTheDocument();
   });
@@ -309,9 +311,9 @@ describe('SelectV2', () => {
     // It's not our case here, so it makes thing difficult to select the right select
     // I workaround this by using setting the aria-label to the select container (cf: test below)
     const singleSelect = screen.getByRole('listbox');
-    await userEvent.click(singleSelect);
+    await act(() => userEvent.click(singleSelect));
 
-    await userEvent.click(screen.getByRole('option', { name: /account 1/i }));
+    await act(() => userEvent.click(screen.getByRole('option', { name: /account 1/i })));
   });
 
   it('should be testable if we have several select', async () => {
@@ -372,13 +374,13 @@ describe('SelectV2', () => {
 
     render(<MyWrapperWith2Select />);
 
-    await userEvent.click(screen.getByLabelText(/select account/i));
+    await act(() => userEvent.click(screen.getByLabelText(/select account/i)));
 
-    await userEvent.click(screen.getByRole('option', { name: /account 1/i }));
+    await act(() => userEvent.click(screen.getByRole('option', { name: /account 1/i })));
 
-    await userEvent.click(screen.getByLabelText(/select user/i));
+    await act(() => userEvent.click(screen.getByLabelText(/select user/i)));
 
-    await userEvent.click(screen.getByRole('option', { name: /user 1/i }));
+    await act(() => userEvent.click(screen.getByRole('option', { name: /user 1/i })));
   });
 
   it('should be testable even if we have several select with the same value, the placeholder should be different', async () => {
@@ -439,8 +441,8 @@ describe('SelectV2', () => {
 
     render(<MyWrapperWith2Select />);
 
-    await userEvent.click(screen.getByLabelText(/select account/i));
-    await userEvent.click(screen.getByLabelText(/Select Second Account/i));
+    await act(() => userEvent.click(screen.getByLabelText(/select account/i)));
+    await act(() => userEvent.click(screen.getByLabelText(/Select Second Account/i)));
 
     /**
      * This is possible because only 1 select can be open at a time
@@ -450,7 +452,7 @@ describe('SelectV2', () => {
      * const selectContainer = select?.parentElement?.parentElement;
      * const option = within(selectContainer).getByRole('option', { name: /account 1/i });
      */
-    await userEvent.click(screen.getByRole('option', { name: /account 1/i }));
+    await act(() => userEvent.click(screen.getByRole('option', { name: /account 1/i })));
   });
 
   describe('Ref API', () => {
@@ -478,7 +480,7 @@ describe('SelectV2', () => {
       render(<RefTestComponent />);
       expect(selectors.input()).not.toHaveFocus();
 
-      userEvent.click(screen.getByRole('button', { name: /Focus/i }));
+      await act(() => userEvent.click(screen.getByRole('button', { name: /Focus/i })));
       expect(selectors.input()).toHaveFocus();
     });
 
@@ -511,14 +513,14 @@ describe('SelectV2', () => {
       render(<RefTestComponent />);
       expect(selectors.options()).toHaveLength(0);
 
-      userEvent.click(screen.getByRole('button', { name: /Open Menu/i }));
+      await act(() => userEvent.click(screen.getByRole('button', { name: /Open Menu/i })));
       expect(selectors.options().length).toBeGreaterThan(0);
       simpleOptions.forEach((opt) => {
         const option = selectors.option(opt.props.label);
         expect(option).toBeInTheDocument();
       });
 
-      userEvent.click(screen.getByRole('button', { name: /Close Menu/i }));
+      await act(() => userEvent.click(screen.getByRole('button', { name: /Close Menu/i })));
       expect(selectors.options()).toHaveLength(0);
     });
 
@@ -565,13 +567,16 @@ describe('SelectV2', () => {
 
       render(<RefTestComponent />);
 
-      const select = selectors.select();
+      let select;
+      await act(() => {
+        select = selectors.select();
+      });
       expect(select).toHaveTextContent('Select with ref');
 
-      userEvent.click(screen.getByRole('button', { name: /Set Value/i }));
+      await act(() => userEvent.click(screen.getByRole('button', { name: /Set Value/i })));
       expect(select).toHaveTextContent('Item 0');
 
-      userEvent.click(screen.getByRole('button', { name: /Clear/i }));
+      await act(() => userEvent.click(screen.getByRole('button', { name: /Clear/i })));
       expect(select).toHaveTextContent('Select with ref');
     });
 
@@ -599,10 +604,10 @@ describe('SelectV2', () => {
 
       render(<RefTestComponent />);
 
-      userEvent.click(screen.getByRole('button', { name: /Focus/i }));
+      await act(() => userEvent.click(screen.getByRole('button', { name: /Focus/i })));
       expect(selectors.input()).toHaveFocus();
 
-      userEvent.click(screen.getByRole('button', { name: /Blur/i }));
+      await act(() => userEvent.click(screen.getByRole('button', { name: /Blur/i })));
       expect(selectors.input()).not.toHaveFocus();
     });
   });

package/src/lib/components/steppers/Stepper.component.tsx

@@ -1,5 +1,5 @@
 /// <reference path="./Stepper.component.d.ts" />
-import { createContext, useContext, useState } from 'react';
+import { createContext, useContext, useState, useCallback, useMemo } from 'react';
 import { Steppers } from './Steppers.component';
 import { Box } from '../box/Box';
 export interface StepperContextType {
@@ -33,19 +33,21 @@ export const Stepper: Stepper = ({ steps }) => {
     props: Record<string, unknown>;
   }>({ step: 0, props: {} });
 
-  const next = (props: Record<string, unknown>) => {
-    setStepProps({ step: stepProps.step + 1, props });
-  };
+  const next = useCallback((props: Record<string, unknown>) => {
+    setStepProps(current => ({ step: current.step + 1, props }));
+  }, []);
 
-  const prev = (props: Record<string, unknown>) => {
-    setStepProps({ step: stepProps.step - 1, props });
-  };
+  const prev = useCallback((props: Record<string, unknown>) => {
+    setStepProps(current => ({ step: current.step - 1, props }));
+  }, []);
 
   const { Component } = steps[stepProps.step];
   const StepperContext = window.StepperContext;
 
+  const stepperValue = useMemo(() => ({ next, prev }), [next, prev]);
+
   return (
-    <StepperContext.Provider value={{ next, prev }}>
+    <StepperContext.Provider value={stepperValue}>
       <Box display="flex" gap={32} flex={1} height="100%">
         <Steppers
           activeStep={stepProps.step}

package/src/lib/components/tablev2/TableSync.test.tsx

@@ -1,31 +1,28 @@
-import { fireEvent, render, screen } from '@testing-library/react';
+import { act, fireEvent, render, screen, waitFor } from '@testing-library/react';
 import React from 'react';
 import { TableSync } from './TableSync';
-import { QueryClient, QueryClientProvider } from 'react-query';
 
 describe('TableSync', () => {
-  it('should render correctly', () => {
+  it('should render correctly', async () => {
     const onSync = jest.fn();
     render(
-      <QueryClientProvider client={new QueryClient()}>
-        <TableSync onSync={onSync} />
-      </QueryClientProvider>,
+      <TableSync onSync={onSync} />
     );
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
 
     const button = screen.getByRole('button');
     expect(button).toBeInTheDocument();
   });
 
-  it('should call onSync when clicked', () => {
+  it('should call onSync when clicked', async () => {
     const onSync = jest.fn();
     render(
-      <QueryClientProvider client={new QueryClient()}>
-        <TableSync onSync={onSync} />
-      </QueryClientProvider>,
+      <TableSync onSync={onSync} />
     );
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
 
     const button = screen.getByRole('button');
-    fireEvent.click(button);
+    await act(() => fireEvent.click(button));
     expect(onSync).toHaveBeenCalledTimes(1);
   });
 });

package/src/lib/components/tablev2/Tablev2.test.tsx

@@ -1,7 +1,6 @@
 import { Table } from './Tablev2.component';
 import React from 'react';
-import { render } from '@testing-library/react';
-import { QueryClient, QueryClientProvider } from 'react-query';
+import { render, screen, waitFor } from '@testing-library/react';
 
 jest.mock('./TableUtils', () => ({
   ...jest.requireActual('./TableUtils'),
@@ -66,17 +65,17 @@ const columns = [
 describe('TableV2', () => {
   test('it should display all the data', async () => {
     const { getAllByRole } = render(
-      <QueryClientProvider client={new QueryClient()}>
-        <div>
-          <Table columns={columns} data={data} defaultSortingKey={'health'}>
-            <Table.SingleSelectableContent
-              rowHeight="h40"
-              separationLineVariant="backgroundLevel3"
-            />
-          </Table>
-        </div>
-      </QueryClientProvider>,
+      <div>
+        <Table columns={columns} data={data} defaultSortingKey={'health'}>
+          <Table.SingleSelectableContent
+            rowHeight="h40"
+            separationLineVariant="backgroundLevel3"
+          />
+        </Table>
+      </div>
     );
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
+
     // we check that the table is displaying all the data
     const rows = getAllByRole('row');
     expect(rows[4]).toHaveTextContent(/Ninette/i);
@@ -85,17 +84,17 @@ describe('TableV2', () => {
   });
   test('it should sort by defaultSortingKey', async () => {
     const { getAllByRole } = render(
-      <QueryClientProvider client={new QueryClient()}>
-        <div>
-          <Table columns={columns} data={data} defaultSortingKey={'firstName'}>
-            <Table.SingleSelectableContent
-              rowHeight="h40"
-              separationLineVariant="backgroundLevel3"
-            />
-          </Table>
-        </div>
-      </QueryClientProvider>,
+      <div>
+        <Table columns={columns} data={data} defaultSortingKey={'firstName'}>
+          <Table.SingleSelectableContent
+            rowHeight="h40"
+            separationLineVariant="backgroundLevel3"
+          />
+        </Table>
+      </div>
     );
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
+
     // we check that the table is displaying all the data
     const rows = getAllByRole('row');
     expect(rows[1]).toHaveTextContent(/ninette/i);
@@ -104,22 +103,22 @@ describe('TableV2', () => {
   });
   test('it should filterGlobally', async () => {
     const { getAllByRole } = render(
-      <QueryClientProvider client={new QueryClient()}>
-        <div>
-          <Table
-            columns={columns}
-            data={data}
-            defaultSortingKey={'firstName'}
-            globalFilter="an"
-          >
-            <Table.SingleSelectableContent
-              rowHeight="h40"
-              separationLineVariant="backgroundLevel3"
-            />
-          </Table>
-        </div>
-      </QueryClientProvider>,
+      <div>
+        <Table
+          columns={columns}
+          data={data}
+          defaultSortingKey={'firstName'}
+          globalFilter="an"
+        >
+          <Table.SingleSelectableContent
+            rowHeight="h40"
+            separationLineVariant="backgroundLevel3"
+          />
+        </Table>
+      </div>
     );
+    await waitFor(() => screen.queryAllByRole('img', { hidden: true }));
+
     // we check that the table is displaying all the data
     const rows = getAllByRole('row');
     expect(rows[1]).toHaveTextContent(/an/i); //first name yoh-an-n

package/src/lib/components/toast/ToastProvider.tsx

@@ -1,4 +1,4 @@
-import { ReactNode, createContext, useContext, useState } from 'react';
+import { ReactNode, createContext, useCallback, useContext, useMemo, useState } from 'react';
 import { Toast, ToastProps } from './Toast.component';
 
 export type ToastContextState = Omit<ToastProps, 'onClose'>;
@@ -14,20 +14,27 @@ export const ToastContext = createContext<ToastContextType | undefined>(
 interface ToastProviderProps {
   children: ReactNode;
 }
+
 export const ToastProvider: React.FC<
   React.PropsWithChildren<ToastProviderProps>
 > = ({ children }) => {
   const [toastProps, setToastProps] = useState<ToastContextState | null>(null);
 
-  const showToast = (toastProps: ToastContextState) => {
-    setToastProps(toastProps);
-  };
+  const toastCtxValue = useMemo(
+    () => ({ showToast: setToastProps }),
+    [],
+  );
+
+  const closeToast = useCallback(
+    () => setToastProps(null),
+    []
+  );
 
   return (
-    <ToastContext.Provider value={{ showToast }}>
+    <ToastContext.Provider value={toastCtxValue}>
       {children}
       {toastProps && (
-        <Toast {...toastProps} onClose={() => setToastProps(null)} />
+        <Toast {...toastProps} onClose={closeToast} />
       )}
     </ToastContext.Provider>
   );
@@ -38,5 +45,6 @@ export const useToast = () => {
   if (!context) {
     throw new Error('useToast must be used within a ToastProvider');
   }
+
   return context;
 };

package/src/lib/components/vegachartv2/SyncedCursorCharts.tsx

@@ -1,4 +1,4 @@
-import { useState, createContext, useContext } from 'react';
+import { useState, createContext, useContext, useMemo, useCallback } from 'react';
 export const SyncedCursorChartsContext = createContext<{
   cursorX: number;
   setCursorX: (cursorX: number) => void;
@@ -17,13 +17,11 @@ export const useCursorX = (): {
 };
 export function SyncedCursorCharts({ children }: { children: JSX.Element }) {
   const [cursorX, setCursorX] = useState(0);
+
+  const contextValue = useMemo(() => ({cursorX, setCursorX}), [cursorX]);
+
   return (
-    <SyncedCursorChartsContext.Provider
-      value={{
-        cursorX,
-        setCursorX,
-      }}
-    >
+    <SyncedCursorChartsContext.Provider value={contextValue}>
       {children}
     </SyncedCursorChartsContext.Provider>
   );

package/src/lib/index.ts

@@ -82,3 +82,4 @@ export { InfoMessage } from './components/infomessage/InfoMessage.component';
 export { InputList } from './components/inputlist/InputList.component';
 export { InlineInput } from './components/inlineinput/InlineInput';
 export { UnsuccessfulResult } from './components/UnsuccessfulResult.component';
+export { LineTimeSerieChart } from './components/linetimeseriechart/linetimeseriechart.component';

package/src/lib/next.ts

@@ -21,3 +21,4 @@ export {
   BarchartSortFn,
   BarchartTooltipFn,
 } from './components/barchartv2/Barchart.component';
+export { ChartLegendWrapper } from './components/chartlegend/ChartLegendWrapper';

package/src/lib/style/theme.ts

@@ -1,3 +1,4 @@
+import { lighten, darken } from 'polished';
 //== Colors
 export const hotPink = '#E40046';
 export const pink = '#EB4962';
@@ -291,3 +292,31 @@ export const navbarItemWidth = '4.286rem';
 //sidebar
 export const sidebarItemHeight = spacing.sp40;
 export const sidebarWidth = spacing.sp40;
+
+// We use 8 main color from the palette and decline them (lighter/ darker) when we have more than 8 datasets
+export const lineTimeSeriesColorRange = [
+  lineColor1,
+  lineColor2,
+  lineColor3,
+  lineColor4,
+  lineColor5,
+  lineColor6,
+  lineColor7,
+  lineColor8,
+  lighten(0.3, lineColor1),
+  lighten(0.3, lineColor2),
+  lighten(0.3, lineColor3),
+  lighten(0.3, lineColor4),
+  lighten(0.3, lineColor5),
+  lighten(0.3, lineColor6),
+  lighten(0.3, lineColor7),
+  lighten(0.3, lineColor8),
+  darken(0.2, lineColor1),
+  darken(0.2, lineColor2),
+  darken(0.2, lineColor3),
+  darken(0.2, lineColor4),
+  darken(0.3, lineColor5),
+  darken(0.3, lineColor6),
+  darken(0.3, lineColor7),
+  darken(0.3, lineColor8),
+];