chore(action): add test coverage

action/dist/main.js

@@ -83310,6 +83310,12 @@ var PutObjectCommand = class extends Command2.classBuilder().ep({
 }).s("AmazonS3", "PutObject", {}).n("S3Client", "PutObjectCommand").sc(PutObject$).build() {
 };
 
+// src/s3-client.ts
+var s3Client = new S3Client();
+var listObjects = (input) => s3Client.send(new ListObjectsV2Command(input));
+var getObject = (input) => s3Client.send(new GetObjectCommand(input));
+var putObject = (input) => s3Client.send(new PutObjectCommand(input));
+
 // ../shared/index.ts
 var VISUAL_REGRESSION_CONTEXT = "Visual Regression";
 var BASE_IMAGES_DIRECTORY = "base-images";
@@ -159664,8 +159670,7 @@ var Jimp = createJimp({
   plugins: defaultPlugins
 });
 
-// src/s3-operations.ts
-var s3Client = new S3Client();
+// src/resize.ts
 async function resizeImageIfNeeded(buffer) {
   const resizeWidth = getInput("resize-width");
   const resizeHeight = getInput("resize-height");
@@ -159687,26 +159692,26 @@ async function resizeImageIfNeeded(buffer) {
   }
   return image2.getBuffer("image/png");
 }
