From 5548f8e97898c858c8a5a8cc8cf6288183cb4c33 Mon Sep 17 00:00:00 2001 From: Michael Brooks Date: Fri, 9 Oct 2026 12:22:20 -0700 Subject: [PATCH] refactor: remove unused resize image functions --- internal/image/image.go | 33 --------------------------- internal/image/image_test.go | 43 ------------------------------------ 2 files changed, 76 deletions(-) diff --git a/internal/image/image.go b/internal/image/image.go index 59331568..527def92 100644 --- a/internal/image/image.go +++ b/internal/image/image.go @@ -83,36 +83,3 @@ func CropResizeImageRatioToBytes(reader io.Reader, width uint, widthRatio, heigh return buf.Bytes(), nil } - -func ResizeImage(reader io.Reader, width, height uint) (image.Image, error) { - originalImage, _, err := image.Decode(reader) - if err != nil { - return nil, err - } - return resize.Resize(width, height, originalImage, resize.Lanczos3), nil -} - -func ResizeImageToBytes(reader io.Reader, width, height uint) ([]byte, error) { - resizedImg, err := ResizeImage(reader, width, height) - if err != nil { - return nil, err - } - - buf := new(bytes.Buffer) - err = png.Encode(buf, resizedImg) - if err != nil { - return nil, err - } - - return buf.Bytes(), nil -} - -func ResizeImageFromFileToBytes(fs afero.Fs, filepath string, width, height uint) ([]byte, error) { - reader, err := fs.Open(filepath) - if err != nil { - return nil, err - } - defer reader.Close() - - return ResizeImageToBytes(reader, width, height) -} diff --git a/internal/image/image_test.go b/internal/image/image_test.go index 10712e91..3c969b45 100644 --- a/internal/image/image_test.go +++ b/internal/image/image_test.go @@ -41,43 +41,6 @@ func createTestPNG(t *testing.T, width, height int) []byte { return buf.Bytes() } -func Test_ResizeImage(t *testing.T) { - pngData := createTestPNG(t, 100, 100) - reader := bytes.NewReader(pngData) - - resized, err := ResizeImage(reader, 50, 50) - require.NoError(t, err) - assert.NotNil(t, resized) - assert.Equal(t, 50, resized.Bounds().Dx()) - assert.Equal(t, 50, resized.Bounds().Dy()) -} - -func Test_ResizeImageToBytes(t *testing.T) { - pngData := createTestPNG(t, 100, 100) - reader := bytes.NewReader(pngData) - - result, err := ResizeImageToBytes(reader, 50, 50) - require.NoError(t, err) - assert.NotEmpty(t, result) -} - -func Test_ResizeImageFromFileToBytes(t *testing.T) { - pngData := createTestPNG(t, 100, 100) - fs := slackdeps.NewFsMock() - err := afero.WriteFile(fs, "/test.png", pngData, 0644) - require.NoError(t, err) - - result, err := ResizeImageFromFileToBytes(fs, "/test.png", 50, 50) - require.NoError(t, err) - assert.NotEmpty(t, result) -} - -func Test_ResizeImageFromFileToBytes_FileNotFound(t *testing.T) { - fs := slackdeps.NewFsMock() - _, err := ResizeImageFromFileToBytes(fs, "/nonexistent.png", 50, 50) - assert.Error(t, err) -} - func Test_CropResizeImageRatio(t *testing.T) { pngData := createTestPNG(t, 200, 100) reader := bytes.NewReader(pngData) @@ -130,9 +93,3 @@ func Test_CropResizeImageRatioFromFileToBytes_FileNotFound(t *testing.T) { _, err := CropResizeImageRatioFromFileToBytes(fs, "/nonexistent.png", 100, 1, 1) assert.Error(t, err) } - -func Test_ResizeImage_InvalidReader(t *testing.T) { - reader := bytes.NewReader([]byte("not a valid image")) - _, err := ResizeImage(reader, 50, 50) - assert.Error(t, err) -}