Remove sinon and replace sinon.spy() with jest.fn() (#358)

react-ui-org · Sep 1, 2023 · fef54e6 · fef54e6
1 parent 4d49a8c
commit fef54e6
Show file tree

Hide file tree

Showing 16 changed files with 39 additions and 205 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -91,7 +91,6 @@
     "prop-types": "^15.8.1",
     "sass": "^1.66.1",
     "sass-loader": "^13.3.2",
-    "sinon": "^14.0.2",
     "stylelint": "^14.16.1",
     "stylelint-webpack-plugin": "^4.1.1",
     "terser-webpack-plugin": "^5.3.9",

diff --git a/src/components/Alert/__tests__/Alert.test.jsx b/src/components/Alert/__tests__/Alert.test.jsx
@@ -1,5 +1,4 @@
 import React from 'react';
-import sinon from 'sinon';
 import {
   render,
   screen,
@@ -51,7 +50,7 @@ describe('rendering', () => {
 
 describe('functionality', () => {
   it('calls onClose() on Close button click', async () => {
-    const spy = sinon.spy();
+    const spy = jest.fn();
     render((
       <Alert
         {...mandatoryProps}
@@ -60,6 +59,6 @@ describe('functionality', () => {
     ));
 
     await userEvent.click(screen.getByTitle('Close'));
-    expect(spy.calledOnce).toEqual(true);
+    expect(spy).toHaveBeenCalled();
   });
 });
diff --git a/src/components/Button/__tests__/Button.test.jsx b/src/components/Button/__tests__/Button.test.jsx
@@ -4,7 +4,6 @@ import {
   screen,
   within,
 } from '@testing-library/react';
-import sinon from 'sinon';
 import userEvent from '@testing-library/user-event';
 import { actionColorPropTest } from '../../../../tests/propTests/actionColorPropTest';
 import { blockPropTest } from '../../../../tests/propTests/blockPropTest';
@@ -157,7 +156,7 @@ describe('rendering', () => {
 
 describe('functionality', () => {
   it('calls synthetic event onClick()', async () => {
-    const spy = sinon.spy();
+    const spy = jest.fn();
     render((
       <Button
         {...mandatoryProps}
@@ -166,6 +165,6 @@ describe('functionality', () => {
     ));
 
     await userEvent.click(screen.getByText('label'));
-    expect(spy.calledOnce).toEqual(true);
+    expect(spy).toHaveBeenCalled();
   });
 });
diff --git a/src/components/CheckboxField/__tests__/CheckboxField.test.jsx b/src/components/CheckboxField/__tests__/CheckboxField.test.jsx
@@ -1,5 +1,4 @@
 import React from 'react';
-import sinon from 'sinon';
 import {
   render,
   screen,
@@ -61,7 +60,7 @@ describe('rendering', () => {
 
 describe('functionality', () => {
   it('calls synthetic event onChange()', async () => {
-    const spy = sinon.spy();
+    const spy = jest.fn();
     render((
       <CheckboxField
         {...mandatoryProps}
@@ -70,7 +69,7 @@ describe('functionality', () => {
     ));
 
     await userEvent.click(screen.getByLabelText('label'));
-    expect(spy.calledOnce).toEqual(true);
+    expect(spy).toHaveBeenCalled();
   });
 });
 
diff --git a/src/components/FileInputField/__tests__/FileInputField.test.jsx b/src/components/FileInputField/__tests__/FileInputField.test.jsx
@@ -1,5 +1,4 @@
 import React from 'react';
-import sinon from 'sinon';
 import {
   render,
   screen,
@@ -65,7 +64,7 @@ describe('rendering', () => {
 
 describe('functionality', () => {
   it('calls synthetic event onChange()', async () => {
-    const spy = sinon.spy();
+    const spy = jest.fn();
     render((
       <FileInputField
         {...mandatoryProps}
@@ -75,6 +74,6 @@ describe('functionality', () => {
 
     const file = new File(['hello'], 'hello.png', { type: 'image/png' });
     await userEvent.upload(screen.getByLabelText('label'), file);
-    expect(spy.calledOnce).toEqual(true);
+    expect(spy).toHaveBeenCalled();
   });
 });