+
+// src/s3-operations.ts
 async function checkS3PrefixExists(bucketName, prefix) {
   try {
-    const command = new ListObjectsV2Command({
+    const response = await listObjects({
       Bucket: bucketName,
       Prefix: prefix,
       MaxKeys: 1
     });
-    const response = await s3Client.send(command);
     return (response.Contents?.length ?? 0) > 0;
   } catch {
     return false;
   }
 }
 async function downloadS3Directory(bucketName, s3Prefix, localDir) {
   info(`Downloading base images from s3://${bucketName}/${s3Prefix}`);
-  const command = new ListObjectsV2Command({
+  const response = await listObjects({
     Bucket: bucketName,
     Prefix: s3Prefix
   });
-  const response = await s3Client.send(command);
   const allObjects = response.Contents ?? [];
   const baseObjects = allObjects.filter((obj2) => obj2.Key?.endsWith("base.png"));
   info(`Found ${baseObjects.length} base image(s) to download`);
@@ -159715,11 +159720,10 @@ async function downloadS3Directory(bucketName, s3Prefix, localDir) {
     const relativePath = Key.substring(s3Prefix.length);
     const localFilePath = path4.join(localDir, relativePath);
     await import_fs7.promises.mkdir(path4.dirname(localFilePath), { recursive: true });
-    const getCommand = new GetObjectCommand({
+    const { Body } = await getObject({
       Bucket: bucketName,
       Key
     });
-    const { Body } = await s3Client.send(getCommand);
     if (Body instanceof import_stream10.Readable) {
       const writeStream = fs6.createWriteStream(localFilePath);
       await new Promise((resolve3, reject2) => {
@@ -159748,12 +159752,11 @@ async function uploadLocalDirectoryWithResize(localDir, bucketName, s3Prefix) {
     const s3Key = path4.join(s3Prefix, file);
     const fileBuffer = await import_fs7.promises.readFile(localFilePath);
     const resizedBuffer = await resizeImageIfNeeded(fileBuffer);
-    const command = new PutObjectCommand({
+    await putObject({
       Bucket: bucketName,
       Key: s3Key,
       Body: resizedBuffer
     });
-    await s3Client.send(command);
   });
   info(
     `Uploaded ${filesFromFailingTests.length} file(s) to s3://${bucketName}/${s3Prefix}`
@@ -159763,12 +159766,11 @@ async function uploadSingleFile(localFilePath, s3Key) {
   const bucketName = getInput("bucket-name", { required: true });
   const fileBuffer = await import_fs7.promises.readFile(localFilePath);
   const resizedBuffer = await resizeImageIfNeeded(fileBuffer);
-  const command = new PutObjectCommand({
+  await putObject({
     Bucket: bucketName,
     Key: s3Key,
     Body: resizedBuffer
   });
-  await s3Client.send(command);
   info(`Uploaded ${localFilePath} to s3://${bucketName}/${s3Key}`);
 }
 var downloadBaseImages = async () => {
@@ -159836,12 +159838,11 @@ async function uploadOriginalNewPngs(localDir, bucketName, s3Prefix) {
     const localFilePath = path4.join(localDir, file);
     const s3Key = path4.join(s3Prefix, file);
     const fileBuffer = await import_fs7.promises.readFile(localFilePath);
-    const command = new PutObjectCommand({
+    await putObject({
       Bucket: bucketName,
       Key: s3Key,
       Body: fileBuffer
     });
-    await s3Client.send(command);
   });
   info(
     `Uploaded ${files.length} original new.png file(s) to s3://${bucketName}/${s3Prefix}`

action/src/resize.ts

@@ -0,0 +1,27 @@
+import { getInput } from '@actions/core';
+import { Jimp } from 'jimp';
+
+export async function resizeImageIfNeeded(buffer: Buffer): Promise<Buffer> {
+  const resizeWidth = getInput('resize-width');
+  const resizeHeight = getInput('resize-height');
+
+  if (!resizeWidth && !resizeHeight) {
+    return buffer;
+  }
+  const width = resizeWidth ? Number(resizeWidth) : undefined;
+  const height = resizeHeight ? Number(resizeHeight) : undefined;
+  if ((width && isNaN(width)) || (height && isNaN(height))) {
+    throw new Error('resize-width and resize-height must be valid numbers');
+  }
+
+  const image = await Jimp.read(buffer);
+  if (width && height) {
+    image.cover({ w: width, h: height });
+  } else if (width) {
+    image.resize({ w: width });
+  } else if (height) {
+    image.resize({ h: height });
+  }
+
+  return image.getBuffer('image/png');
+}

action/src/s3-client.ts

@@ -0,0 +1,29 @@
+import {
+  S3Client,
+  ListObjectsV2Command,
+  ListObjectsV2CommandInput,
+  ListObjectsV2CommandOutput,
+  GetObjectCommand,
+  GetObjectCommandInput,
+  GetObjectCommandOutput,
+  PutObjectCommand,
+  PutObjectCommandInput,
+  PutObjectCommandOutput
+} from '@aws-sdk/client-s3';
+
+const s3Client = new S3Client();
+
+export const listObjects = (
+  input: ListObjectsV2CommandInput
+): Promise<ListObjectsV2CommandOutput> =>
+  s3Client.send(new ListObjectsV2Command(input));
+
+export const getObject = (
+  input: GetObjectCommandInput
+): Promise<GetObjectCommandOutput> =>
+  s3Client.send(new GetObjectCommand(input));
+
+export const putObject = (
+  input: PutObjectCommandInput
+): Promise<PutObjectCommandOutput> =>
+  s3Client.send(new PutObjectCommand(input));

action/src/s3-operations.ts

@@ -1,10 +1,5 @@
 import { getInput, info } from '@actions/core';
-import {
-  S3Client,
-  ListObjectsV2Command,
-  GetObjectCommand,
-  PutObjectCommand
-} from '@aws-sdk/client-s3';
+import { listObjects, getObject, putObject } from './s3-client';
 import {
   BASE_IMAGE_NAME,
   BASE_IMAGES_DIRECTORY,
@@ -18,46 +13,18 @@ import * as fs from 'fs';
 import { promises as fsPromises } from 'fs';
 import { glob } from 'glob';
 import { Readable } from 'stream';
-import { Jimp } from 'jimp';
-
-const s3Client = new S3Client();
-
-async function resizeImageIfNeeded(buffer: Buffer): Promise<Buffer> {
-  const resizeWidth = getInput('resize-width');
-  const resizeHeight = getInput('resize-height');
-
-  if (!resizeWidth && !resizeHeight) {
-    return buffer;
-  }
-  const width = resizeWidth ? Number(resizeWidth) : undefined;
-  const height = resizeHeight ? Number(resizeHeight) : undefined;
-  if ((width && isNaN(width)) || (height && isNaN(height))) {
-    throw new Error('resize-width and resize-height must be valid numbers');
-  }
-
-  const image = await Jimp.read(buffer);
-  if (width && height) {
-    image.cover({ w: width, h: height });
-  } else if (width) {
-    image.resize({ w: width });
-  } else if (height) {
-    image.resize({ h: height });
-  }
-
-  return image.getBuffer('image/png');
-}
+import { resizeImageIfNeeded } from './resize';
 
 async function checkS3PrefixExists(
   bucketName: string,
   prefix: string
 ): Promise<boolean> {
   try {
-    const command = new ListObjectsV2Command({
+    const response = await listObjects({
       Bucket: bucketName,
       Prefix: prefix,
       MaxKeys: 1
     });
-    const response = await s3Client.send(command);
     return (response.Contents?.length ?? 0) > 0;
   } catch {
     return false;
@@ -71,12 +38,10 @@ async function downloadS3Directory(
 ): Promise<void> {
   info(`Downloading base images from s3://${bucketName}/${s3Prefix}`);
 
-  const command = new ListObjectsV2Command({
+  const response = await listObjects({
     Bucket: bucketName,
     Prefix: s3Prefix
   });
-
-  const response = await s3Client.send(command);
   const allObjects = response.Contents ?? [];
   const baseObjects = allObjects.filter(obj => obj.Key?.endsWith('base.png'));
 
@@ -90,12 +55,10 @@ async function downloadS3Directory(
 
     await fsPromises.mkdir(path.dirname(localFilePath), { recursive: true });
 
-    const getCommand = new GetObjectCommand({
+    const { Body } = await getObject({
       Bucket: bucketName,
       Key
     });
-
-    const { Body } = await s3Client.send(getCommand);
     if (Body instanceof Readable) {
       const writeStream = fs.createWriteStream(localFilePath);
       await new Promise((resolve, reject) => {
@@ -137,13 +100,11 @@ async function uploadLocalDirectoryWithResize(
     const fileBuffer = await fsPromises.readFile(localFilePath);
     const resizedBuffer = await resizeImageIfNeeded(fileBuffer);
 
-    const command = new PutObjectCommand({
+    await putObject({
       Bucket: bucketName,
       Key: s3Key,
       Body: resizedBuffer
     });
-
-    await s3Client.send(command);
   });
 
   info(
@@ -158,13 +119,11 @@ async function uploadSingleFile(
   const bucketName = getInput('bucket-name', { required: true });
   const fileBuffer = await fsPromises.readFile(localFilePath);
   const resizedBuffer = await resizeImageIfNeeded(fileBuffer);
-  const command = new PutObjectCommand({
+  await putObject({
     Bucket: bucketName,
     Key: s3Key,
     Body: resizedBuffer
   });
-
-  await s3Client.send(command);
   info(`Uploaded ${localFilePath} to s3://${bucketName}/${s3Key}`);
 }
 
@@ -248,13 +207,11 @@ async function uploadOriginalNewPngs(
 
     const fileBuffer = await fsPromises.readFile(localFilePath);
 
-    const command = new PutObjectCommand({
+    await putObject({
       Bucket: bucketName,
       Key: s3Key,
       Body: fileBuffer
     });
-
-    await s3Client.send(command);
   });
 
   info(