Skip to content

Commit

Permalink
Format code
Browse files Browse the repository at this point in the history
  • Loading branch information
dominic.silva committed Oct 13, 2023
1 parent 7e0022c commit 8ea193e
Show file tree
Hide file tree
Showing 5 changed files with 35 additions and 30 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -64,12 +64,12 @@ export let EditTearsheet = forwardRef(
const handleCurrentForm = (formIndex) => {
setCurrentForm(formIndex);
if (onFormChange) {
onFormChange(formIndex);
onFormChange(formIndex);
}
};

function defaultInfluencer() {
return(
return (
<div className={`${blockClass}__side-nav-wrapper`}>
<SideNav
aria-label="Side navigation"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -183,16 +183,25 @@ describe(componentName, () => {
});

it('renders the influencer with a nav item that matches the form title', async () => {
const { container } = renderEditTearsheet({...defaultProps});
const { container } = renderEditTearsheet({ ...defaultProps });

expect(container.querySelector(`.${carbon.prefix}--side-nav__link-text`)).toHaveTextContent(form1Title);
expect(screen.getByRole('heading', { name: form1Title })).toBeInTheDocument();
expect(
container.querySelector(`.${carbon.prefix}--side-nav__link-text`)
).toHaveTextContent(form1Title);
expect(
screen.getByRole('heading', { name: form1Title })
).toBeInTheDocument();
});

it('should call the provided callback function when the form is changed', async () => {
const onFormChange = jest.fn();
const { container } = renderEditTearsheet({...defaultProps, onFormChange });
const form2NavLink = container.querySelectorAll(`.${carbon.prefix}--side-nav__link-text`)[2];
const { container } = renderEditTearsheet({
...defaultProps,
onFormChange,
});
const form2NavLink = container.querySelectorAll(
`.${carbon.prefix}--side-nav__link-text`
)[2];

await act(() => click(form2NavLink));
expect(onFormChange).toHaveBeenCalledTimes(1);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import { Column, FormGroup, Grid } from '@carbon/react';
import { FormContext, FormNumberContext } from './EditTearsheet';
import { pkg } from '../../settings';
import pconsole from '../../global/js/utils/pconsole';
import {useRetrieveFormTitles} from "../../global/js/hooks/useRetrieveFormTitles";
import { useRetrieveFormTitles } from '../../global/js/hooks/useRetrieveFormTitles';

const componentName = 'EditTearsheetForm';
const blockClass = `${pkg.prefix}--tearsheet-edit__form`;
Expand Down Expand Up @@ -42,7 +42,7 @@ export let EditTearsheetForm = forwardRef(
) => {
const formContext = useContext(FormContext);
const formNumber = useContext(FormNumberContext);
useRetrieveFormTitles({formContext, formNumber, title});
useRetrieveFormTitles({ formContext, formNumber, title });

return formContext ? (
<div
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ import cx from 'classnames';
import { pkg } from '../../../settings';
import { EditTearsheet } from '../EditTearsheet';
import { EditTearsheetForm } from '../EditTearsheetForm';
import {action} from "@storybook/addon-actions";
import { action } from '@storybook/addon-actions';

const blockClass = `${pkg.prefix}--tearsheet-edit-multi-form`;

Expand Down Expand Up @@ -66,8 +66,8 @@ export const MultiFormEditTearsheet = ({
};

const handleFormChange = () => {
action("handleFormChange")();
}
action('handleFormChange')();
};

return (
<div>
Expand Down
32 changes: 14 additions & 18 deletions packages/ibm-products/src/global/js/hooks/useRetrieveFormTitles.js
Original file line number Diff line number Diff line change
@@ -1,21 +1,17 @@
import { useEffect } from 'react';

export const useRetrieveFormTitles = ({
formContext,
formNumber,
title,
}) => {
useEffect(() => {
if (formContext) {
formContext.setFormTitles((prev) => {
const prevTitle = prev[formNumber];
if (prevTitle !== title) {
const clone = [...prev];
clone[formNumber] = title;
return clone;
}
return prev;
});
export const useRetrieveFormTitles = ({ formContext, formNumber, title }) => {
useEffect(() => {
if (formContext) {
formContext.setFormTitles((prev) => {
const prevTitle = prev[formNumber];
if (prevTitle !== title) {
const clone = [...prev];
clone[formNumber] = title;
return clone;
}
}, [title, formContext, formNumber]);
}
return prev;
});
}
}, [title, formContext, formNumber]);
};

0 comments on commit 8ea193e

Please sign in to comment.