test: align runner tests with provider modules (github-aws-runners#5232)

## Description

Align test-file ownership with the runner-provider module layout
introduced in github-aws-runners#5203 and requested in github-aws-runners#5230.

This PR only renames or relocates existing tests. Provider-neutral tests
remain beside the shared orchestration modules, while EC2-specific tests
move beside their EC2 implementations.

I plan to create following PR to reduced some duplicated tests. I didn't
change in this PR to prevent a hard review

### Test cases moved or renamed

- Renamed `control-plane/src/aws/runners.test.ts` to
`ec2-runners.test.ts`.
- Renamed `webhook/src/runners/dynamic-labels-policy.test.ts` to
`ec2-dynamic-labels-policy.test.ts`.
- Moved 19 existing `canRunJob` test declarations from
`dispatch.test.ts` to `labels.test.ts`.
- Moved four existing AWS dynamic-label test declarations from
`dispatch.test.ts` to `aws-dynamic-labels.test.ts`.
- Moved three existing EC2 dynamic-label test declarations from
`dispatch.test.ts` to `ec2-dynamic-labels.test.ts`; 18 dispatch
integration tests remain in `dispatch.test.ts`.
- Moved four existing EC2 pool-size test declarations from
`pool.test.ts` to `ec2-pool.test.ts`; 17 provider-neutral pool tests
remain in `pool.test.ts`.
- Moved 20 existing EC2 scale-down test declarations from
`scale-down.test.ts` to `ec2-scale-down.test.ts`; four provider-neutral
eviction-order tests remain in `scale-down.test.ts`.
- Moved 251 existing EC2 scale-up test declarations from
`scale-up.test.ts` to `ec2-scale-up.test.ts`; the provider-selection
test remains in `scale-up.test.ts`.

### Test cases created

None.

### Duplicated test cases merged

None.

### Test cases deleted

None. The test titles and coverage from `origin/main` are preserved.

## Test Plan

- Renamed EC2 suites: 85 tests passed.
- Webhook runner suites: 44 tests passed.
- Pool suites: 21 tests passed.
- Scale-down suites: 130 tests passed.
- Scale-up suites: 254 tests passed.
- ESLint passed for every affected test file.
- `git diff --check` passed.
- Repository pre-commit hooks passed for all five commits.

## Related Issues

Closes github-aws-runners#5230

Author: edersonbrilhante

…ns/control-plane/src/aws/runners.test.ts …ontrol-plane/src/aws/ec2-runners.test.tslambdas/functions/control-plane/src/aws/runners.test.ts renamed to lambdas/functions/control-plane/src/aws/ec2-runners.test.ts lambdas/functions/control-plane/src/aws/runners.test.ts renamed to lambdas/functions/control-plane/src/aws/ec2-runners.test.ts

@@ -0,0 +1,49 @@
+import { bootTimeExceeded } from '../aws/ec2-runners';
+import type { RunnerList } from '../aws/ec2-runners.d';
+import { calculateEc2PoolSize } from './ec2-pool';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+
+vi.mock('../aws/ec2-runners', () => ({
+  bootTimeExceeded: vi.fn(),
+}));
+
+const mockBootTimeExceeded = vi.mocked(bootTimeExceeded);
+
+describe('calculateEc2PoolSize', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('counts registered online idle runners', () => {
+    const runners: RunnerList[] = [{ instanceId: 'i-idle' }];
+    const runnerStatus = new Map([['i-idle', { busy: false, status: 'online' }]]);
+
+    expect(calculateEc2PoolSize(runners, runnerStatus)).toBe(1);
+    expect(mockBootTimeExceeded).not.toHaveBeenCalled();
+  });
+
+  it('does not count registered busy or offline runners', () => {
+    const runners: RunnerList[] = [{ instanceId: 'i-busy' }, { instanceId: 'i-offline' }];
+    const runnerStatus = new Map([
+      ['i-busy', { busy: true, status: 'online' }],
+      ['i-offline', { busy: false, status: 'offline' }],
+    ]);
+
+    expect(calculateEc2PoolSize(runners, runnerStatus)).toBe(0);
+    expect(mockBootTimeExceeded).not.toHaveBeenCalled();
+  });
+
+  it('counts unregistered runners that are still booting', () => {
+    const runners: RunnerList[] = [{ instanceId: 'i-booting' }];
+    mockBootTimeExceeded.mockReturnValue(false);
+
+    expect(calculateEc2PoolSize(runners, new Map())).toBe(1);
+  });
+
+  it('does not count unregistered runners whose boot time expired', () => {
+    const runners: RunnerList[] = [{ instanceId: 'i-expired' }];
+    mockBootTimeExceeded.mockReturnValue(true);
+
+    expect(calculateEc2PoolSize(runners, new Map())).toBe(0);
+  });
+});

