carbon-react 106.6.1 → 106.6.4

package/lib/__spec_helper__/test-utils.js

@@ -21,6 +21,11 @@ var _themes = require("../style/themes");
 
 var _mockMatchMedia = require("./mock-match-media");
 
+/* eslint-disable jest/no-conditional-expect */
+
+/* eslint-disable jest/no-identical-title */
+
+/* eslint-disable jest/no-export */
 const isUpper = char => char.toUpperCase() === char;
 
 const humpToDash = (acc, char) => `${acc}${isUpper(char) ? `-${char.toLowerCase()}` : char}`;
@@ -51,27 +56,24 @@ const dispatchKeyPress = code => {
 };
 
 const keyMap = {
-  UpArrow: "38",
-  DownArrow: "40",
-  RightArrow: "39",
-  LeftArrow: "37",
-  Enter: "13",
-  Tab: "9",
-  Space: "32",
-  Escape: "27",
-  End: "35",
-  Home: "36",
-  D: "68",
-  E: "69",
-  P: "80",
-  Z: "90",
-  1: "49"
+  UpArrow: 38,
+  DownArrow: 40,
+  RightArrow: 39,
+  LeftArrow: 37,
+  Enter: 13,
+  Tab: 9,
+  Space: 32,
+  Escape: 27,
+  End: 35,
+  Home: 36,
+  D: 68,
+  E: 69,
+  P: 80,
+  Z: 90,
+  1: 49
 };
-
-const repeat = action => (n = 1) => makeArrayKeys(n).forEach(() => action());
-
 const keyboard = Object.keys(keyMap).reduce((acc, key) => {
-  acc[`press${key}`] = () => repeat(dispatchKeyPress(keyMap[key]));
+  acc[`press${key}`] = () => dispatchKeyPress(keyMap[key]);
 
   return acc;
 }, {}); // Build an object of Enzyme simulate helpers
