refactor: fix testing failures

openedx · Feb 13, 2024 · 1df4c23 · 1df4c23
1 parent e516b19
commit 1df4c23
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/courseware/course/sequence/sequence-navigation/SequenceNavigation.test.jsx b/src/courseware/course/sequence/sequence-navigation/SequenceNavigation.test.jsx
@@ -181,7 +181,6 @@ describe('Sequence Navigation', () => {
     };
     render(<SequenceNavigation {...testData} unitId={unitBlocks[0].id} />, { store: testStore, wrapWithRouter: true });
     expect(screen.queryByRole('link', { name: /previous/i })).not.toBeInTheDocument();
-    expect(screen.queryByRole('link', { name: /next/i })).not.toBeInTheDocument();
   });
 
   it('removes "Next" for last unit in sequence when navigation is disabled', async () => {
@@ -202,7 +201,6 @@ describe('Sequence Navigation', () => {
       onNavigate: jest.fn(),
     };
     render(<SequenceNavigation {...testData} unitId={unitBlocks[unitBlocks.length - 1].id} />, { store: testStore, wrapWithRouter: true });
-    expect(screen.queryByRole('link', { name: /previous/i })).not.toBeInTheDocument();
     expect(screen.queryByRole('link', { name: /next/i })).not.toBeInTheDocument();
   });
 });
diff --git a/src/courseware/course/sequence/sequence-navigation/hooks.js b/src/courseware/course/sequence/sequence-navigation/hooks.js
@@ -11,10 +11,11 @@ export function useSequenceNavigationMetadata(currentSequenceId, currentUnitId)
   const courseStatus = useSelector(state => state.courseware.courseStatus);
   const sequenceStatus = useSelector(state => state.courseware.sequenceStatus);
 
-
   // If we don't know the sequence and unit yet, then assume no.
   if (courseStatus !== 'loaded' || sequenceStatus !== 'loaded' || !currentSequenceId || !currentUnitId) {
-    return { isFirstUnit: false, isLastUnit: false, navigationDisabledNextSequence: false, navigationDisabledPrevSequence: false };
+    return {
+      isFirstUnit: false, isLastUnit: false, navigationDisabledNextSequence: false, navigationDisabledPrevSequence: false,
+    };
   }
 
   const sequenceIndex = sequenceIds.indexOf(currentSequenceId);