Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Emotion] Fix/regression test last batch of misc components with childProps #6255

Merged
merged 5 commits into from
Sep 21, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
exports[`renders EuiColorStopThumb 1`] = `
<div
class="euiPopover euiColorStopPopover testClass1 testClass2 emotion-euiPopover"
data-test-subj="test subject string"
style="left:0%"
>
<div
Expand All @@ -29,6 +30,7 @@ exports[`renders EuiColorStopThumb 1`] = `
exports[`renders disabled EuiColorStopThumb 1`] = `
<div
class="euiPopover euiColorStopPopover testClass1 testClass2 emotion-euiPopover"
data-test-subj="test subject string"
style="left:0%"
>
<div
Expand Down Expand Up @@ -56,6 +58,7 @@ exports[`renders disabled EuiColorStopThumb 1`] = `
exports[`renders picker-only EuiColorStopThumb 1`] = `
<div
class="euiPopover euiColorStopPopover testClass1 testClass2 emotion-euiPopover"
data-test-subj="test subject string"
style="left:0%"
>
<div
Expand All @@ -82,6 +85,7 @@ exports[`renders picker-only EuiColorStopThumb 1`] = `
exports[`renders readOnly EuiColorStopThumb 1`] = `
<div
class="euiPopover euiColorStopPopover testClass1 testClass2 emotion-euiPopover"
data-test-subj="test subject string"
style="left:0%"
>
<div
Expand All @@ -108,6 +112,7 @@ exports[`renders readOnly EuiColorStopThumb 1`] = `
exports[`renders swatch-only EuiColorStopThumb 1`] = `
<div
class="euiPopover euiColorStopPopover testClass1 testClass2 emotion-euiPopover"
data-test-subj="test subject string"
style="left:0%"
>
<div
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,9 @@

exports[`renders EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down Expand Up @@ -113,8 +114,9 @@ exports[`renders EuiColorStops 1`] = `

exports[`renders compressed EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down Expand Up @@ -224,8 +226,9 @@ exports[`renders compressed EuiColorStops 1`] = `

exports[`renders disabled EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops euiColorStops-isDisabled testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="-1"
>
<p
Expand Down Expand Up @@ -338,8 +341,9 @@ exports[`renders disabled EuiColorStops 1`] = `

exports[`renders empty EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down Expand Up @@ -374,8 +378,9 @@ exports[`renders empty EuiColorStops 1`] = `

exports[`renders fixed stop EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down Expand Up @@ -485,8 +490,9 @@ exports[`renders fixed stop EuiColorStops 1`] = `

exports[`renders free-range EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down Expand Up @@ -521,8 +527,9 @@ exports[`renders free-range EuiColorStops 1`] = `

exports[`renders fullWidth EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiRangeWrapper--fullWidth euiColorStops testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down Expand Up @@ -632,8 +639,9 @@ exports[`renders fullWidth EuiColorStops 1`] = `

exports[`renders max-only EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down Expand Up @@ -668,8 +676,9 @@ exports[`renders max-only EuiColorStops 1`] = `

exports[`renders min-only EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down Expand Up @@ -704,8 +713,9 @@ exports[`renders min-only EuiColorStops 1`] = `

exports[`renders readOnly EuiColorStops 1`] = `
<div
aria-label="aria-label"
class="euiRangeWrapper euiColorStops euiColorStops-isReadOnly testClass1 testClass2"
data-test-subj="euiColorStops"
data-test-subj="euiColorStops test subject string"
tabindex="0"
>
<p
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import { render } from 'enzyme';
import { EuiColorStopThumb } from './color_stop_thumb';

import { requiredProps } from '../../../test';
import { shouldRenderCustomStyles } from '../../../test/internal';