@@ -88,7 +90,7 @@ const keydown = Object.keys(keyMap).reduce((acc, key) => {
     target.simulate("keydown", {
       shiftKey,
       key,
-      which: parseInt(keyMap[key], 0)
+      which: keyMap[key]
     });
   };
 
@@ -162,18 +164,19 @@ const backgroundProps = [["background", "background", "lightblue url('foo.jpg')
 
 const getDefaultValue = value => {
   const spaceArrayLength = _baseTheme.space.length - 1;
+  if (value === undefined) return value;
   const parsedValue = +value;
 
-  if (typeof value === "string" && value > spaceArrayLength) {
+  if (typeof value === "string" && parsedValue > spaceArrayLength) {
     return `${value}px`;
   }
 
   if (parsedValue <= spaceArrayLength) {
-    return _baseTheme.space[value];
+    return _baseTheme.space[parsedValue];
   }
 
   if (parsedValue > spaceArrayLength) {
-    return `${value * 8}px`;
+    return `${parsedValue * 8}px`;
   }
 
   return value;
@@ -194,18 +197,18 @@ const testStyledSystemMargin = (component, defaults, styleContainer, assertOpts)
       let marginBottom;
 
       if (defaults) {
-        margin = getDefaultValue(defaults.m || undefined);
-        marginLeft = getDefaultValue(defaults.ml || defaults.mx || undefined);
-        marginRight = getDefaultValue(defaults.mr || defaults.mx || undefined);
-        marginTop = getDefaultValue(defaults.mt || defaults.my || undefined);
-        marginBottom = getDefaultValue(defaults.mb || defaults.my || undefined);
-        expect(assertStyleMatch({
+        margin = getDefaultValue(defaults.m);
+        marginLeft = getDefaultValue(defaults.ml || defaults.mx);
+        marginRight = getDefaultValue(defaults.mr || defaults.mx);
+        marginTop = getDefaultValue(defaults.mt || defaults.my);
+        marginBottom = getDefaultValue(defaults.mb || defaults.my);
+        assertStyleMatch({
           margin,
           marginLeft,
           marginRight,
           marginTop,
           marginBottom
-        }, StyleElement, assertOpts));
+        }, StyleElement, assertOpts);
       } else {
         expect(StyleElement).not.toHaveStyleRule("marginLeft");
         expect(StyleElement).not.toHaveStyleRule("marginRight");
@@ -217,15 +220,14 @@ const testStyledSystemMargin = (component, defaults, styleContainer, assertOpts)
   });
   describe.each(marginProps)('when a custom spacing is specified using the "%s" styled system props', (styledSystemProp, propName) => {
     it(`then that ${propName} should have been set correctly`, () => {
-      let wrapper = (0, _enzyme.mount)(component());
       const props = {
         [styledSystemProp]: 2
       };
-      wrapper = (0, _enzyme.mount)(component({ ...props
+      const wrapper = (0, _enzyme.mount)(component({ ...props
       }));
-      expect(assertStyleMatch({
+      assertStyleMatch({
         [propName]: "var(--spacing200)"
-      }, styleContainer ? styleContainer(wrapper) : wrapper, assertOpts));
+      }, styleContainer ? styleContainer(wrapper) : wrapper, assertOpts);
     });
   });
 };
@@ -245,18 +247,18 @@ const testStyledSystemPadding = (component, defaults, styleContainer, assertOpts
       let paddingBottom;
 
       if (defaults) {
-        padding = getDefaultValue(defaults.p || undefined);
-        paddingLeft = getDefaultValue(defaults.pl || defaults.px || undefined);
-        paddingRight = getDefaultValue(defaults.pr || defaults.px || undefined);
-        paddingTop = getDefaultValue(defaults.pt || defaults.py || undefined);
-        paddingBottom = getDefaultValue(defaults.pb || defaults.py || undefined);
-        expect(assertStyleMatch({
+        padding = getDefaultValue(defaults.p);
+        paddingLeft = getDefaultValue(defaults.pl || defaults.px);
+        paddingRight = getDefaultValue(defaults.pr || defaults.px);
+        paddingTop = getDefaultValue(defaults.pt || defaults.py);
+        paddingBottom = getDefaultValue(defaults.pb || defaults.py);
+        assertStyleMatch({
           padding,
           paddingLeft,
           paddingRight,
           paddingTop,
           paddingBottom
-        }, StyleElement, assertOpts));
+        }, StyleElement, assertOpts);
       } else {
         expect(StyleElement).not.toHaveStyleRule("paddingLeft");
         expect(StyleElement).not.toHaveStyleRule("paddingRight");
@@ -268,15 +270,14 @@ const testStyledSystemPadding = (component, defaults, styleContainer, assertOpts
   });
   describe.each(paddingProps)('when a custom spacing is specified using the "%s" styled system props', (styledSystemProp, propName) => {
     it(`then that ${propName} should have been set correctly`, () => {
-      let wrapper = (0, _enzyme.mount)(component());
       const props = {
         [styledSystemProp]: 2
       };
-      wrapper = (0, _enzyme.mount)(component({ ...props
+      const wrapper = (0, _enzyme.mount)(component({ ...props
       }));
-      expect(assertStyleMatch({
+      assertStyleMatch({
         [propName]: "var(--spacing200)"
-      }, styleContainer ? styleContainer(wrapper) : wrapper, assertOpts));
+      }, styleContainer ? styleContainer(wrapper) : wrapper, assertOpts);
     });
   });
 };
@@ -293,14 +294,14 @@ exports.testStyledSystemSpacing = testStyledSystemSpacing;
 const testStyledSystemColor = (component, styleContainer) => {
   describe.each(colorProps)('when a prop is specified using the "%s" styled system props', (styledSystemProp, propName, value) => {
     it(`then ${propName} should have been set correctly`, () => {
-      let wrapper = (0, _enzyme.mount)(component());
       const props = {
         [styledSystemProp]: value
       };
-      wrapper = (0, _enzyme.mount)(component({ ...props
-      })); // Some props need to have camelcase so used toHaveStyleRule rather than assertStyleMatch
+      const wrapper = (0, _enzyme.mount)(component({ ...props
+      }));
+      const StyleElement = styleContainer ? styleContainer(wrapper) : wrapper; // Some props need to have camelcase so used toHaveStyleRule rather than assertStyleMatch
 
-      expect(wrapper).toHaveStyleRule(propName, value, styleContainer ? styleContainer(wrapper) : wrapper);
+      expect(StyleElement).toHaveStyleRule(propName, value);
     });
   });
 };
@@ -311,13 +312,13 @@ const testStyledSystemWidth = (component, styleContainer) => {
   describe("when a width prop is specified using styled system props", () => {
     it("then width should have been set correctly", () => {
       const [styledSystemProp, propName, value] = widthProps;
-      let wrapper = (0, _enzyme.mount)(component());
       const props = {
         [styledSystemProp]: value
       };
-      wrapper = (0, _enzyme.mount)(component({ ...props
+      const wrapper = (0, _enzyme.mount)(component({ ...props
       }));
-      expect(wrapper).toHaveStyleRule(propName, value, styleContainer ? styleContainer(wrapper) : wrapper);
+      const StyleElement = styleContainer ? styleContainer(wrapper) : wrapper;
+      expect(StyleElement).toHaveStyleRule(propName, value);
     });
   });
 };
@@ -327,14 +328,14 @@ exports.testStyledSystemWidth = testStyledSystemWidth;
 const testStyledSystemLayout = (component, styleContainer) => {
   describe.each(layoutProps)('when a prop is specified using the "%s" styled system props', (styledSystemProp, propName, value) => {
     it(`then ${propName} should have been set correctly`, () => {
-      let wrapper = (0, _enzyme.mount)(component());
       const props = {
         [styledSystemProp]: value
       };
-      wrapper = (0, _enzyme.mount)(component({ ...props
-      })); // Some props need to have camelcase so used toHaveStyleRule rather than assertStyleMatch
+      const wrapper = (0, _enzyme.mount)(component({ ...props
+      }));
+      const StyleElement = styleContainer ? styleContainer(wrapper) : wrapper; // Some props need to have camelcase so used toHaveStyleRule rather than assertStyleMatch
 
-      expect(wrapper).toHaveStyleRule(propName, value, styleContainer ? styleContainer(wrapper) : wrapper);
+      expect(StyleElement).toHaveStyleRule(propName, value);
     });
   });
 };
@@ -344,15 +345,13 @@ exports.testStyledSystemLayout = testStyledSystemLayout;
 const testStyledSystemFlexBox = (component, styleContainer) => {
   describe.each(flexBoxProps)('when a prop is specified using the "%s" styled system props', (styledSystemProp, propName, value) => {
     it(`then ${propName} should have been set correctly`, () => {
-      let wrapper = (0, _enzyme.mount)(component());
       const props = {
         [styledSystemProp]: value
       };
-      wrapper = (0, _enzyme.mount)(component({ ...props
-      }));
-      expect(assertStyleMatch({
+      const wrapper = (0, _enzyme.mount)(component(props));
+      assertStyleMatch({
         [propName]: value
-      }, styleContainer ? styleContainer(wrapper) : wrapper));
+      }, styleContainer ? styleContainer(wrapper) : wrapper);
     });
   });
 };
@@ -362,15 +361,14 @@ exports.testStyledSystemFlexBox = testStyledSystemFlexBox;
 const testStyledSystemBackground = (component, styleContainer) => {
   describe.each(backgroundProps)('when a prop is specified using the "%s" styled system props', (styledSystemProp, propName, value) => {
     it(`then ${propName} should have been set correctly`, () => {
-      let wrapper = (0, _enzyme.mount)(component());
       const props = {
         [styledSystemProp]: value
       };
-      wrapper = (0, _enzyme.mount)(component({ ...props
+      const wrapper = (0, _enzyme.mount)(component({ ...props
       }));
-      expect(assertStyleMatch({
+      assertStyleMatch({
         [styledSystemProp]: value
-      }, styleContainer ? styleContainer(wrapper) : wrapper));
+      }, styleContainer ? styleContainer(wrapper) : wrapper);
     });
   });
 }; // this util will catch that a console output occurred without polluting the output when running the unit tests

package/lib/components/action-popover/action-popover-context.d.ts

@@ -1,3 +1,8 @@
-export default ActionPopoverContext;
-declare const ActionPopoverContext: React.Context<{}>;
 import React from "react";
+declare type ActionPopoverContextType = {
+    setOpenPopover: (isOpen: boolean) => void;
+    focusButton: () => void;
+    isOpenPopover: boolean;
+};
+declare const ActionPopoverContext: React.Context<ActionPopoverContextType | null>;
+export default ActionPopoverContext;

package/lib/components/action-popover/action-popover-context.js

@@ -9,7 +9,7 @@ var _react = _interopRequireDefault(require("react"));
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-const ActionPopoverContext = /*#__PURE__*/_react.default.createContext({});
+const ActionPopoverContext = /*#__PURE__*/_react.default.createContext(null);
 
 var _default = ActionPopoverContext;
 exports.default = _default;

package/lib/components/action-popover/action-popover-divider/action-popover-divider.component.d.ts

@@ -1,2 +1,2 @@
-export default ActionPopoverDivider;
 import { MenuItemDivider as ActionPopoverDivider } from "../action-popover.style";
+export default ActionPopoverDivider;

package/lib/components/action-popover/action-popover-item/action-popover-item.component.d.ts

@@ -1,42 +1,29 @@
-export default ActionPopoverItem;
-export function MenuItem({ children, icon, disabled, onClick: onClickProp, submenu, theme, placement, focusItem, download, href, horizontalAlignment, ...rest }: {
-    [x: string]: any;
-    children: any;
-    icon: any;
-    disabled?: boolean | undefined;
-    onClick: any;
-    submenu: any;
-    theme: any;
-    placement?: string | undefined;
-    focusItem: any;
-    download: any;
-    href: any;
-    horizontalAlignment: any;
-}): JSX.Element;
-export namespace MenuItem {
-    namespace propTypes {
-        const children: PropTypes.Validator<string>;
-        const disabled: PropTypes.Requireable<boolean>;
-        const icon: PropTypes.Requireable<string>;
-        const onClick: PropTypes.Requireable<(...args: any[]) => any>;
-        const download: PropTypes.Requireable<boolean>;
-        const href: PropTypes.Requireable<string>;
-        /** Submenu component for item */
-        function submenu(props: any, propName: any, componentName: any): Error | undefined;
-        /** Submenu component for item */
-        function submenu(props: any, propName: any, componentName: any): Error | undefined;
-        const placement: PropTypes.Requireable<string>;
-        const focusItem: PropTypes.Requireable<boolean>;
-        const horizontalAlignment: PropTypes.Requireable<string>;
-    }
-    const displayName: string;
-}
-declare const ActionPopoverItem: React.ForwardRefExoticComponent<{
-    [x: string]: any;
-    [x: number]: any;
-    [x: symbol]: any;
-} & {
-    theme?: any;
-}>;
-import PropTypes from "prop-types";
 import React from "react";
+import { IconType } from "../../icon";
+export interface ActionPopoverItemProps {
+    /** The text label to display for this Item */
+    children: string;
+    /** Flag to indicate if item is disabled */
+    disabled?: boolean;
+    /** allows to provide download prop that works dependent with href */
+    download?: boolean;
+    /** allows to provide href prop */
+    href?: string;
+    /** The name of the icon to display next to the label */
+    icon?: IconType;
+    /** Callback to run when item is clicked */
+    onClick?: (ev: React.MouseEvent<HTMLButtonElement> | React.KeyboardEvent<HTMLButtonElement>) => void;
+    /** Submenu component for item */
+    submenu?: React.ReactNode;
+    /** @ignore @private */
+    placement?: "top" | "bottom";
+    /** @ignore @private */
+    focusItem?: boolean;
+    /** @ignore @private */
+    horizontalAlignment?: "left" | "right";
+}
+export declare const ActionPopoverItem: {
+    ({ children, icon, disabled, onClick: onClickProp, submenu, placement, focusItem, download, href, horizontalAlignment, ...rest }: ActionPopoverItemProps): JSX.Element;
+    displayName: string;
+};
+export default ActionPopoverItem;