Prefer strict camel case class names

github · Sep 20, 2021 · 750e512 · 750e512
1 parent 906c592
commit 750e512
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 19 deletions.
diff --git a/__tests__/api_client.test.ts b/__tests__/api_client.test.ts
@@ -1,10 +1,10 @@
-import {APIClient} from '../src/api-client'
+import {ApiClient} from '../src/api-client'
 
-describe('APIClient', () => {
+describe('ApiClient', () => {
   const mockAxios: any = {
     get: jest.fn()
   }
-  const api = new APIClient(mockAxios, {
+  const api = new ApiClient(mockAxios, {
     jobId: 1,
     jobToken: 'xxx',
     credentialsToken: 'yyy',

diff --git a/__tests__/main.test.ts b/__tests__/main.test.ts
@@ -1,6 +1,6 @@
 import * as core from '@actions/core'
 import {Context} from '@actions/github/lib/context'
-import {APIClient} from '../src/api-client'
+import {ApiClient} from '../src/api-client'
 import {Updater} from '../src/updater'
 import {ImageService} from '../src/image-service'
 import * as inputs from '../src/inputs'
@@ -21,10 +21,10 @@ describe('run', () => {
 
   beforeEach(async () => {
     markJobAsProcessedSpy = jest.spyOn(
-      APIClient.prototype,
+      ApiClient.prototype,
       'markJobAsProcessed'
     )
-    reportJobErrorSpy = jest.spyOn(APIClient.prototype, 'reportJobError')
+    reportJobErrorSpy = jest.spyOn(ApiClient.prototype, 'reportJobError')
 
     jest.spyOn(core, 'info').mockImplementation(jest.fn())
     jest.spyOn(core, 'setFailed').mockImplementation(jest.fn())
@@ -114,7 +114,7 @@ describe('run', () => {
   describe('when there is an error retrieving job details from DependabotAPI', () => {
     beforeEach(() => {
       jest
-        .spyOn(APIClient.prototype, 'getJobDetails')
+        .spyOn(ApiClient.prototype, 'getJobDetails')
         .mockImplementationOnce(
           jest.fn(async () =>
             Promise.reject(new Error('error getting job details'))
@@ -148,7 +148,7 @@ describe('run', () => {
   describe('when there is an error retrieving job credentials from DependabotAPI', () => {
     beforeEach(() => {
       jest
-        .spyOn(APIClient.prototype, 'getCredentials')
+        .spyOn(ApiClient.prototype, 'getCredentials')
         .mockImplementationOnce(
           jest.fn(async () =>
             Promise.reject(new Error('error getting credentials'))

diff --git a/__tests__/updater-integration.test.ts b/__tests__/updater-integration.test.ts
@@ -1,6 +1,6 @@
 import axios from 'axios'
 
-import {APIClient, JobParameters} from '../src/api-client'
+import {ApiClient, JobParameters} from '../src/api-client'
 import {ImageService} from '../src/image-service'
 import {UPDATER_IMAGE_NAME, PROXY_IMAGE_NAME} from '../src/main'
 import {Updater} from '../src/updater'
@@ -27,7 +27,7 @@ describe('Updater', () => {
   )
 
   const client = axios.create({baseURL: dependabotApiUrl})
-  const apiClient = new APIClient(client, params)
+  const apiClient = new ApiClient(client, params)
 
   beforeAll(async () => {
     // Skip the test when we haven't preloaded the updater image

diff --git a/__tests__/updater.test.ts b/__tests__/updater.test.ts
@@ -2,7 +2,7 @@ import {UPDATER_IMAGE_NAME, PROXY_IMAGE_NAME} from '../src/main'
 import {Updater} from '../src/updater'
 
 describe('Updater', () => {
-  const mockAPIClient: any = {
+  const mockApiClient: any = {
     getJobDetails: jest.fn(),
     getCredentials: jest.fn(),
     params: {
@@ -26,13 +26,13 @@ describe('Updater', () => {
   const updater = new Updater(
     UPDATER_IMAGE_NAME,
     PROXY_IMAGE_NAME,
-    mockAPIClient,
+    mockApiClient,
     mockJobDetails,
     []
   )
 
   it('should fetch job details', async () => {
-    mockAPIClient.getJobDetails.mockImplementation(() => {
+    mockApiClient.getJobDetails.mockImplementation(() => {
       throw new Error('kaboom')
     })
     updater.runUpdater()

diff --git a/src/api-client.ts b/src/api-client.ts
@@ -33,7 +33,7 @@ export type Credential = {
   token?: string
 }
 
-export class APIClient {
+export class ApiClient {
   constructor(
     private readonly client: AxiosInstance,
     readonly params: JobParameters

diff --git a/src/main.ts b/src/main.ts
@@ -4,7 +4,7 @@ import {Context} from '@actions/github/lib/context'
 import {getJobParameters} from './inputs'
 import {ImageService} from './image-service'
 import {Updater} from './updater'
-import {APIClient} from './api-client'
+import {ApiClient} from './api-client'
 import axios from 'axios'
 
 export const UPDATER_IMAGE_NAME =
@@ -34,7 +34,7 @@ export async function run(context: Context): Promise<void> {
     core.setSecret(params.credentialsToken)
 
     const client = axios.create({baseURL: params.dependabotApiUrl})
-    const apiClient = new APIClient(client, params)
+    const apiClient = new ApiClient(client, params)
 
     try {
       core.info('Fetching job details')
@@ -86,7 +86,7 @@ export async function run(context: Context): Promise<void> {
 }
 
 async function failJob(
-  apiClient: APIClient,
+  apiClient: ApiClient,
   error: Error,
   errorType = DependabotErrorType.Unknown
 ): Promise<void> {

diff --git a/src/updater.ts b/src/updater.ts
@@ -2,7 +2,7 @@ import * as core from '@actions/core'
 import Docker, {Container} from 'dockerode'
 import path from 'path'
 import fs from 'fs'
-import {JobDetails, APIClient, Credential} from './api-client'
+import {JobDetails, ApiClient, Credential} from './api-client'
 import {ContainerService} from './container-service'
 import {base64DecodeDependencyFile} from './utils'
 import {DependencyFile, FetchedFiles, FileUpdaterInput} from './config-types'
@@ -22,7 +22,7 @@ export class Updater {
   constructor(
     private readonly updaterImage: string,
     private readonly proxyImage: string,
-    private readonly apiClient: APIClient,
+    private readonly apiClient: ApiClient,
     private readonly details: JobDetails,
     private readonly credentials: Credential[]
   ) {