Hide auto-scroll check for transcripts without time-synced content

samvera-labs · Feb 1, 2024 · 130a7a6 · 130a7a6
1 parent 6f8334c
commit 130a7a6
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 12 deletions.
diff --git a/src/components/Transcript/TranscriptMenu/TranscriptSelector.js b/src/components/Transcript/TranscriptMenu/TranscriptSelector.js
@@ -1,6 +1,7 @@
 import React from 'react';
 import PropTypes from 'prop-types';
 import TranscriptDownloader from './TranscriptDownloader';
+import { TRANSCRIPT_TYPES } from '@Services/transcript-parser';
 
 const MACHINE_GEN_MESSAGE = 'Machine-generated transcript may contain errors.';
 
@@ -11,7 +12,7 @@ const TanscriptSelector = ({
   noTranscript,
   setAutoScroll
 }) => {
-  const { filename, id, tUrl, tFileExt, isMachineGen } = transcriptInfo;
+  const { filename, id, tUrl, tFileExt, tType, isMachineGen } = transcriptInfo;
 
   const [autoScrollCheck, setAutoScrollCheck] = React.useState(true);
 
@@ -63,17 +64,20 @@ const TanscriptSelector = ({
             {MACHINE_GEN_MESSAGE}
           </p>
         }
-        <div className="ramp--transcript_auto_scroll_check">
-          <input
-            type="checkbox"
-            id="auto-scroll-check"
-            name="autoscrollcheck"
-            aria-checked={autoScrollCheck}
-            checked={autoScrollCheck}
-            onChange={handleOnChange}
-          />
-          <label htmlFor="auto-scroll-check">Auto-scroll with media</label>
-        </div>
+        {tType === TRANSCRIPT_TYPES.timedText && (
+          <div className="ramp--transcript_auto_scroll_check"
+            data-testid="transcript-auto-scroll-check">
+            <input
+              type="checkbox"
+              id="auto-scroll-check"
+              name="autoscrollcheck"
+              aria-checked={autoScrollCheck}
+              checked={autoScrollCheck}
+              onChange={handleOnChange}
+            />
+            <label htmlFor="auto-scroll-check">Auto-scroll with media</label>
+          </div>)
+        }
       </div>
     );
   } else {

diff --git a/src/components/Transcript/TranscriptMenu/TranscriptSelector.test.js b/src/components/Transcript/TranscriptMenu/TranscriptSelector.test.js
@@ -52,6 +52,7 @@ describe('TranscriptSelector component', () => {
       tFileExt: 'json'
     },
     noTranscript: false,
+    setAutoScroll: jest.fn()
   };
   describe('with default props', () => {
     beforeEach(() => {
@@ -96,4 +97,32 @@ describe('TranscriptSelector component', () => {
       "Machine-generated transcript may contain errors."
     );
   });
+
+  test('with time synced transcript content', () => {
+    let updatedProps = {
+      ...props,
+      transcriptInfo: {
+        ...props.transcriptInfo,
+        tType: 1,
+      }
+    };
+    render(<TranscriptSelector {...updatedProps} />);
+    expect(screen.getByTestId('transcript-selector')).toBeInTheDocument();
+    expect(screen.getByTestId('transcript-downloader')).toBeInTheDocument();
+    expect(screen.queryByTestId('transcript-auto-scroll-check')).toBeInTheDocument();
+  });
+
+  test('without time synced transcript content', () => {
+    let updatedProps = {
+      ...props,
+      transcriptInfo: {
+        ...props.transcriptInfo,
+        tType: 3,
+      }
+    };
+    render(<TranscriptSelector {...updatedProps} />);
+    expect(screen.getByTestId('transcript-selector')).toBeInTheDocument();
+    expect(screen.getByTestId('transcript-downloader')).toBeInTheDocument();
+    expect(screen.queryByTestId('transcript-auto-scroll-check')).not.toBeInTheDocument();
+  });
 });