Skip to content

Commit c0137ea

Browse files
committed
chore: fix format
1 parent c392f4b commit c0137ea

File tree

3 files changed

+16
-23
lines changed

3 files changed

+16
-23
lines changed

lambdas/.vscode/settings.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
{
2+
"jest.jestCommandLine": "yarn run test --",
3+
}

lambdas/functions/control-plane/src/lambda.test.ts

Lines changed: 10 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,15 @@
11
import { captureLambdaHandler, logger } from '@aws-github-runner/aws-powertools-util';
22
import { Context, SQSEvent, SQSRecord } from 'aws-lambda';
33

4-
import { addMiddleware, adjustPool, scaleDownHandler, scaleUpHandler, ssmHousekeeper, jobRetryCheck, cleanupOrgRunnersHandler } from './lambda';
4+
import {
5+
addMiddleware,
6+
adjustPool,
7+
scaleDownHandler,
8+
scaleUpHandler,
9+
ssmHousekeeper,
10+
jobRetryCheck,
11+
cleanupOrgRunnersHandler,
12+
} from './lambda';
513
import { adjust } from './pool/pool';
614
import ScaleError from './scale-runners/ScaleError';
715
import { scaleDown } from './scale-runners/scale-down';
@@ -62,32 +70,14 @@ const context: Context = {
6270
},
6371
};
6472

65-
<<<<<<< HEAD
6673
vi.mock('./pool/pool');
6774
vi.mock('./scale-runners/scale-down');
6875
vi.mock('./scale-runners/scale-up');
6976
vi.mock('./scale-runners/ssm-housekeeper');
7077
vi.mock('./scale-runners/job-retry');
78+
vi.mock('./scale-runners/cleanup-org-runners');
7179
vi.mock('@aws-github-runner/aws-powertools-util');
7280
vi.mock('@aws-github-runner/aws-ssm-util');
73-
||||||| parent of b4283485 (feat: add lambda function to cleanup org runners)
74-
jest.mock('./pool/pool');
75-
jest.mock('./scale-runners/scale-down');
76-
jest.mock('./scale-runners/scale-up');
77-
jest.mock('./scale-runners/ssm-housekeeper');
78-
jest.mock('./scale-runners/job-retry');
79-
jest.mock('@aws-github-runner/aws-powertools-util');
80-
jest.mock('@aws-github-runner/aws-ssm-util');
81-
=======
82-
jest.mock('./pool/pool');
83-
jest.mock('./scale-runners/scale-down');
84-
jest.mock('./scale-runners/scale-up');
85-
jest.mock('./scale-runners/ssm-housekeeper');
86-
jest.mock('./scale-runners/job-retry');
87-
jest.mock('./scale-runners/cleanup-org-runners');
88-
jest.mock('@aws-github-runner/aws-powertools-util');
89-
jest.mock('@aws-github-runner/aws-ssm-util');
90-
>>>>>>> b4283485 (feat: add lambda function to cleanup org runners)
9181

9282
describe('Test scale up lambda wrapper.', () => {
9383
it('Do not handle multiple record sets.', async () => {

lambdas/functions/control-plane/src/scale-runners/cleanup-org-runners.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ describe('cleanup-org-runners', () => {
203203
{ id: 1, name: 'runner-1', status: 'offline', labels: [] }, // Empty labels
204204
],
205205
expectedDeletedIds: [1], // Based on actual behavior, it deletes runners with empty labels
206-
}
206+
},
207207
];
208208

209209
for (const testCase of testCases) {
@@ -217,10 +217,10 @@ describe('cleanup-org-runners', () => {
217217

218218
// Verify
219219
expect(mockOctokit.actions.deleteSelfHostedRunnerFromOrg).toHaveBeenCalledTimes(
220-
testCase.expectedDeletedIds.length
220+
testCase.expectedDeletedIds.length,
221221
);
222222

223-
testCase.expectedDeletedIds.forEach(id => {
223+
testCase.expectedDeletedIds.forEach((id) => {
224224
expect(mockOctokit.actions.deleteSelfHostedRunnerFromOrg).toHaveBeenCalledWith({
225225
runner_id: id,
226226
org: 'test-org',

0 commit comments

Comments
 (0)