fix: temporarily remove end exam button when viewing non exam content (…

…#33)

* fix: temporarily remove end exam button when viewing non exam content

* fix: use default prop to limit code impact

src/exam/Exam.jsx

@@ -49,6 +49,10 @@ const Exam = ({ isTimeLimited, children }) => {
 
   const sequenceContent = <>{children}</>;
 
+  // Temporary fix for CR-3842. We need to show this timer but the end exam button does not
+  // fully work yet.
+  const allowEndExam = !!examId;
+
   return (
     <div className="d-flex flex-column justify-content-center">
       {showTimer && (
@@ -58,6 +62,7 @@ const Exam = ({ isTimeLimited, children }) => {
           expireExamAttempt={expireExam}
           pollExamAttempt={pollAttempt}
           pingAttempt={pingAttempt}
+          allowEndExam={allowEndExam}
         />
       )}
       {apiErrorMsg && <ExamAPIError />}

src/timer/ExamTimerBlock.jsx

@@ -16,7 +16,7 @@ import {
  * Exam timer block component.
  */
 const ExamTimerBlock = injectIntl(({
-  attempt, stopExamAttempt, expireExamAttempt, pollExamAttempt, intl, pingAttempt,
+  attempt, stopExamAttempt, expireExamAttempt, pollExamAttempt, intl, pingAttempt, allowEndExam,
 }) => {
   const [isShowMore, showMore, showLess] = useToggle(false);
   const [alertVariant, setAlertVariant] = useState('info');
@@ -94,6 +94,7 @@ const ExamTimerBlock = injectIntl(({
           >
 
             {attempt.attempt_status !== ExamStatus.READY_TO_SUBMIT
+              && allowEndExam
               && (
               <Button className="mr-3" variant="outline-primary" onClick={stopExamAttempt}>
                 <FormattedMessage
@@ -123,6 +124,11 @@ ExamTimerBlock.propTypes = {
   }),
   stopExamAttempt: PropTypes.func.isRequired,
   expireExamAttempt: PropTypes.func.isRequired,
+  allowEndExam: PropTypes.bool,
+};
+
+ExamTimerBlock.defaultProps = {
+  allowEndExam: true,
 };
 
 export default ExamTimerBlock;