Skip to content

Commit

Permalink
Fix linter complaints
Browse files Browse the repository at this point in the history
  • Loading branch information
Link- authored Dec 2, 2024
1 parent 87171e2 commit 7ad18fd
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 21 deletions.
30 changes: 15 additions & 15 deletions packages/cache/__tests__/saveCacheV2.test.ts
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
import * as core from '@actions/core'
import * as path from 'path'
import { saveCache } from '../src/cache'
import {saveCache} from '../src/cache'
import * as cacheUtils from '../src/internal/cacheUtils'
import { CacheFilename, CompressionMethod } from '../src/internal/constants'
import {CacheFilename, CompressionMethod} from '../src/internal/constants'
import * as config from '../src/internal/config'
import * as tar from '../src/internal/tar'
import { CacheServiceClientJSON } from '../src/generated/results/api/v1/cache.twirp'
import {CacheServiceClientJSON} from '../src/generated/results/api/v1/cache.twirp'
import * as cacheHttpClient from '../src/internal/cacheHttpClient'
import { UploadOptions } from '../src/options'
import {UploadOptions} from '../src/options'

let logDebugMock: jest.SpyInstance

Expand All @@ -27,11 +27,11 @@ jest.mock('@azure/storage-blob', () => ({

beforeAll(() => {
process.env['ACTIONS_RUNTIME_TOKEN'] = 'token'
jest.spyOn(console, 'log').mockImplementation(() => { })
jest.spyOn(core, 'debug').mockImplementation(() => { })
jest.spyOn(core, 'info').mockImplementation(() => { })
jest.spyOn(core, 'warning').mockImplementation(() => { })
jest.spyOn(core, 'error').mockImplementation(() => { })
jest.spyOn(console, 'log').mockImplementation(() => {})
jest.spyOn(core, 'debug').mockImplementation(() => {})
jest.spyOn(core, 'info').mockImplementation(() => {})
jest.spyOn(core, 'warning').mockImplementation(() => {})
jest.spyOn(core, 'error').mockImplementation(() => {})
jest.spyOn(cacheUtils, 'resolvePaths').mockImplementation(async filePaths => {
return filePaths.map(x => path.resolve(x))
})
Expand Down Expand Up @@ -99,7 +99,7 @@ test('create cache entry failure', async () => {

const createCacheEntryMock = jest
.spyOn(CacheServiceClientJSON.prototype, 'CreateCacheEntry')
.mockReturnValue(Promise.resolve({ ok: false, signedUploadUrl: '' }))
.mockReturnValue(Promise.resolve({ok: false, signedUploadUrl: ''}))

const createTarMock = jest.spyOn(tar, 'createTar')
const finalizeCacheEntryMock = jest.spyOn(
Expand Down Expand Up @@ -149,7 +149,7 @@ test('save cache fails if a signedUploadURL was not passed', async () => {
const createCacheEntryMock = jest
.spyOn(CacheServiceClientJSON.prototype, 'CreateCacheEntry')
.mockReturnValue(
Promise.resolve({ ok: true, signedUploadUrl: signedUploadURL })
Promise.resolve({ok: true, signedUploadUrl: signedUploadURL})
)

const createTarMock = jest.spyOn(tar, 'createTar')
Expand Down Expand Up @@ -207,7 +207,7 @@ test('finalize save cache failure', async () => {
const createCacheEntryMock = jest
.spyOn(CacheServiceClientJSON.prototype, 'CreateCacheEntry')
.mockReturnValue(
Promise.resolve({ ok: true, signedUploadUrl: signedUploadURL })
Promise.resolve({ok: true, signedUploadUrl: signedUploadURL})
)

const createTarMock = jest.spyOn(tar, 'createTar')
Expand All @@ -227,7 +227,7 @@ test('finalize save cache failure', async () => {

const finalizeCacheEntryMock = jest
.spyOn(CacheServiceClientJSON.prototype, 'FinalizeCacheEntryUpload')
.mockReturnValue(Promise.resolve({ ok: false, entryId: '' }))
.mockReturnValue(Promise.resolve({ok: false, entryId: ''}))

const cacheId = await saveCache([paths], key, options)

Expand Down Expand Up @@ -286,7 +286,7 @@ test('save with valid inputs uploads a cache', async () => {
jest
.spyOn(CacheServiceClientJSON.prototype, 'CreateCacheEntry')
.mockReturnValue(
Promise.resolve({ ok: true, signedUploadUrl: signedUploadURL })
Promise.resolve({ok: true, signedUploadUrl: signedUploadURL})
)

const saveCacheMock = jest.spyOn(cacheHttpClient, 'saveCache')
Expand All @@ -299,7 +299,7 @@ test('save with valid inputs uploads a cache', async () => {

const finalizeCacheEntryMock = jest
.spyOn(CacheServiceClientJSON.prototype, 'FinalizeCacheEntryUpload')
.mockReturnValue(Promise.resolve({ ok: true, entryId: cacheId.toString() }))
.mockReturnValue(Promise.resolve({ok: true, entryId: cacheId.toString()}))

const expectedCacheId = await saveCache([paths], key)

Expand Down
12 changes: 6 additions & 6 deletions packages/cache/__tests__/uploadUtils.test.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import * as uploadUtils from '../src/internal/uploadUtils'
import { TransferProgressEvent } from '@azure/ms-rest-js'
import {TransferProgressEvent} from '@azure/ms-rest-js'

test('upload progress tracked correctly', () => {
const progress = new uploadUtils.UploadProgress(1000)
Expand All @@ -11,7 +11,7 @@ test('upload progress tracked correctly', () => {
expect(progress.getTransferredBytes()).toBe(0)
expect(progress.isDone()).toBe(false)

progress.onProgress()({ loadedBytes: 0 } as TransferProgressEvent)
progress.onProgress()({loadedBytes: 0} as TransferProgressEvent)

expect(progress.contentLength).toBe(1000)
expect(progress.sentBytes).toBe(0)
Expand All @@ -20,7 +20,7 @@ test('upload progress tracked correctly', () => {
expect(progress.getTransferredBytes()).toBe(0)
expect(progress.isDone()).toBe(false)

progress.onProgress()({ loadedBytes: 250 } as TransferProgressEvent)
progress.onProgress()({loadedBytes: 250} as TransferProgressEvent)

expect(progress.contentLength).toBe(1000)
expect(progress.sentBytes).toBe(250)
Expand All @@ -29,7 +29,7 @@ test('upload progress tracked correctly', () => {
expect(progress.getTransferredBytes()).toBe(250)
expect(progress.isDone()).toBe(false)

progress.onProgress()({ loadedBytes: 500 } as TransferProgressEvent)
progress.onProgress()({loadedBytes: 500} as TransferProgressEvent)

expect(progress.contentLength).toBe(1000)
expect(progress.sentBytes).toBe(500)
Expand All @@ -38,7 +38,7 @@ test('upload progress tracked correctly', () => {
expect(progress.getTransferredBytes()).toBe(500)
expect(progress.isDone()).toBe(false)

progress.onProgress()({ loadedBytes: 750 } as TransferProgressEvent)
progress.onProgress()({loadedBytes: 750} as TransferProgressEvent)

expect(progress.contentLength).toBe(1000)
expect(progress.sentBytes).toBe(750)
Expand All @@ -47,7 +47,7 @@ test('upload progress tracked correctly', () => {
expect(progress.getTransferredBytes()).toBe(750)
expect(progress.isDone()).toBe(false)

progress.onProgress()({ loadedBytes: 1000 } as TransferProgressEvent)
progress.onProgress()({loadedBytes: 1000} as TransferProgressEvent)

expect(progress.contentLength).toBe(1000)
expect(progress.sentBytes).toBe(1000)
Expand Down

0 comments on commit 7ad18fd

Please sign in to comment.