fix: code cleanup and renaming components (#42)

src/App.tsx

@@ -6,14 +6,12 @@ import {
 
 import './custom.scss';
 
-import Form from './views/Form';
-import Table from './views/Table';
 import ProtectedRoute from './routes/ProtectedRoute';
 import { useAuth } from './contexts/AuthContext';
 import SilentCheckSso from './components/SilentCheckSso';
 import Logout from './components/Logout';
 import Layout from './layout/PrivateLayout';
-import NewLanding from './views/NewLanding';
+import Landing from './views/Landing';
 import Dashboard from './views/Dashboard/dashboard';
 import SeedlotDashboard from './views/SeedlotDashboard';
 
@@ -28,7 +26,7 @@ const App: React.FC = () => {
   return (
     <BrowserRouter>
       <Routes>
-        <Route path="/" element={<NewLanding />} />
+        <Route path="/" element={<Landing />} />
         <Route path="/" element={<Layout />}>
           <Route path="/silent-check-sso" element={<SilentCheckSso />} />
           <Route path="/logout" element={<Logout />} />
@@ -50,24 +48,6 @@ const App: React.FC = () => {
               </ProtectedRoute>
             )}
           />
-
-          <Route
-            path="/form"
-            element={(
-              <ProtectedRoute signed={signed}>
-                <Form />
-              </ProtectedRoute>
-            )}
-          />
-
-          <Route
-            path="/table"
-            element={(
-              <ProtectedRoute signed={signed}>
-                <Table />
-              </ProtectedRoute>
-            )}
-          />
         </Route>
       </Routes>
     </BrowserRouter>

src/__test__/components/FavouriteActivities.test.tsx

@@ -13,7 +13,7 @@ import makeServer from '../../mock-api/server';
 jest.setTimeout(10000);
 
 // empty section should be tested in the future
-describe('the Favorite Activities component', () => {
+describe('the Favourite Activities component', () => {
   beforeAll(() => {
     makeServer('test');
   });

src/__test__/components/FavoriteCard.test.tsx → src/__test__/components/FavouriteCard.test.tsx

@@ -1,13 +1,13 @@
 /* eslint-disable no-undef */
 import React from 'react';
 import { fireEvent, render, screen } from '@testing-library/react';
-import FavoriteCard from '../../components/Card/FavoriteCard';
+import FavouriteCard from '../../components/Card/FavouriteCard';
 import '@testing-library/jest-dom/extend-expect';
 
-describe('the favorite card component', () => {
+describe('the favourite card component', () => {
   it('should render correctly with headers', async () => {
     const { container } = render(
-      <FavoriteCard
+      <FavouriteCard
         header="Test"
         description="For testing"
         icon="SoilMoistureField"
@@ -24,7 +24,7 @@ describe('the favorite card component', () => {
 
   it('should render card highlighted with different style', () => {
     const { container } = render(
-      <FavoriteCard
+      <FavouriteCard
         header="Test"
         description="For testing"
         icon="SoilMoistureField"
@@ -38,7 +38,7 @@ describe('the favorite card component', () => {
 
   it('should click in the button and open the options', () => {
     const { container } = render(
-      <FavoriteCard
+      <FavouriteCard
         header="Test"
         description="For testing"
         icon="SoilMoistureField"
@@ -53,7 +53,7 @@ describe('the favorite card component', () => {
 
   it('should close the dropdown menu when click outside', () => {
     const { container } = render(
-      <FavoriteCard
+      <FavouriteCard
         header="Test"
         description="For testing"
         icon="SoilMoistureField"

src/__test__/components/PageTitle.test.tsx

@@ -14,9 +14,9 @@ describe('the page title component', () => {
     expect(screen.getByText('Test Subtitle')).toBeInTheDocument();
   });
 
-  it('should render correctly with favorite option', () => {
+  it('should render correctly with favourite option', () => {
     render(
-      <PageTitle title="Test title" subtitle="Test Subtitle" favorite />
+      <PageTitle title="Test title" subtitle="Test Subtitle" favourite />
     );
 
     expect(screen.getAllByRole('button')[0]).toBeInTheDocument();

src/__test__/views/NewLanding.test.tsx → src/__test__/views/Landing.test.tsx

@@ -3,15 +3,15 @@ import React from 'react';
 import { BrowserRouter } from 'react-router-dom';
 import { render } from '@testing-library/react';
 import renderer from 'react-test-renderer';
-import NewLanding from '../../views/NewLanding';
+import Landing from '../../views/Landing';
 import { AuthProvider } from '../../contexts/AuthContext';
 
-describe('NewLanding component test', () => {
+describe('Landing component test', () => {
   it('should have the correct title', () => {
     const { getByTestId } = render(
       <BrowserRouter>
         <AuthProvider>
-          <NewLanding />
+          <Landing />
         </AuthProvider>
       </BrowserRouter>
     );
@@ -33,15 +33,15 @@ describe('NewLanding component test', () => {
   });
 
   it('should match the snapshot', () => {
-    const newLanding = renderer
+    const landing = renderer
       .create(
         <BrowserRouter>
           <AuthProvider>
-            <NewLanding />
+            <Landing />
           </AuthProvider>
         </BrowserRouter>
       ).toJSON();
 
-    expect(newLanding).toMatchSnapshot();
+    expect(landing).toMatchSnapshot();
   });
 });

src/__test__/views/__snapshots__/NewLanding.test.tsx.snap → src/__test__/views/__snapshots__/Landing.test.tsx.snap

@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`NewLanding component test should match the snapshot 1`] = `
+exports[`Landing component test should match the snapshot 1`] = `
 <div
   className="landing-grid cds--css-grid cds--css-grid--full-width"
 >