Skip to content

Commit 9e6cdd5

Browse files
style: fix yarn lint
1 parent c807ffe commit 9e6cdd5

File tree

3 files changed

+34
-33
lines changed

3 files changed

+34
-33
lines changed

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

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

849-
const result = await createRunner(createRunnerConfig({
850-
...dedicatedHostRunnerConfig,
851-
useDedicatedHost: false,
852-
}));
849+
const result = await createRunner(
850+
createRunnerConfig({
851+
...dedicatedHostRunnerConfig,
852+
useDedicatedHost: false,
853+
}),
854+
);
853855

854856
expect(result).toEqual(['i-fleet-1']);
855857
expect(mockEC2Client).toHaveReceivedCommand(CreateFleetCommand);
@@ -859,10 +861,12 @@ describe('create runner with useDedicatedHost', () => {
859861
it('uses CreateFleet when useDedicatedHost is undefined', async () => {
860862
mockEC2Client.on(CreateFleetCommand).resolves({ Instances: [{ InstanceIds: ['i-fleet-1'] }] });
861863

862-
const result = await createRunner(createRunnerConfig({
863-
...dedicatedHostRunnerConfig,
864-
useDedicatedHost: undefined,
865-
}));
864+
const result = await createRunner(
865+
createRunnerConfig({
866+
...dedicatedHostRunnerConfig,
867+
useDedicatedHost: undefined,
868+
}),
869+
);
866870

867871
expect(result).toEqual(['i-fleet-1']);
868872
expect(mockEC2Client).toHaveReceivedCommand(CreateFleetCommand);
@@ -949,28 +953,28 @@ describe('create runner with useDedicatedHost', () => {
949953

950954
it('throws error when spot is used with dedicated host', async () => {
951955
await expect(
952-
createRunner(createRunnerConfig({
953-
...dedicatedHostRunnerConfig,
954-
capacityType: 'spot',
955-
})),
956+
createRunner(
957+
createRunnerConfig({
958+
...dedicatedHostRunnerConfig,
959+
capacityType: 'spot',
960+
}),
961+
),
956962
).rejects.toThrow('Spot instances are not supported with RunInstances');
957963
expect(mockEC2Client).not.toHaveReceivedCommand(RunInstancesCommand);
958964
});
959965

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

963-
await expect(
964-
createRunner(createRunnerConfig(dedicatedHostRunnerConfig)),
965-
).rejects.toThrow('RunInstances returned no instances for dedicated host.');
969+
await expect(createRunner(createRunnerConfig(dedicatedHostRunnerConfig))).rejects.toThrow(
970+
'RunInstances returned no instances for dedicated host.',
971+
);
966972
});
967973

968974
it('throws error when RunInstances fails', async () => {
969975
mockEC2Client.on(RunInstancesCommand).rejects(new Error('EC2 error'));
970976

971-
await expect(
972-
createRunner(createRunnerConfig(dedicatedHostRunnerConfig)),
973-
).rejects.toThrow('EC2 error');
977+
await expect(createRunner(createRunnerConfig(dedicatedHostRunnerConfig))).rejects.toThrow('EC2 error');
974978
});
975979

976980
it('uses ami id override from ssm parameter', async () => {
@@ -981,10 +985,12 @@ describe('create runner with useDedicatedHost', () => {
981985
};
982986
mockSSMClient.on(GetParameterCommand).resolves(paramValue);
983987

984-
await createRunner(createRunnerConfig({
985-
...dedicatedHostRunnerConfig,
986-
amiIdSsmParameterName: 'my-ami-id-param',
987-
}));
988+
await createRunner(
989+
createRunnerConfig({
990+
...dedicatedHostRunnerConfig,
991+
amiIdSsmParameterName: 'my-ami-id-param',
992+
}),
993+
);
988994

989995
expect(mockEC2Client).toHaveReceivedCommandWith(RunInstancesCommand, {
990996
LaunchTemplate: {

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

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

327327
try {
328328
if (runnerParameters.ec2instanceCriteria.targetCapacityType === 'spot') {
329-
throw new Error('Spot instances are not supported with RunInstances. Please set targetCapacityType to on-demand for dedicated hosts.');
329+
throw new Error(
330+
'Spot instances are not supported with RunInstances. Please set targetCapacityType to on-demand for dedicated hosts.',
331+
);
330332
}
331333

332334
const instanceType = runnerParameters.ec2instanceCriteria.instanceTypes[0] as _InstanceType;
@@ -367,7 +369,6 @@ async function createInstancesWithRunInstances(
367369
}
368370
}
369371

370-
371372
// If launchTime is undefined, this will return false
372373
export function bootTimeExceeded(ec2Runner: { launchTime?: Date }): boolean {
373374
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
@@ -2033,25 +2033,19 @@ describe('useDedicatedHost', () => {
20332033
it('defaults to false when USE_DEDICATED_HOST env var is not set', async () => {
20342034
delete process.env.USE_DEDICATED_HOST;
20352035
await scaleUpModule.scaleUp(TEST_DATA);
2036-
expect(createRunner).toHaveBeenCalledWith(
2037-
expect.objectContaining({ useDedicatedHost: false }),
2038-
);
2036+
expect(createRunner).toHaveBeenCalledWith(expect.objectContaining({ useDedicatedHost: false }));
20392037
});
20402038

20412039
it('is true when USE_DEDICATED_HOST is "true"', async () => {
20422040
process.env.USE_DEDICATED_HOST = 'true';
20432041
await scaleUpModule.scaleUp(TEST_DATA);
2044-
expect(createRunner).toHaveBeenCalledWith(
2045-
expect.objectContaining({ useDedicatedHost: true }),
2046-
);
2042+
expect(createRunner).toHaveBeenCalledWith(expect.objectContaining({ useDedicatedHost: true }));
20472043
});
20482044

20492045
it('is false when USE_DEDICATED_HOST is "false"', async () => {
20502046
process.env.USE_DEDICATED_HOST = 'false';
20512047
await scaleUpModule.scaleUp(TEST_DATA);
2052-
expect(createRunner).toHaveBeenCalledWith(
2053-
expect.objectContaining({ useDedicatedHost: false }),
2054-
);
2048+
expect(createRunner).toHaveBeenCalledWith(expect.objectContaining({ useDedicatedHost: false }));
20552049
});
20562050
});
20572051

0 commit comments

Comments
 (0)