Skip to content

Commit

Permalink
refactor(aws): Make datasource names consistent (#30865)
Browse files Browse the repository at this point in the history
  • Loading branch information
zharinov authored Aug 19, 2024
1 parent 2fb7be4 commit 3c1c67a
Show file tree
Hide file tree
Showing 5 changed files with 22 additions and 22 deletions.
8 changes: 4 additions & 4 deletions lib/modules/datasource/api.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { ArtifactoryDatasource } from './artifactory';
import { AwsMachineImageDataSource } from './aws-machine-image';
import { AwsRdsDataSource } from './aws-rds';
import { AwsMachineImageDatasource } from './aws-machine-image';
import { AwsRdsDatasource } from './aws-rds';
import { AzureBicepResourceDatasource } from './azure-bicep-resource';
import { AzurePipelinesTasksDatasource } from './azure-pipelines-tasks';
import { BazelDatasource } from './bazel';
Expand Down Expand Up @@ -68,8 +68,8 @@ const api = new Map<string, DatasourceApi>();
export default api;

api.set(ArtifactoryDatasource.id, new ArtifactoryDatasource());
api.set(AwsMachineImageDataSource.id, new AwsMachineImageDataSource());
api.set(AwsRdsDataSource.id, new AwsRdsDataSource());
api.set(AwsMachineImageDatasource.id, new AwsMachineImageDatasource());
api.set(AwsRdsDatasource.id, new AwsRdsDatasource());
api.set(AzureBicepResourceDatasource.id, new AzureBicepResourceDatasource());
api.set(AzurePipelinesTasksDatasource.id, new AzurePipelinesTasksDatasource());
api.set(BazelDatasource.id, new BazelDatasource());
Expand Down
12 changes: 6 additions & 6 deletions lib/modules/datasource/aws-machine-image/index.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,9 @@ import {
} from '@aws-sdk/client-ec2';
import { mockClient } from 'aws-sdk-client-mock';
import { getDigest, getPkgReleases } from '..';
import { AwsMachineImageDataSource } from '.';
import { AwsMachineImageDatasource } from '.';

const datasource = AwsMachineImageDataSource.id;
const datasource = AwsMachineImageDatasource.id;
const ec2Mock = mockClient(EC2Client);

/**
Expand Down Expand Up @@ -140,7 +140,7 @@ describe('modules/datasource/aws-machine-image/index', () => {
describe('getSortedAwsMachineImages()', () => {
it('with 3 returned images', async () => {
mockDescribeImagesCommand(mock3Images);
const ec2DataSource = new AwsMachineImageDataSource();
const ec2DataSource = new AwsMachineImageDatasource();
const res = await ec2DataSource.getSortedAwsMachineImages(
'[{"Name":"owner-id","Values":["602401143452"]},{"Name":"name","Values":["3images"]}]',
);
Expand All @@ -151,7 +151,7 @@ describe('modules/datasource/aws-machine-image/index', () => {

it('with 1 returned image', async () => {
mockDescribeImagesCommand(mock1Image);
const ec2DataSource = new AwsMachineImageDataSource();
const ec2DataSource = new AwsMachineImageDatasource();
const res = await ec2DataSource.getSortedAwsMachineImages(
'[{"Name":"owner-id","Values":["602401143452"]},{"Name":"name","Values":["1image"]}]',
);
Expand All @@ -162,7 +162,7 @@ describe('modules/datasource/aws-machine-image/index', () => {

it('without returned images', async () => {
mockDescribeImagesCommand(mockEmpty);
const ec2DataSource = new AwsMachineImageDataSource();
const ec2DataSource = new AwsMachineImageDatasource();
const res = await ec2DataSource.getSortedAwsMachineImages(
'[{"Name":"owner-id","Values":["602401143452"]},{"Name":"name","Values":["noiamge"]}]',
);
Expand Down Expand Up @@ -300,7 +300,7 @@ describe('modules/datasource/aws-machine-image/index', () => {
});

describe('loadConfig()', () => {
const ec2DataSource = new AwsMachineImageDataSource();
const ec2DataSource = new AwsMachineImageDatasource();

it('loads filters without aws config', () => {
const res = ec2DataSource.loadConfig(
Expand Down
10 changes: 5 additions & 5 deletions lib/modules/datasource/aws-machine-image/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import { Datasource } from '../datasource';
import type { GetReleasesConfig, ReleaseResult } from '../types';
import type { AwsClientConfig, ParsedConfig } from './types';

export class AwsMachineImageDataSource extends Datasource {
export class AwsMachineImageDatasource extends Datasource {
static readonly id = 'aws-machine-image';

override readonly defaultVersioning = amazonMachineImageVersioning.id;
Expand Down Expand Up @@ -42,7 +42,7 @@ export class AwsMachineImageDataSource extends Datasource {
private readonly now: number;

constructor() {
super(AwsMachineImageDataSource.id);
super(AwsMachineImageDatasource.id);
this.now = Date.now();
}

Expand Down Expand Up @@ -81,7 +81,7 @@ export class AwsMachineImageDataSource extends Datasource {
}

@cache({
namespace: `datasource-${AwsMachineImageDataSource.id}`,
namespace: `datasource-${AwsMachineImageDatasource.id}`,
key: (serializedAmiFilter: string) =>
`getSortedAwsMachineImages:${serializedAmiFilter}`,
})
Expand All @@ -106,7 +106,7 @@ export class AwsMachineImageDataSource extends Datasource {
}

@cache({
namespace: `datasource-${AwsMachineImageDataSource.id}`,
namespace: `datasource-${AwsMachineImageDatasource.id}`,
key: ({ packageName }: GetReleasesConfig, newValue: string) =>
`getDigest:${packageName}:${newValue ?? ''}`,
})
Expand Down Expand Up @@ -136,7 +136,7 @@ export class AwsMachineImageDataSource extends Datasource {
}

@cache({
namespace: `datasource-${AwsMachineImageDataSource.id}`,
namespace: `datasource-${AwsMachineImageDatasource.id}`,
key: ({ packageName }: GetReleasesConfig) => `getReleases:${packageName}`,
})
async getReleases({
Expand Down
8 changes: 4 additions & 4 deletions lib/modules/datasource/aws-rds/index.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import {
} from '@aws-sdk/client-rds';
import { mockClient } from 'aws-sdk-client-mock';
import { getPkgReleases } from '..';
import { AwsRdsDataSource } from '.';
import { AwsRdsDatasource } from '.';

const rdsMock = mockClient(RDSClient);

Expand Down Expand Up @@ -104,7 +104,7 @@ describe('modules/datasource/aws-rds/index', () => {
$metadata: {},
});
const res = await getPkgReleases({
datasource: AwsRdsDataSource.id,
datasource: AwsRdsDatasource.id,
packageName: '[{"Name":"engine","Values":["mysql"]}]',
});
expect(res).toBeNull();
Expand All @@ -116,7 +116,7 @@ describe('modules/datasource/aws-rds/index', () => {
DBEngineVersions: [version2],
});
const res = await getPkgReleases({
datasource: AwsRdsDataSource.id,
datasource: AwsRdsDatasource.id,
packageName: '[{"Name":"engine","Values":["mysql"]}]',
});
expect(res).toEqual({
Expand All @@ -135,7 +135,7 @@ describe('modules/datasource/aws-rds/index', () => {
DBEngineVersions: [version1, version2, version3],
});
const res = await getPkgReleases({
datasource: AwsRdsDataSource.id,
datasource: AwsRdsDatasource.id,
packageName: '[{"Name":"engine","Values":["mysql"]}]',
});
expect(res).toEqual({
Expand Down
6 changes: 3 additions & 3 deletions lib/modules/datasource/aws-rds/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,20 +7,20 @@ import { Lazy } from '../../../util/lazy';
import { Datasource } from '../datasource';
import type { GetReleasesConfig, ReleaseResult } from '../types';

export class AwsRdsDataSource extends Datasource {
export class AwsRdsDatasource extends Datasource {
static readonly id = 'aws-rds';

override readonly caching = true;

private readonly rds: Lazy<RDSClient>;

constructor() {
super(AwsRdsDataSource.id);
super(AwsRdsDatasource.id);
this.rds = new Lazy(() => new RDSClient({}));
}

@cache({
namespace: `datasource-${AwsRdsDataSource.id}`,
namespace: `datasource-${AwsRdsDatasource.id}`,
key: ({ packageName }: GetReleasesConfig) => `getReleases:${packageName}`,
})
async getReleases({
Expand Down

0 comments on commit 3c1c67a

Please sign in to comment.