fix(code-viewer): remove Suspense (#190)

* feat(code-viewer): remove Suspense

* chore: rename things so diff is reasonable

* chore: adjust exports so it's not a breaking change

* chore: actually lets not even rename index.tsx to index.ts

Co-authored-by: William Hilton <wmhilton@gmail.com>

Author: P0lip

src/CodeViewer/__tests__/Viewer.spec.tsx

@@ -10,10 +10,6 @@ import { parseCode } from '../utils/parseCode';
 jest.mock('../utils/astToReact');
 jest.mock('../utils/parseCode');
 
-jest.mock('../components/BlockCodeViewer', () => ({
-  BlockCodeViewer: jest.requireActual('../components/BlockCodeViewer/BlockCodeViewer').default,
-}));
-
 jest.mock('../components/BlockCodeViewer/ObservableSet', () => ({
   ObservableSet: class extends Set {
     public readonly addListener = jest.fn().mockImplementation((node, listener) => {

src/CodeViewer/components/BlockCodeViewer/index.tsx

@@ -1,20 +1,2 @@
-import * as React from 'react';
-
-import { FAIcon } from '../../../FAIcon';
-import type { IBlockCodeViewerProps } from './BlockCodeViewer';
-
-const BlockCodeViewerComponent = React.lazy(() => import('./BlockCodeViewer'));
-
-export const BlockCodeViewer: React.FC<IBlockCodeViewerProps> = props => (
-  <React.Suspense fallback={<CodeViewerLoading />}>
-    <BlockCodeViewerComponent {...props} />
-  </React.Suspense>
-);
-
-const CodeViewerLoading = () => (
-  <div>
-    Preparing... <FAIcon icon={['fas', 'code']} size="lg" />
-  </div>
-);
-
-export { IBlockCodeViewerProps };
+export type { IBlockCodeViewerProps } from './BlockCodeViewer';
+export { default as BlockCodeViewer } from './BlockCodeViewer';