Skip to content

Commit 348dfcf

Browse files
committed
Resolved PR comments
1 parent beb1192 commit 348dfcf

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

app/javascript/components/page/components/page-content.jsx

+5-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,11 @@ import css from "../styles.css";
66
const PageContent = ({ children, flex = false, hasNav = false }) => {
77
const contentClasses = clsx(css.content, { [css.contentFlex]: flex, [css.hasNav]: hasNav });
88

9-
return <div className={contentClasses}>{children}</div>;
9+
return (
10+
<div className={contentClasses} data-testid="page-content">
11+
{children}
12+
</div>
13+
);
1014
};
1115

1216
PageContent.propTypes = {

app/javascript/components/pages/export-list/container.spec.js

+5
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,11 @@ describe("<ExportList />", () => {
105105
expect(screen.getByTestId("page-heading")).toBeInTheDocument();
106106
});
107107

108+
it("should render <PageContent>", () => {
109+
mountedComponent(<ExportList />, {}, initialState);
110+
expect(screen.getByTestId("page-content")).toBeInTheDocument();
111+
});
112+
108113
it("should render <IndexTable>", () => {
109114
mountedComponent(<ExportList />, {}, initialState);
110115
expect(screen.getByText(/messages.record_list.rows_per_page/i)).toBeInTheDocument();

0 commit comments

Comments
 (0)