test: addressed comments

Author: diana-villalvazo-wgu

src/containers/Dashboard/DashboardLayout.test.jsx

@@ -52,15 +52,15 @@ describe('DashboardLayout', () => {
       expect(sidebarCol.children[0]).toHaveAttribute('id', 'org.openedx.frontend.learner_dashboard.widget_sidebar.v1');
     });
   };
-  const testSidebarLayout = ({ isCollapsed = true }) => {
+  const testSidebarLayout = ({ isCollapsed }) => {
     it('displays withSidebar width for course list column', () => {
       const courseListCol = screen.getByText('test children').parentElement;
       expect(courseListCol).toHaveClass('col-xl-8');
       const sidebarCol = courseListCol.nextSibling;
       expect(sidebarCol).toHaveClass('sidebar-column', !isCollapsed && 'not-collapsed');
     });
   };
-  const testNoSidebarLayout = ({ isCollapsed = true }) => {
+  const testNoSidebarLayout = ({ isCollapsed }) => {
     it('displays noSidebar width for course list column', () => {
       const courseListCol = screen.getByText('test children').parentElement;
       expect(courseListCol).toHaveClass('col-xl-12');
@@ -74,14 +74,14 @@ describe('DashboardLayout', () => {
         hooks.useDashboardLayoutData.mockReturnValueOnce({ ...hookProps, sidebarShowing: true });
       });
       testColumns();
-      testSidebarLayout({});
+      testSidebarLayout({ isCollapsed: true });
     });
     describe('sidebar not showing', () => {
       beforeEach(() => {
         hooks.useDashboardLayoutData.mockReturnValueOnce({ ...hookProps });
       });
       testColumns();
-      testNoSidebarLayout({});
+      testNoSidebarLayout({ isCollapsed: true });
     });
   });
 

src/containers/Dashboard/index.test.jsx

@@ -27,7 +27,7 @@ jest.mock('./DashboardLayout', () => jest.fn(() => <div>DashboardLayout</div>));
 const pageTitle = 'test-page-title';
 
 describe('Dashboard', () => {
-  const createWrapper = (props) => {
+  const createWrapper = (props = {}) => {
     const {
       hasCourses = true,
       initIsPending = true,
@@ -42,7 +42,7 @@ describe('Dashboard', () => {
 
   describe('render', () => {
     it('page title is displayed in sr-only h1 tag', () => {
-      createWrapper({});
+      createWrapper();
       const heading = screen.getByText(pageTitle);
       expect(heading).toHaveClass('sr-only');
     });