Skip to content

Commit 7895310

Browse files
committed
test: remove redundant mock cleanup
Amp-Thread-ID: https://ampcode.com/threads/T-019feeff-305b-73a5-bb82-3ae3a74e816d
1 parent 631e0aa commit 7895310

120 files changed

Lines changed: 21 additions & 362 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

apps/website/src/utils/cloudNodes.build.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@ describe('loadPacksForBuild', () => {
3535
const savedVercelEnv = process.env.VERCEL_ENV
3636

3737
beforeEach(() => {
38-
fetchCloudNodesMock.mockReset()
39-
reportCloudNodesOutcomeMock.mockReset()
4038
delete process.env.VERCEL_ENV
4139
})
4240

apps/website/src/utils/cloudNodes.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,9 +94,7 @@ describe('fetchCloudNodesForBuild', () => {
9494

9595
beforeEach(() => {
9696
resetCloudNodesFetcherForTests()
97-
fetchRegistryPacksWithNodesMock.mockReset()
9897
fetchRegistryPacksWithNodesMock.mockResolvedValue(new Map())
99-
sanitizeCallSpy.mockReset()
10098
delete process.env.WEBSITE_CLOUD_API_KEY
10199
})
102100

src/components/dialog/confirm/confirmDialog.test.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
* Confirm* sections carry their own). Catches accidental reverts of the
55
* Phase 6 renderer flip.
66
*/
7-
import { beforeEach, describe, expect, it, vi } from 'vitest'
7+
import { describe, expect, it, vi } from 'vitest'
88

99
const showDialog = vi.hoisted(() => vi.fn())
1010

@@ -18,10 +18,6 @@ import ConfirmHeader from '@/components/dialog/confirm/ConfirmHeader.vue'
1818
import { showConfirmDialog } from '@/components/dialog/confirm/confirmDialog'
1919

2020
describe('showConfirmDialog Reka renderer opt-in', () => {
21-
beforeEach(() => {
22-
showDialog.mockReset()
23-
})
24-
2521
it("sets renderer 'reka' with size 'md' and zeroed section padding", () => {
2622
showConfirmDialog()
2723

src/components/dialog/content/signin/ApiKeyForm.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ const i18n = createI18n({
5858

5959
describe('ApiKeyForm', () => {
6060
beforeEach(() => {
61-
mockStoreApiKey.mockReset()
6261
mockLoadingRef.value = false
6362
})
6463

src/components/dialog/content/signin/SignInForm.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,6 @@ const loginButtonText = enMessages.auth.login.loginButton
6161

6262
describe('SignInForm', () => {
6363
beforeEach(() => {
64-
mockSendPasswordReset.mockReset()
65-
mockToastAdd.mockReset()
6664
mockLoadingRef.value = false
6765
})
6866

src/components/dialog/content/signin/SignUpForm.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,6 @@ describe('SignUpForm', () => {
108108
mockTurnstileEnabled.value = false
109109
mockTurnstileToken.value = ''
110110
mockTurnstileUnavailable.value = false
111-
mockReset.mockClear()
112111
emitTurnstileToken = undefined
113112
emitTurnstileUnavailable = undefined
114113
})

src/components/error/ErrorOverlay.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,11 +125,9 @@ describe('ErrorOverlay', () => {
125125
mockErrorGroups.missingModelGroups.value = []
126126
mockErrorGroups.missingMediaGroups.value = []
127127
mockErrorGroups.swapNodeGroups.value = []
128-
mockOpenPanel.mockClear()
129128
mockCanvasStore.linearMode = false
130129
mockCanvasStore.canvas = null
131130
mockCanvasStore.currentGraph = null
132-
mockCanvasStore.updateSelectedItems.mockClear()
133131
})
134132

135133
it('renders a single overlay message without list markup', async () => {

src/components/helpcenter/HelpCenterMenuContent.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,6 @@ describe('HelpCenterMenuContent feedback item', () => {
120120
distribution.isCloud = false
121121
distribution.isDesktop = false
122122
distribution.isNightly = false
123-
commandStoreExecute.mockReset()
124123
openSpy = vi.spyOn(window, 'open').mockReturnValue(null)
125124
})
126125

src/components/load3d/Load3DScene.test.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,6 @@ describe('Load3DScene', () => {
7474
beforeEach(() => {
7575
dragState.isDragging = ref(false)
7676
dragState.dragMessage = ref('')
77-
dragState.handleDragOver.mockReset()
78-
dragState.handleDragLeave.mockReset()
79-
dragState.handleDrop.mockReset()
8077
dragState.capturedOptions = null
8178
})
8279

src/components/load3d/controls/HDRIControls.test.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/* eslint-disable testing-library/no-container, testing-library/no-node-access -- hidden file input has no role/label, queried by selector */
22
import { render, screen } from '@testing-library/vue'
33
import userEvent from '@testing-library/user-event'
4-
import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
4+
import { afterEach, describe, expect, it, vi } from 'vitest'
55
import { ref } from 'vue'
66
import { createI18n } from 'vue-i18n'
77

@@ -67,10 +67,6 @@ function renderComponent(opts: RenderOpts = {}) {
6767
}
6868

6969
describe('HDRIControls', () => {
70-
beforeEach(() => {
71-
addAlert.mockClear()
72-
})
73-
7470
afterEach(() => {
7571
document.body.innerHTML = ''
7672
})

0 commit comments

Comments
 (0)