summaryrefslogtreecommitdiff
path: root/internal/media
diff options
context:
space:
mode:
authorLibravatar Daenney <git@noreply.sourcery.dny.nu>2025-05-22 12:26:11 +0200
committerLibravatar kim <gruf@noreply.codeberg.org>2025-05-22 12:26:11 +0200
commitd5c9c4adc167cdb05e73f5105702cf340293e61c (patch)
tree1d21fe376099864900837eba675a965517f98e5d /internal/media
parent[feature] Allow exposing allows, implement `/api/v1/domain_blocks` and `/api/... (diff)
downloadgotosocial-d5c9c4adc167cdb05e73f5105702cf340293e61c.tar.xz
[chore] Upgrade to Go 1.24 (#4187)
* Set `go.mod` to 1.24 now that it's been out for 3 months. * Update all the test to use `testing.T.Context()`. Reviewed-on: https://codeberg.org/superseriousbusiness/gotosocial/pulls/4187 Co-authored-by: Daenney <git@noreply.sourcery.dny.nu> Co-committed-by: Daenney <git@noreply.sourcery.dny.nu>
Diffstat (limited to 'internal/media')
-rw-r--r--internal/media/ffmpeg.go7
-rw-r--r--internal/media/manager_test.go36
-rw-r--r--internal/media/refetch_test.go7
3 files changed, 24 insertions, 26 deletions
diff --git a/internal/media/ffmpeg.go b/internal/media/ffmpeg.go
index 41cc2666f..676eba9be 100644
--- a/internal/media/ffmpeg.go
+++ b/internal/media/ffmpeg.go
@@ -348,11 +348,10 @@ func (res *result) GetFileType() (gtsmodel.FileType, string, string) {
// video file aka gifv.
return gtsmodel.FileTypeGifv,
"video/mp4", "mp4"
- } else {
- // Video file (with or without audio).
- return gtsmodel.FileTypeVideo,
- "video/mp4", "mp4"
}
+ // Video file (with or without audio).
+ return gtsmodel.FileTypeVideo,
+ "video/mp4", "mp4"
case len(res.audio) > 0 &&
res.audio[0].codec == "aac":
// m4a only supports [aac] audio.
diff --git a/internal/media/manager_test.go b/internal/media/manager_test.go
index 4988c2485..6182babbb 100644
--- a/internal/media/manager_test.go
+++ b/internal/media/manager_test.go
@@ -44,7 +44,7 @@ type ManagerTestSuite struct {
}
func (suite *ManagerTestSuite) TestEmojiProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -84,7 +84,7 @@ func (suite *ManagerTestSuite) TestEmojiProcess() {
}
func (suite *ManagerTestSuite) TestEmojiProcessRefresh() {
- ctx := context.Background()
+ ctx := suite.T().Context()
// we're going to 'refresh' the remote 'yell' emoji by changing the image url to the pixellated gts logo
originalEmoji := suite.testEmojis["yell"]
@@ -157,7 +157,7 @@ func (suite *ManagerTestSuite) TestEmojiProcessRefresh() {
}
func (suite *ManagerTestSuite) TestEmojiProcessTooLarge() {
- ctx := context.Background()
+ ctx := suite.T().Context()
// Open test image as file for reading.
file, err := os.Open("./test/big-panda.gif")
@@ -191,7 +191,7 @@ func (suite *ManagerTestSuite) TestEmojiProcessTooLarge() {
}
func (suite *ManagerTestSuite) TestEmojiWebpProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -232,7 +232,7 @@ func (suite *ManagerTestSuite) TestEmojiWebpProcess() {
}
func (suite *ManagerTestSuite) TestSimpleJpegProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -288,7 +288,7 @@ func (suite *ManagerTestSuite) TestSimpleJpegProcess() {
}
func (suite *ManagerTestSuite) TestSimpleJpegProcessTooLarge() {
- ctx := context.Background()
+ ctx := suite.T().Context()
// Open test image as file for reading.
file, err := os.Open("./test/test-jpeg.jpg")
@@ -325,7 +325,7 @@ func (suite *ManagerTestSuite) TestSimpleJpegProcessTooLarge() {
}
func (suite *ManagerTestSuite) TestPDFProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from Frantz
@@ -381,7 +381,7 @@ func (suite *ManagerTestSuite) TestPDFProcess() {
}
func (suite *ManagerTestSuite) TestSlothVineProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test video
@@ -441,7 +441,7 @@ func (suite *ManagerTestSuite) TestSlothVineProcess() {
}
func (suite *ManagerTestSuite) TestAnimatedGifProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -506,7 +506,7 @@ func (suite *ManagerTestSuite) TestAnimatedGifProcess() {
}
func (suite *ManagerTestSuite) TestLongerMp4Process() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test video
@@ -566,7 +566,7 @@ func (suite *ManagerTestSuite) TestLongerMp4Process() {
}
func (suite *ManagerTestSuite) TestBirdnestMp4Process() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test video
@@ -626,7 +626,7 @@ func (suite *ManagerTestSuite) TestBirdnestMp4Process() {
}
func (suite *ManagerTestSuite) TestOpusProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -678,7 +678,7 @@ func (suite *ManagerTestSuite) TestOpusProcess() {
}
func (suite *ManagerTestSuite) TestPngNoAlphaChannelProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -734,7 +734,7 @@ func (suite *ManagerTestSuite) TestPngNoAlphaChannelProcess() {
}
func (suite *ManagerTestSuite) TestPngAlphaChannelProcess() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -790,7 +790,7 @@ func (suite *ManagerTestSuite) TestPngAlphaChannelProcess() {
}
func (suite *ManagerTestSuite) TestSimpleJpegProcessWithCallback() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -846,7 +846,7 @@ func (suite *ManagerTestSuite) TestSimpleJpegProcessWithCallback() {
}
func (suite *ManagerTestSuite) TestSimpleJpegProcessWithDiskStorage() {
- ctx := context.Background()
+ ctx := suite.T().Context()
data := func(_ context.Context) (io.ReadCloser, error) {
// load bytes from a test image
@@ -951,7 +951,7 @@ func (suite *ManagerTestSuite) TestSmallSizedMediaTypeDetection_issue2263() {
},
} {
suite.Run(test.name, func() {
- ctx, cncl := context.WithTimeout(context.Background(), time.Second*60)
+ ctx, cncl := context.WithTimeout(suite.T().Context(), time.Second*60)
defer cncl()
data := func(_ context.Context) (io.ReadCloser, error) {
@@ -1002,7 +1002,7 @@ func TestManagerTestSuite(t *testing.T) {
// equalFiles checks whether
func equalFiles(t *testing.T, st *storage.Driver, storagePath, testPath string) {
- b1, err := st.Get(context.Background(), storagePath)
+ b1, err := st.Get(t.Context(), storagePath)
if err != nil {
t.Fatalf("error reading file %s: %v", storagePath, err)
}
diff --git a/internal/media/refetch_test.go b/internal/media/refetch_test.go
index c59b37d8d..cd9fb9468 100644
--- a/internal/media/refetch_test.go
+++ b/internal/media/refetch_test.go
@@ -18,7 +18,6 @@
package media_test
import (
- "context"
"testing"
"github.com/stretchr/testify/suite"
@@ -29,7 +28,7 @@ type RefetchTestSuite struct {
}
func (suite *RefetchTestSuite) TestRefetchEmojisNothingToDo() {
- ctx := context.Background()
+ ctx := suite.T().Context()
adminAccount := suite.testAccounts["admin_account"]
transport, err := suite.transportController.NewTransportForUsername(ctx, adminAccount.Username)
@@ -43,7 +42,7 @@ func (suite *RefetchTestSuite) TestRefetchEmojisNothingToDo() {
}
func (suite *RefetchTestSuite) TestRefetchEmojis() {
- ctx := context.Background()
+ ctx := suite.T().Context()
if err := suite.storage.Delete(ctx, suite.testEmojis["yell"].ImagePath); err != nil {
suite.FailNow(err.Error())
@@ -61,7 +60,7 @@ func (suite *RefetchTestSuite) TestRefetchEmojis() {
}
func (suite *RefetchTestSuite) TestRefetchEmojisLocal() {
- ctx := context.Background()
+ ctx := suite.T().Context()
// delete the image for a LOCAL emoji
if err := suite.storage.Delete(ctx, suite.testEmojis["rainbow"].ImagePath); err != nil {