refactor: move tests from __tests__ to src directory, fix AWS SDK Filter properties

Author: dgokcin

…ns/ami-updater/src/__tests__/ami.test.ts …as/functions/ami-updater/src/ami.test.tslambdas/functions/ami-updater/src/__tests__/ami.test.ts renamed to lambdas/functions/ami-updater/src/ami.test.ts lambdas/functions/ami-updater/src/__tests__/ami.test.ts renamed to lambdas/functions/ami-updater/src/ami.test.ts

@@ -1,6 +1,13 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { EC2Client, DescribeImagesCommand, DescribeLaunchTemplatesCommand, DescribeLaunchTemplateVersionsCommand, CreateLaunchTemplateVersionCommand, ModifyLaunchTemplateCommand } from '@aws-sdk/client-ec2';
-import { AMIManager } from '../ami';
+import {
+  EC2Client,
+  DescribeImagesCommand,
+  DescribeLaunchTemplatesCommand,
+  DescribeLaunchTemplateVersionsCommand,
+  CreateLaunchTemplateVersionCommand,
+  ModifyLaunchTemplateCommand,
+} from '@aws-sdk/client-ec2';
+import { AMIManager } from './ami';
 
 vi.mock('@aws-sdk/client-ec2');
 vi.mock('../../shared/aws-powertools-util', () => ({
@@ -34,7 +41,7 @@ describe('AMIManager', () => {
 
       const config = {
         owners: ['self'],
-        filters: [{ name: 'tag:Environment', values: ['prod'] }],
+        filters: [{ Name: 'tag:Environment', Values: ['prod'] }],
       };
 
       const result = await amiManager.getLatestAmi(config);
@@ -47,7 +54,7 @@ describe('AMIManager', () => {
 
       const config = {
         owners: ['self'],
-        filters: [{ name: 'tag:Environment', values: ['prod'] }],
+        filters: [{ Name: 'tag:Environment', Values: ['prod'] }],
       };
 
       await expect(amiManager.getLatestAmi(config)).rejects.toThrow('No matching AMIs found');
@@ -57,13 +64,16 @@ describe('AMIManager', () => {
   describe('updateLaunchTemplate', () => {
     it('should update launch template with new AMI ID', async () => {
       vi.mocked(ec2Client.send)
-        .mockResolvedValueOnce({ // getCurrentAmiId - DescribeLaunchTemplatesCommand
+        .mockResolvedValueOnce({
+          // getCurrentAmiId - DescribeLaunchTemplatesCommand
           LaunchTemplates: [{ LatestVersionNumber: 1 }],
         })
-        .mockResolvedValueOnce({ // getCurrentAmiId - DescribeLaunchTemplateVersionsCommand
+        .mockResolvedValueOnce({
+          // getCurrentAmiId - DescribeLaunchTemplateVersionsCommand
           LaunchTemplateVersions: [{ LaunchTemplateData: { ImageId: 'ami-old' } }],
         })
-        .mockResolvedValueOnce({ // updateLaunchTemplate - DescribeLaunchTemplatesCommand
+        .mockResolvedValueOnce({
+          // updateLaunchTemplate - DescribeLaunchTemplatesCommand
           LaunchTemplates: [{ LatestVersionNumber: 1 }],
         });
 
@@ -77,10 +87,12 @@ describe('AMIManager', () => {
 
     it('should not update if AMI ID is the same', async () => {
       vi.mocked(ec2Client.send)
-        .mockResolvedValueOnce({ // getCurrentAmiId - DescribeLaunchTemplatesCommand
+        .mockResolvedValueOnce({
+          // getCurrentAmiId - DescribeLaunchTemplatesCommand
           LaunchTemplates: [{ LatestVersionNumber: 1 }],
         })
-        .mockResolvedValueOnce({ // getCurrentAmiId - DescribeLaunchTemplateVersionsCommand
+        .mockResolvedValueOnce({
+          // getCurrentAmiId - DescribeLaunchTemplateVersionsCommand
           LaunchTemplateVersions: [{ LaunchTemplateData: { ImageId: 'ami-1' } }],
         });
 
@@ -93,10 +105,12 @@ describe('AMIManager', () => {
 
     it('should handle dry run mode', async () => {
       vi.mocked(ec2Client.send)
-        .mockResolvedValueOnce({ // getCurrentAmiId - DescribeLaunchTemplatesCommand
+        .mockResolvedValueOnce({
+          // getCurrentAmiId - DescribeLaunchTemplatesCommand
           LaunchTemplates: [{ LatestVersionNumber: 1 }],
         })
-        .mockResolvedValueOnce({ // getCurrentAmiId - DescribeLaunchTemplateVersionsCommand
+        .mockResolvedValueOnce({
+          // getCurrentAmiId - DescribeLaunchTemplateVersionsCommand
           LaunchTemplateVersions: [{ LaunchTemplateData: { ImageId: 'ami-old' } }],
         });
 
@@ -107,4 +121,4 @@ describe('AMIManager', () => {
       expect(ec2Client.send).not.toHaveBeenCalledWith(expect.any(CreateLaunchTemplateVersionCommand));
     });
   });
-});
+});

…ami-updater/src/__tests__/config.test.ts …functions/ami-updater/src/config.test.tslambdas/functions/ami-updater/src/__tests__/config.test.ts renamed to lambdas/functions/ami-updater/src/config.test.ts lambdas/functions/ami-updater/src/__tests__/config.test.ts renamed to lambdas/functions/ami-updater/src/config.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { getConfig } from '../config';
+import { getConfig } from './config';
 
 vi.mock('../../shared/aws-powertools-util', () => ({
   logger: {
@@ -76,4 +76,4 @@ describe('getConfig', () => {
 
     expect(() => getConfig()).toThrow('AMI_FILTER must contain owners (array) and filters (array)');
   });
-});
+});

…/ami-updater/src/__tests__/index.test.ts …/functions/ami-updater/src/index.test.tslambdas/functions/ami-updater/src/__tests__/index.test.ts renamed to lambdas/functions/ami-updater/src/index.test.ts lambdas/functions/ami-updater/src/__tests__/index.test.ts renamed to lambdas/functions/ami-updater/src/index.test.ts

@@ -1,10 +1,10 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { handler } from '../lambda';
-import { AMIManager } from '../ami';
-import { getConfig } from '../config';
+import { handler } from './lambda';
+import { AMIManager } from './ami';
+import { getConfig } from './config';
 
-vi.mock('../ami');
-vi.mock('../config');
+vi.mock('./ami');
+vi.mock('./config');
 vi.mock('@aws-github-runner/aws-powertools-util', () => ({
   logger: {
     addContext: vi.fn(),
@@ -24,7 +24,7 @@ describe('Lambda Handler', () => {
     dryRun: false,
     amiFilter: {
       owners: ['self'],
-      filters: [{ name: 'tag:Environment', values: ['prod'] }],
+      filters: [{ Name: 'tag:Environment', Values: ['prod'] }],
     },
   };