chore: remove react callback for this one because hook inconsistency (#182)

* chore: remove react callback for this one because hook inconsistency

* chore: lint

Author: leangseu-edx

src/hooks/actions/useLoadNextAction.js

@@ -1,4 +1,3 @@
-import React from 'react';
 import { useIntl } from '@edx/frontend-platform/i18n';
 
 import { usePageDataStatus, useRefreshPageData, useStepInfo } from 'hooks/app';
@@ -28,12 +27,12 @@ export default () => {
 
   return {
     action: {
-      onClick: React.useCallback(() => {
+      onClick: () => {
         if (isMounted.current) {
           refreshPageData();
           resetAssessment();
         }
-      }, [refreshPageData, resetAssessment, isMounted]),
+      },
       labels: {
         default: label(messages.loadNext),
         [MutationStatus.idle]: label(messages.loadNext),

src/hooks/actions/useLoadNextAction.test.js

@@ -96,14 +96,14 @@ describe('useLoadNextAction', () => {
       });
       it('refreshes page data and assessment on click if mounted', () => {
         const { onClick } = useLoadNextAction().action;
-        onClick.useCallback.cb();
+        onClick();
         expect(refreshPageData).toHaveBeenCalledWith();
         expect(resetAssessment).toHaveBeenCalledWith();
       });
       it('does not refresh data or assessment if not mounted', () => {
         when(useIsMounted).calledWith().mockReturnValueOnce({ current: false });
         const { onClick } = useLoadNextAction().action;
-        onClick.useCallback.cb();
+        onClick();
         expect(refreshPageData).not.toHaveBeenCalled();
         expect(resetAssessment).not.toHaveBeenCalled();
       });