Filling out the rest of the chart tests

Co-authored-by: Alexander Holden <[email protected]>
Co-authored-by: William Owen <[email protected]>
Co-authored-by: Jessica Lee <[email protected]>

Author: 4 people

__tests__/charts/MemoryChart.test.tsx

@@ -0,0 +1,42 @@
+import React from 'react';
+import { render, screen } from '@testing-library/react';
+import MemoryChart from '../../app/charts/MemoryChart';
+import { HealthContext } from '../../app/context/HealthContext';
+import mockData from '../mock_data.json';
+import '@testing-library/jest-dom';
+
+jest.mock('electron', () => ({
+  ipcRenderer: {
+    send: () => jest.fn(),
+    on: () => mockData,
+  },
+}));
+
+describe('Speed Chart', () => {
+  const props = {
+    healthData: mockData,
+    setHealthData: jest.fn(),
+    fetchHealthData: jest.fn(),
+    parseHealthData: jest.fn(),
+    services: jest.fn(),
+  };
+  beforeEach(() => {
+    render(
+      <HealthContext.Provider value={props}>
+        <MemoryChart sizing="solo" colourGenerator={jest.fn()} />
+      </HealthContext.Provider>
+    );
+  });
+
+  it('Should render', () => {
+    expect(screen).toBeTruthy();
+  });
+
+  it('Should render graph', () => {
+    expect(screen.getByTestId('Speed Chart').firstChild).toBeTruthy();
+  });
+
+  it('Should render graph', () => {
+    expect(screen.getByTestId('Speed Chart').firstChild).toBeTruthy();
+  });
+});

__tests__/charts/SpeedChart.test.tsx

@@ -11,7 +11,6 @@ jest.mock('electron', () => ({
     on: () => mockData,
   },
 }));
-
 describe('Speed Chart', () => {
   const props = {
     healthData: mockData,
@@ -20,39 +19,37 @@ describe('Speed Chart', () => {
     parseHealthData: jest.fn(),
     services: jest.fn(),
   };
+  let graph;
   beforeEach(() => {
     render(
       <HealthContext.Provider value={props}>
         <SpeedChart sizing="solo" colourGenerator={jest.fn()} />
       </HealthContext.Provider>
     );
+    graph = screen.getByTestId('Speed Chart').firstChild;
   });
 
   it('Should render', () => {
     expect(screen).toBeTruthy();
   });
 
   it('Should render graph', () => {
-    expect(screen.getByTestId('Speed Chart').firstChild).toBeTruthy();
+    expect(graph).toBeTruthy();
   });
 
-  it('Should render graph', () => {
-    expect(screen.getByTestId('Speed Chart').firstChild).toBeTruthy();
+  it('Should be alone', () => {
+    expect(graph.previousSibling).toBe(null);
+    expect(graph.nextSibling).toBe(null);
   });
 
-  /**
-   * expect 'Speed Chart' to be on screen
-   */
-
-  // it('Should have three p tags', () => {
-  //   expect(element.querySelectorAll('p').length).toBe(3);
-  // });
-
-  // it('Should have three h3 tags', () => {
-  //   expect(element.querySelectorAll('h3').length).toBe(3);
-  // });
+  it('Should not scroll', () => {
+    expect(graph.scrollWidth).toBe(0);
+    expect(graph.scrollHeight).toBe(0);
+    expect(graph.scrollLeft).toBe(0);
+    expect(graph.scrollTop).toBe(0);
+  });
 
-  // it('Should have one div', () => {
-  //   expect(element.querySelectorAll('div').length).toBe(1);
-  // });
+  it('Should render graph', () => {
+    console.log(graph.outerHTML);
+  });
 });

__tests__/components/CreateAdmin.test.tsx

@@ -28,13 +28,13 @@ describe('Create Admin Page', () => {
     expect(element.querySelectorAll('input').length).toBe(3);
   });
 
-  xit('Create Account button should submit email, username, and password to addUser', () => {
+  it('Create Account button should submit email, username, and password to addUser', () => {
     const element = screen.getByTestId('CreateAdmin');
     const inputs = element.querySelectorAll('input');
     inputs[0].value = 'me';
     inputs[1].value = '[email protected]';
     inputs[2].value = 'me123';
-    fireEvent.click(screen.getByText('Create Accp[ount'));
+    fireEvent.click(screen.getByText('Create Account'));
     expect(ipcRenderer.sendSync).toHaveBeenCalledWith('addUser', {
       email: '[email protected]',
       username: 'me',

jest_setup/windowMock.js

@@ -1,15 +1,8 @@
-/* istanbul ignore file */
-/**
- * Mock window environment
- */
+// Mock window environment
 window.require = require;
 
-/**
-* Mock import statements for Plotly
-*/
+// Mock import statements for Plotly
 window.URL.createObjectURL = () => {};
 
-/**
-* Mock getContext
-*/
+// Mock get context
 HTMLCanvasElement.prototype.getContext = () => {};

package.json

@@ -12,7 +12,7 @@
     "electronTest": "cross-env NODE_ENV=development electron .",
     "electron:wsl": "electron . --no-sandbox",
     "test": "jest",
-    "test:watch": "jest ./*",
+    "test:watch": "jest --watch",
     "test:electron": "jest --detectOpenHandles ./__tests__/electron/*",
     "test:app": "jest ./__tests__/app/*",
     "both": "npm run start & sleep 5 && npm run electronTest",