Remove the "refresh the session when 401 errors happen"

Shopify · Dec 2, 2024 · 11fd136 · 11fd136
1 parent 4915786
commit 11fd136
Showing 1 changed file with 0 additions and 26 deletions.
diff --git a/packages/cli-kit/src/public/node/themes/api.test.ts b/packages/cli-kit/src/public/node/themes/api.test.ts
@@ -356,7 +356,6 @@ describe('themeDelete', () => {
 describe('request errors', () => {
   test(`returns AbortError when graphql returns user error`, async () => {
     // Given
-
     vi.mocked(adminRequestDoc).mockResolvedValue({
       themeDelete: {
         deletedThemeId: null,
@@ -371,31 +370,6 @@ describe('request errors', () => {
       // Then
     }).rejects.toThrowError(AbortError)
   })
-
-  test(`refresh the session when 401 errors happen`, async () => {
-    // Given
-    const id = 123
-    const assets: AssetParams[] = []
-
-    vi.spyOn(session, 'refresh').mockImplementation(vi.fn())
-    vi.mocked(restRequest)
-      .mockResolvedValueOnce({
-        json: {},
-        status: 401,
-        headers: {},
-      })
-      .mockResolvedValueOnce({
-        json: {},
-        status: 207,
-        headers: {},
-      })
-
-    // When
-    await bulkUploadThemeAssets(id, assets, session)
-
-    // Then
-    expect(session.refresh).toHaveBeenCalledOnce()
-  })
 })
 
 describe('bulkUploadThemeAssets', async () => {