diff --git a/package-lock.json b/package-lock.json index 2e7f0c22..1ba44e04 100644 --- a/package-lock.json +++ b/package-lock.json @@ -32,7 +32,6 @@ "react-router-dom": "^6.27.0", "reactstrap": "^9.2.3", "redux-localstorage-simple": "^2.5.1", - "uuid": "^10.0.0", "workbox-core": "^7.1.0", "workbox-expiration": "^7.1.0", "workbox-precaching": "^7.1.0", @@ -11580,19 +11579,6 @@ "dev": true, "license": "MIT" }, - "node_modules/uuid": { - "version": "10.0.0", - "resolved": "https://registry.npmjs.org/uuid/-/uuid-10.0.0.tgz", - "integrity": "sha512-8XkAphELsDnEGrDxUOHB3RGvXz6TeuYSGEZBOjtTtPm2lwhGBjLgOzLHB63IUWfBpNucQjND6d3AOudO+H3RWQ==", - "funding": [ - "https://github.com/sponsors/broofa", - "https://github.com/sponsors/ctavan" - ], - "license": "MIT", - "bin": { - "uuid": "dist/bin/uuid" - } - }, "node_modules/validate-npm-package-license": { "version": "3.0.4", "resolved": "https://registry.npmjs.org/validate-npm-package-license/-/validate-npm-package-license-3.0.4.tgz", @@ -20190,11 +20176,6 @@ "version": "1.0.2", "dev": true }, - "uuid": { - "version": "10.0.0", - "resolved": "https://registry.npmjs.org/uuid/-/uuid-10.0.0.tgz", - "integrity": "sha512-8XkAphELsDnEGrDxUOHB3RGvXz6TeuYSGEZBOjtTtPm2lwhGBjLgOzLHB63IUWfBpNucQjND6d3AOudO+H3RWQ==" - }, "validate-npm-package-license": { "version": "3.0.4", "resolved": "https://registry.npmjs.org/validate-npm-package-license/-/validate-npm-package-license-3.0.4.tgz", diff --git a/package.json b/package.json index 080a9f54..65bee6d1 100644 --- a/package.json +++ b/package.json @@ -49,7 +49,6 @@ "react-router-dom": "^6.27.0", "reactstrap": "^9.2.3", "redux-localstorage-simple": "^2.5.1", - "uuid": "^10.0.0", "workbox-core": "^7.1.0", "workbox-expiration": "^7.1.0", "workbox-precaching": "^7.1.0", diff --git a/src/utils/utils.ts b/src/utils/utils.ts index d16080cc..b1b11ca0 100644 --- a/src/utils/utils.ts +++ b/src/utils/utils.ts @@ -1,9 +1,6 @@ import type { SyntheticEvent } from 'react'; -import { v4 } from 'uuid'; export const handleEventPreventingDefault = (handler: () => T) => (e: SyntheticEvent) => { e.preventDefault(); handler(); }; - -export const randomUUID = () => v4(); diff --git a/test/servers/helpers/ImportServersBtn.test.tsx b/test/servers/helpers/ImportServersBtn.test.tsx index f28618dc..62a1888c 100644 --- a/test/servers/helpers/ImportServersBtn.test.tsx +++ b/test/servers/helpers/ImportServersBtn.test.tsx @@ -68,12 +68,15 @@ describe('', () => { { btnName: 'Save anyway',savesDuplicatedServers: true }, { btnName: 'Discard', savesDuplicatedServers: false }, ])('creates expected servers depending on selected option in modal', async ({ btnName, savesDuplicatedServers }) => { - const existingServer: ServerWithId = { + const existingServerData: ServerData = { name: 'existingServer', - id: 'existingserver-s.test', url: 'http://s.test/existingUrl', apiKey: 'existingApiKey', }; + const existingServer: ServerWithId = { + ...existingServerData, + id: 'existingserver-s.test', + }; const newServer: ServerData = { name: 'newServer', url: 'http://s.test/newUrl', apiKey: 'newApiKey' }; const { user } = setUp({}, { [existingServer.id]: existingServer }); @@ -86,7 +89,7 @@ describe('', () => { expect(createServersMock).toHaveBeenCalledWith( savesDuplicatedServers - ? [existingServer, expect.objectContaining(newServer)] + ? [expect.objectContaining(existingServerData), expect.objectContaining(newServer)] : [expect.objectContaining(newServer)], ); expect(onImportMock).toHaveBeenCalledTimes(1); diff --git a/test/servers/reducers/selectedServer.test.ts b/test/servers/reducers/selectedServer.test.ts index fac4b708..8c602052 100644 --- a/test/servers/reducers/selectedServer.test.ts +++ b/test/servers/reducers/selectedServer.test.ts @@ -9,7 +9,6 @@ import { selectedServerReducerCreator, selectServer as selectServerCreator, } from '../../../src/servers/reducers/selectedServer'; -import { randomUUID } from '../../../src/utils/utils'; describe('selectedServerReducer', () => { const dispatch = vi.fn(); @@ -41,7 +40,7 @@ describe('selectedServerReducer', () => { ['latest', MAX_FALLBACK_VERSION, 'latest'], ['%invalid_semver%', MIN_FALLBACK_VERSION, '%invalid_semver%'], ])('dispatches proper actions', async (serverVersion, expectedVersion, expectedPrintableVersion) => { - const id = randomUUID(); + const id = crypto.randomUUID(); const getState = createGetStateMock(id); const expectedSelectedServer = { id, @@ -60,7 +59,7 @@ describe('selectedServerReducer', () => { }); it('dispatches error when health endpoint fails', async () => { - const id = randomUUID(); + const id = crypto.randomUUID(); const getState = createGetStateMock(id); const expectedSelectedServer = fromPartial({ id, serverNotReachable: true }); @@ -73,7 +72,7 @@ describe('selectedServerReducer', () => { }); it('dispatches error when server is not found', async () => { - const id = randomUUID(); + const id = crypto.randomUUID(); const getState = vi.fn(() => fromPartial({ servers: {} })); const expectedSelectedServer: NotFoundServer = { serverNotFound: true };