lambdas/functions/control-plane/src/pool/ec2-pool.test.ts

@@ -2,13 +2,11 @@ import { Octokit } from '@octokit/rest';
 import moment from 'moment-timezone';
 import * as nock from 'nock';
 
-import { bootTimeExceeded, listEC2Runners } from '../aws/ec2-runners';
-import type { RunnerList } from '../aws/ec2-runners.d';
+import { listEC2Runners } from '../aws/ec2-runners';
 import * as ghAuth from '../github/auth';
 import { createRunners } from '../scale-runners/ec2';
 import { getGitHubEnterpriseApiUrl } from '../scale-runners/github-runner';
 import { adjust } from './pool';
-import { calculateEc2PoolSize } from './ec2-pool';
 import { describe, it, expect, beforeEach, vi, MockedClass } from 'vitest';
 
 const mockOctokit = {
@@ -57,7 +55,6 @@ const mockedAppAuth = vi.mocked(ghAuth.createGithubAppAuth);
 const mockedInstallationAuth = vi.mocked(ghAuth.createGithubInstallationAuth);
 const mockCreateClient = vi.mocked(ghAuth.createOctokitClient);
 const mockListRunners = vi.mocked(listEC2Runners);
-const mockBootTimeExceeded = vi.mocked(bootTimeExceeded);
 
 const cleanEnv = process.env;
 
@@ -496,42 +493,3 @@ describe('Test simple pool.', () => {
     });
   });
 });
-
-describe('calculateEc2PoolSize', () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
-  it('counts registered online idle runners', () => {
-    const runners: RunnerList[] = [{ instanceId: 'i-idle' }];
-    const runnerStatus = new Map([['i-idle', { busy: false, status: 'online' }]]);
-
-    expect(calculateEc2PoolSize(runners, runnerStatus)).toBe(1);
-    expect(mockBootTimeExceeded).not.toHaveBeenCalled();
-  });
-
-  it('does not count registered busy or offline runners', () => {
-    const runners: RunnerList[] = [{ instanceId: 'i-busy' }, { instanceId: 'i-offline' }];
-    const runnerStatus = new Map([
-      ['i-busy', { busy: true, status: 'online' }],
-      ['i-offline', { busy: false, status: 'offline' }],
-    ]);
-
-    expect(calculateEc2PoolSize(runners, runnerStatus)).toBe(0);
-    expect(mockBootTimeExceeded).not.toHaveBeenCalled();
-  });
-
-  it('counts unregistered runners that are still booting', () => {
-    const runners: RunnerList[] = [{ instanceId: 'i-booting' }];
-    mockBootTimeExceeded.mockReturnValue(false);
-
-    expect(calculateEc2PoolSize(runners, new Map())).toBe(1);
-  });
-
-  it('does not count unregistered runners whose boot time expired', () => {
-    const runners: RunnerList[] = [{ instanceId: 'i-expired' }];
-    mockBootTimeExceeded.mockReturnValue(true);
-
-    expect(calculateEc2PoolSize(runners, new Map())).toBe(0);
-  });
-});