Skip to content

Commit 2d5e486

Browse files
style: fix yarn lint
1 parent c1d7e4d commit 2d5e486

3 files changed

Lines changed: 34 additions & 33 deletions

File tree

lambdas/functions/control-plane/src/aws/runners.test.ts

Lines changed: 28 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1108,10 +1108,12 @@ describe('create runner with useDedicatedHost', () => {
11081108
it('uses CreateFleet when useDedicatedHost is false', async () => {
11091109
mockEC2Client.on(CreateFleetCommand).resolves({ Instances: [{ InstanceIds: ['i-fleet-1'] }] });
11101110

1111-
const result = await createRunner(createRunnerConfig({
1112-
...dedicatedHostRunnerConfig,
1113-
useDedicatedHost: false,
1114-
}));
1111+
const result = await createRunner(
1112+
createRunnerConfig({
1113+
...dedicatedHostRunnerConfig,
1114+
useDedicatedHost: false,
1115+
}),
1116+
);
11151117

11161118
expect(result).toEqual(['i-fleet-1']);
11171119
expect(mockEC2Client).toHaveReceivedCommand(CreateFleetCommand);
@@ -1121,10 +1123,12 @@ describe('create runner with useDedicatedHost', () => {
11211123
it('uses CreateFleet when useDedicatedHost is undefined', async () => {
11221124
mockEC2Client.on(CreateFleetCommand).resolves({ Instances: [{ InstanceIds: ['i-fleet-1'] }] });
11231125

1124-
const result = await createRunner(createRunnerConfig({
1125-
...dedicatedHostRunnerConfig,
1126-
useDedicatedHost: undefined,
1127-
}));
1126+
const result = await createRunner(
1127+
createRunnerConfig({
1128+
...dedicatedHostRunnerConfig,
1129+
useDedicatedHost: undefined,
1130+
}),
1131+
);
11281132

11291133
expect(result).toEqual(['i-fleet-1']);
11301134
expect(mockEC2Client).toHaveReceivedCommand(CreateFleetCommand);
@@ -1211,28 +1215,28 @@ describe('create runner with useDedicatedHost', () => {
12111215

12121216
it('throws error when spot is used with dedicated host', async () => {
12131217
await expect(
1214-
createRunner(createRunnerConfig({
1215-
...dedicatedHostRunnerConfig,
1216-
capacityType: 'spot',
1217-
})),
1218+
createRunner(
1219+
createRunnerConfig({
1220+
...dedicatedHostRunnerConfig,
1221+
capacityType: 'spot',
1222+
}),
1223+
),
12181224
).rejects.toThrow('Spot instances are not supported with RunInstances');
12191225
expect(mockEC2Client).not.toHaveReceivedCommand(RunInstancesCommand);
12201226
});
12211227

12221228
it('throws error when RunInstances returns no instances', async () => {
12231229
mockEC2Client.on(RunInstancesCommand).resolves({ Instances: [] });
12241230

1225-
await expect(
1226-
createRunner(createRunnerConfig(dedicatedHostRunnerConfig)),
1227-
).rejects.toThrow('RunInstances returned no instances for dedicated host.');
1231+
await expect(createRunner(createRunnerConfig(dedicatedHostRunnerConfig))).rejects.toThrow(
1232+
'RunInstances returned no instances for dedicated host.',
1233+
);
12281234
});
12291235

12301236
it('throws error when RunInstances fails', async () => {
12311237
mockEC2Client.on(RunInstancesCommand).rejects(new Error('EC2 error'));
12321238

1233-
await expect(
1234-
createRunner(createRunnerConfig(dedicatedHostRunnerConfig)),
1235-
).rejects.toThrow('EC2 error');
1239+
await expect(createRunner(createRunnerConfig(dedicatedHostRunnerConfig))).rejects.toThrow('EC2 error');
12361240
});
12371241

12381242
it('uses ami id override from ssm parameter', async () => {
@@ -1243,10 +1247,12 @@ describe('create runner with useDedicatedHost', () => {
12431247
};
12441248
mockSSMClient.on(GetParameterCommand).resolves(paramValue);
12451249

1246-
await createRunner(createRunnerConfig({
1247-
...dedicatedHostRunnerConfig,
1248-
amiIdSsmParameterName: 'my-ami-id-param',
1249-
}));
1250+
await createRunner(
1251+
createRunnerConfig({
1252+
...dedicatedHostRunnerConfig,
1253+
amiIdSsmParameterName: 'my-ami-id-param',
1254+
}),
1255+
);
12501256

12511257
expect(mockEC2Client).toHaveReceivedCommandWith(RunInstancesCommand, {
12521258
LaunchTemplate: {

lambdas/functions/control-plane/src/aws/runners.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -335,7 +335,9 @@ async function createInstancesWithRunInstances(
335335

336336
try {
337337
if (runnerParameters.ec2instanceCriteria.targetCapacityType === 'spot') {
338-
throw new Error('Spot instances are not supported with RunInstances. Please set targetCapacityType to on-demand for dedicated hosts.');
338+
throw new Error(
339+
'Spot instances are not supported with RunInstances. Please set targetCapacityType to on-demand for dedicated hosts.',
340+
);
339341
}
340342

341343
const instanceType = runnerParameters.ec2instanceCriteria.instanceTypes[0] as _InstanceType;
@@ -376,7 +378,6 @@ async function createInstancesWithRunInstances(
376378
}
377379
}
378380

379-
380381
// If launchTime is undefined, this will return false
381382
export function bootTimeExceeded(ec2Runner: { launchTime?: Date }): boolean {
382383
const runnerBootTimeInMinutes = process.env.RUNNER_BOOT_TIME_IN_MINUTES;

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

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3099,25 +3099,19 @@ describe('useDedicatedHost', () => {
30993099
it('defaults to false when USE_DEDICATED_HOST env var is not set', async () => {
31003100
delete process.env.USE_DEDICATED_HOST;
31013101
await scaleUpModule.scaleUp(TEST_DATA);
3102-
expect(createRunner).toHaveBeenCalledWith(
3103-
expect.objectContaining({ useDedicatedHost: false }),
3104-
);
3102+
expect(createRunner).toHaveBeenCalledWith(expect.objectContaining({ useDedicatedHost: false }));
31053103
});
31063104

31073105
it('is true when USE_DEDICATED_HOST is "true"', async () => {
31083106
process.env.USE_DEDICATED_HOST = 'true';
31093107
await scaleUpModule.scaleUp(TEST_DATA);
3110-
expect(createRunner).toHaveBeenCalledWith(
3111-
expect.objectContaining({ useDedicatedHost: true }),
3112-
);
3108+
expect(createRunner).toHaveBeenCalledWith(expect.objectContaining({ useDedicatedHost: true }));
31133109
});
31143110

31153111
it('is false when USE_DEDICATED_HOST is "false"', async () => {
31163112
process.env.USE_DEDICATED_HOST = 'false';
31173113
await scaleUpModule.scaleUp(TEST_DATA);
3118-
expect(createRunner).toHaveBeenCalledWith(
3119-
expect.objectContaining({ useDedicatedHost: false }),
3120-
);
3114+
expect(createRunner).toHaveBeenCalledWith(expect.objectContaining({ useDedicatedHost: false }));
31213115
});
31223116
});
31233117

0 commit comments

Comments
 (0)