jest.mock('../../portal', () => ({
EuiPortal: ({ children }: { children: any }) => children,
Expand All @@ -21,6 +22,24 @@ const onChange = jest.fn();

// Note: Unit/interaction tests can be found in ./color_stops.test

shouldRenderCustomStyles(
<EuiColorStopThumb
onChange={onChange}
stop={0}
color="#FF0000"
localMin={0}
localMax={24}
globalMin={0}
globalMax={100}
colorPickerMode="default"
isPopoverOpen={true}
openPopover={() => {}}
closePopover={() => {}}
{...requiredProps}
/>,
{ childProps: ['valueInputProps'] }
);

test('renders EuiColorStopThumb', () => {
const thumb = render(
<EuiColorStopThumb
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@

import React, {
FunctionComponent,
CSSProperties,
ReactChild,
useEffect,
useMemo,
Expand Down Expand Up @@ -65,6 +66,7 @@ interface EuiColorStopThumbProps extends CommonProps, ColorStop {
closePopover: () => void;
'data-index'?: string;
'aria-valuetext'?: string;
style?: CSSProperties;
valueInputProps?: Partial<EuiFieldNumberProps>;
}

Expand Down Expand Up @@ -94,7 +96,9 @@ export const EuiColorStopThumb: FunctionComponent<EuiColorStopThumbProps> = ({
closePopover,
'data-index': dataIndex,
'aria-valuetext': ariaValueText,
style,
valueInputProps = {},
...rest
}) => {
const background = useMemo(() => {
const chromaColor = getChromaColor(color, colorPickerShowAlpha);
Expand Down Expand Up @@ -248,6 +252,7 @@ export const EuiColorStopThumb: FunctionComponent<EuiColorStopThumbProps> = ({

return (
<EuiPopover
{...rest}
ref={popoverRef}
className={classes}
anchorClassName="euiColorStopPopover__anchor"
Expand All @@ -260,6 +265,7 @@ export const EuiColorStopThumb: FunctionComponent<EuiColorStopThumbProps> = ({
numberInputRef ? undefined : 'euiColorStopPopover-isLoadingPanel'
}
style={{
...style,
left: `${getPositionFromStopFn(stop)}%`,
}}
button={
Expand Down
12 changes: 12 additions & 0 deletions src/components/color_picker/color_stops/color_stops.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import {
keys,
} from '../../../services';
import { requiredProps, findTestSubject } from '../../../test';
import { shouldRenderCustomStyles } from '../../../test/internal';
import { EuiFieldNumber } from '../../form/field_number';

jest.mock('../../portal', () => ({
Expand All @@ -35,6 +36,17 @@ const colorStopsArray = [
// - Tab to bypass thumbs (tabindex="-1" not respected)
// - Drag to reposition thumb (we can't get real page position info)

shouldRenderCustomStyles(
<EuiColorStops
label="Test"
onChange={onChange}
colorStops={colorStopsArray}
min={0}
max={100}
{...requiredProps}
/>
);

test('renders EuiColorStops', () => {
const colorStops = render(
<EuiColorStops
Expand Down
4 changes: 3 additions & 1 deletion src/components/color_picker/color_stops/color_stops.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -164,6 +164,7 @@ export const EuiColorStops: FunctionComponent<EuiColorStopsProps> = ({
swatches,
showAlpha = false,
valueInputProps,
...rest
}) => {
const sortedStops = useMemo(() => sortStops(colorStops), [colorStops]);
const rangeMax: number = useMemo(() => {
Expand Down Expand Up @@ -494,7 +495,8 @@ export const EuiColorStops: FunctionComponent<EuiColorStopsProps> = ({

return (
<EuiRangeWrapper
data-test-subj="euiColorStops"
{...rest}
data-test-subj={classNames('euiColorStops', rest['data-test-subj'])}
ref={setWrapperRef}
className={classes}
fullWidth={fullWidth}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`EuiSuperUpdateButton renders 1`] = `
<div>
<div
class="euiPopover testClass1 testClass2 emotion-euiPopover"
data-test-subj="test subject string"
>
<div
class="euiPopover__anchor css-zih94u-render"
>
<button
class="euiDatePopoverButton euiDatePopoverButton--start"
data-test-subj="superDatePickerstartDatePopoverButton"
title=""
/>
</div>
</div>
</div>
`;
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
/*
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
* or more contributor license agreements. Licensed under the Elastic License
* 2.0 and the Server Side Public License, v 1; you may not use this file except
* in compliance with, at your election, the Elastic License 2.0 or the Server
* Side Public License, v 1.
*/

import React from 'react';
import { render } from '@testing-library/react';

import { requiredProps } from '../../../../test';
import { shouldRenderCustomStyles } from '../../../../test/internal';

import { EuiDatePopoverButton } from './date_popover_button';

const noop = () => {};

describe('EuiSuperUpdateButton', () => {
const props = {
...requiredProps,
value: '',
dateFormat: '',
timeFormat: '',
timeOptions: {
timeTenseOptions: [],
timeUnitsOptions: [],
relativeOptions: [],
relativeRoundingLabels: {} as any,
refreshUnitsOptions: [],
commonDurationRanges: [],
},
onChange: noop,
onPopoverClose: noop,
onPopoverToggle: noop,
position: 'start' as const,
isOpen: false,
};

shouldRenderCustomStyles(<EuiDatePopoverButton {...props} />, {
childProps: ['buttonProps'],
});

it('renders', () => {
const { container } = render(<EuiDatePopoverButton {...props} />);

expect(container).toMatchSnapshot();
});
});
10 changes: 10 additions & 0 deletions src/components/header/header.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,20 @@
import React from 'react';
import { render } from 'enzyme';
import { requiredProps } from '../../test/required_props';
import { shouldRenderCustomStyles } from '../../test/internal';

import { EuiHeader } from './header';

describe('EuiHeader', () => {
shouldRenderCustomStyles(<EuiHeader />);
shouldRenderCustomStyles(
<EuiHeader sections={[{ breadcrumbs: [{ text: 'test' }] }]} />,
{
childProps: ['sections[0].breadcrumbProps'],
skipParentTest: true,
}
);

test('is rendered', () => {
const component = render(<EuiHeader {...requiredProps} />);

Expand Down
5 changes: 5 additions & 0 deletions src/components/header/header_links/header_links.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,15 @@
import React from 'react';
import { render } from 'enzyme';
import { requiredProps } from '../../../test';
import { shouldRenderCustomStyles } from '../../../test/internal';

import { EuiHeaderLinks, GUTTER_SIZES } from './header_links';

describe('EuiHeaderLinks', () => {
shouldRenderCustomStyles(<EuiHeaderLinks popoverBreakpoints="all" />, {
childProps: ['popoverButtonProps', 'popoverProps'],
});

test('is rendered', () => {
const component = render(<EuiHeaderLinks {...requiredProps} />);

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`EuiResizablePanel renders 1`] = `
<div>
<div
class="euiResizablePanel euiResizablePanel--middle"
id="resizable-panel_generated-id"
style="width: 100%; height: 0%;"
>
<div
aria-label="aria-label"
class="euiPanel euiPanel--transparent euiPanel--paddingMedium euiResizablePanel__content euiResizablePanel__content--scrollable testClass1 testClass2 emotion-euiPanel-grow-none-m-transparent"
data-test-subj="test subject string"
>
Content
</div>
</div>
</div>
`;
Loading