Merge pull request #11 from oslabs-beta/feature-testing

Made some small last minute bug fixes

Author: minzo-kim

demo-app/src/client/Components/Row.tsx

@@ -13,7 +13,7 @@ const Row = (props: RowProps): JSX.Element => {
   for (let i = 0; i < 3; i++) {
     boxes.push(
       <Box
-        key={i}
+        key={`r${props.row + 1}-b${i + 1}`}
         row={props.row}
         column={i}
         handleBoxClick={props.handleBoxClick}
@@ -22,7 +22,11 @@ const Row = (props: RowProps): JSX.Element => {
     );
   }
 
-  return <><div className='row'>{boxes}</div></>;
+  return (
+    <>
+      <div className='row'>{boxes}</div>
+    </>
+  );
 };
 
 export default Row;

src/app/__tests__/ErrorContainer.test.tsx

@@ -35,9 +35,7 @@ describe('unit testing for ErrorContainer.tsx', () => {
 
   test('Reactime website shows as expected', () => {
     render(<ErrorContainer />);
-    expect(
-      screen.getByText('Please visit the Reactime website for more info.'),
-    ).toBeInTheDocument();
+    expect(screen.getByText('Please visit the Reactime Github for more info.')).toBeInTheDocument();
   });
 
   describe('Loading Checks show up as expected', () => {

src/app/components/FrontendTypes.ts

@@ -32,11 +32,9 @@ export interface TreeProps {
     state?: string | object;
     stateSnaphot?: object;
     children?: any[];
-    snapshots?:[];
-    currLocation?:object;
-
-
   };
+  snapshots?:[];
+  currLocation?:object;
 }
 
 export interface BarStackProp {

src/app/components/StateRoute/Tree.tsx

@@ -35,6 +35,7 @@ const getItemString = (type, data: { state?: object | string; name: string; chil
 
 const Tree = (props: TreeProps) => {
   const { snapshot, snapshots, currLocation } = props;
+  // @ts-ignore
   const [store, dispatch] = useStoreContext();
 
   useEffect(() => {
@@ -44,7 +45,9 @@ const Tree = (props: TreeProps) => {
   return (
     <>
       {snapshot && (
+        // @ts-ignore
         <JSONTree
+          // @ts-ignore
           data={snapshots[currLocation.index] || snapshot}
           theme={{ extend: colors, tree: () => ({ className: 'json-tree' }) }}
           shouldExpandNode={() => true}

src/app/containers/ErrorContainer.tsx

@@ -5,7 +5,7 @@ import Loader from '../components/Loader';
 import ErrorMsg from '../components/ErrorMsg';
 import { useStoreContext } from '../store';
 
-function ErrorContainer(): JSX.Element  {
+function ErrorContainer(): JSX.Element {
   const [store, dispatch] = useStoreContext();
   const { tabs, currentTitle, currentTab } = store;
   // hooks for error checks
@@ -94,8 +94,12 @@ function ErrorContainer(): JSX.Element  {
         <ErrorMsg loadingArray={loadingArray} status={status} launchContent={launch} />
       </div>
       <br />
-      <a href='https://www.reacti.me/' target='_blank' rel='noopener noreferrer'>
-        Please visit the Reactime website for more info.
+      <a
+        href='https://github.com/open-source-labs/reactime'
+        target='_blank'
+        rel='noopener noreferrer'
+      >
+        Please visit the Reactime Github for more info.
       </a>
     </div>
   );

src/app/styles/components/_performanceVisx.scss

@@ -37,9 +37,14 @@
   text-align: left;
 }
 
+.saveSeriesContainer {
+  #routes-select, #snapshot-select {
+    background-color: #ff6569 !important;
+  }
+}
+
 #routes-select, #snapshot-select{
   color: white !important;
-  background-color: #ff6569 !important;
   font: 400 14px 'Roboto', sans-serif;
   text-align: left;
   width: 120px;
@@ -48,7 +53,6 @@
   text-align: center;
 }
 
-
 #seriesname {
   float: right;
   width: 220px;