diff --git a/.github/workflows/codetests.yml b/.github/workflows/codetests.yml index 04194e83..af2a3bc8 100644 --- a/.github/workflows/codetests.yml +++ b/.github/workflows/codetests.yml @@ -49,7 +49,7 @@ jobs: - name: golangci-lint uses: golangci/golangci-lint-action@v6 with: - version: v1.59 + version: v1.64 # Runs golangci-lint on linux against linux and windows. golangci-linux: strategy: @@ -67,4 +67,4 @@ jobs: - name: golangci-lint uses: golangci/golangci-lint-action@v6 with: - version: v1.59 + version: v1.64 diff --git a/.golangci.yml b/.golangci.yml index 82b60527..22c93aad 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -17,8 +17,7 @@ linters: enable-all: true disable: # deprecated - - gomnd - - execinquery + - tenv # unused - tagliatelle - exhaustruct @@ -28,7 +27,7 @@ linters: linters-settings: ireturn: - allow: generic,stdlib,error + allow: [generic,stdlib,error] # One day we'll fix all the field alignments. Anyone got a tool that just does it? # govet: # enable: @@ -41,6 +40,8 @@ linters-settings: allow: - $gostd - github.com/stretchr/testify + - golift.io/starr + - golang.org/x/net run: timeout: 5m diff --git a/go.mod b/go.mod index 67a97b57..93da6417 100644 --- a/go.mod +++ b/go.mod @@ -1,8 +1,8 @@ module golift.io/starr -go 1.20 +go 1.23.0 -require golang.org/x/net v0.22.0 // publicsuffix, cookiejar. +require golang.org/x/net v0.37.0 // publicsuffix, cookiejar. // All of this is for the tests. require ( diff --git a/go.sum b/go.sum index 08bdd7e9..a2af5c22 100644 --- a/go.sum +++ b/go.sum @@ -4,8 +4,8 @@ github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZb github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4= github.com/stretchr/testify v1.9.0 h1:HtqpIVDClZ4nwg75+f6Lvsy/wHu+3BoSGCbBAcpTsTg= github.com/stretchr/testify v1.9.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY= -golang.org/x/net v0.22.0 h1:9sGLhx7iRIHEiX0oAJ3MRZMUCElJgy7Br1nO+AMN3Tc= -golang.org/x/net v0.22.0/go.mod h1:JKghWKKOSdJwpW2GEx0Ja7fmaKnMsbu+MWVZTokSYmg= +golang.org/x/net v0.37.0 h1:1zLorHbz+LYj7MQlSf1+2tPIIgibq2eL5xkrGk6f+2c= +golang.org/x/net v0.37.0/go.mod h1:ivrbrMbzFq5J41QOQh0siUuly180yBYtLp+CKbEaFx8= gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM= gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0= gopkg.in/yaml.v3 v3.0.1 h1:fxVm/GzAzEWqLHuvctI91KS9hhNmmWOoWu0XTYJS7CA= diff --git a/lidarr/calendar_test.go b/lidarr/calendar_test.go index c76c65c7..c3931fc0 100644 --- a/lidarr/calendar_test.go +++ b/lidarr/calendar_test.go @@ -190,7 +190,6 @@ func TestGetCalendar(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -229,7 +228,6 @@ func TestGetCalendarID(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/lidarr/command_test.go b/lidarr/command_test.go index 0878211a..b8e02e56 100644 --- a/lidarr/command_test.go +++ b/lidarr/command_test.go @@ -63,7 +63,6 @@ func TestGetCommands(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -137,7 +136,6 @@ func TestSendCommand(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/lidarr/downloadclient_test.go b/lidarr/downloadclient_test.go index 07d58741..b3ff4bf4 100644 --- a/lidarr/downloadclient_test.go +++ b/lidarr/downloadclient_test.go @@ -131,7 +131,6 @@ func TestGetDownloadClients(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -208,7 +207,6 @@ func TestGetDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -333,7 +331,6 @@ func TestAddDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -460,7 +457,6 @@ func TestUpdateDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -497,7 +493,6 @@ func TestDeleteDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/lidarr/downloadclientconfig_test.go b/lidarr/downloadclientconfig_test.go index 3e679361..2b8b9147 100644 --- a/lidarr/downloadclientconfig_test.go +++ b/lidarr/downloadclientconfig_test.go @@ -51,7 +51,6 @@ func TestGetDownloadClientConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -109,7 +108,6 @@ func TestUpdateDownloadClientConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/lidarr/indexer_test.go b/lidarr/indexer_test.go index 859dabf2..8f09fa3d 100644 --- a/lidarr/indexer_test.go +++ b/lidarr/indexer_test.go @@ -121,7 +121,6 @@ func TestGetIndexers(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -194,7 +193,6 @@ func TestGetIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -309,7 +307,6 @@ func TestAddIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -426,7 +423,6 @@ func TestUpdateIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -463,7 +459,6 @@ func TestDeleteIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/lidarr/indexerconfig_test.go b/lidarr/indexerconfig_test.go index c77d693d..2298f4cf 100644 --- a/lidarr/indexerconfig_test.go +++ b/lidarr/indexerconfig_test.go @@ -51,7 +51,6 @@ func TestGetIndexerConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -110,7 +109,6 @@ func TestUpdateIndexerConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/lidarr/notification_test.go b/lidarr/notification_test.go index 509ef94f..f9d26ae7 100644 --- a/lidarr/notification_test.go +++ b/lidarr/notification_test.go @@ -136,7 +136,6 @@ func TestGetNotifications(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -212,7 +211,6 @@ func TestGetNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -312,7 +310,6 @@ func TestAddNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -414,7 +411,6 @@ func TestUpdateNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -451,7 +447,6 @@ func TestDeleteNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/lidarr/remotepathmapping_test.go b/lidarr/remotepathmapping_test.go index 1f2c6738..bb3822a3 100644 --- a/lidarr/remotepathmapping_test.go +++ b/lidarr/remotepathmapping_test.go @@ -53,7 +53,6 @@ func TestGetRemotePathMappings(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -97,7 +96,6 @@ func TestGetRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -151,7 +149,6 @@ func TestAddRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -207,7 +204,6 @@ func TestUpdateRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -244,7 +240,6 @@ func TestDeleteRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/lidarr/tag_test.go b/lidarr/tag_test.go index fd4a0b61..2a6012a9 100644 --- a/lidarr/tag_test.go +++ b/lidarr/tag_test.go @@ -46,7 +46,6 @@ func TestGetTags(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -88,7 +87,6 @@ func TestGetTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -136,7 +134,6 @@ func TestAddTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -186,7 +183,6 @@ func TestUpdateTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -223,7 +219,6 @@ func TestDeleteTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/prowlarr/downloadclient_test.go b/prowlarr/downloadclient_test.go index 559ff125..dbf93e2a 100644 --- a/prowlarr/downloadclient_test.go +++ b/prowlarr/downloadclient_test.go @@ -127,7 +127,6 @@ func TestGetDownloadClients(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -204,7 +203,6 @@ func TestGetDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -325,7 +323,6 @@ func TestAddDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -448,7 +445,6 @@ func TestUpdateDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -485,7 +481,6 @@ func TestDeleteDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/prowlarr/indexer_test.go b/prowlarr/indexer_test.go index a158a0e5..ed5e1678 100644 --- a/prowlarr/indexer_test.go +++ b/prowlarr/indexer_test.go @@ -222,7 +222,6 @@ func TestGetIndexers(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -262,7 +261,6 @@ func TestGetIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -339,7 +337,6 @@ func TestAddIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -418,7 +415,6 @@ func TestUpdateIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -455,7 +451,6 @@ func TestDeleteIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/prowlarr/notification_test.go b/prowlarr/notification_test.go index 92689c26..2316abfd 100644 --- a/prowlarr/notification_test.go +++ b/prowlarr/notification_test.go @@ -121,7 +121,6 @@ func TestGetNotifications(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -197,7 +196,6 @@ func TestGetNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -297,7 +295,6 @@ func TestAddNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -399,7 +396,6 @@ func TestUpdateNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -436,7 +432,6 @@ func TestDeleteNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/prowlarr/tag_test.go b/prowlarr/tag_test.go index a02822cc..a968dba2 100644 --- a/prowlarr/tag_test.go +++ b/prowlarr/tag_test.go @@ -46,7 +46,6 @@ func TestGetTags(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -88,7 +87,6 @@ func TestGetTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -136,7 +134,6 @@ func TestAddTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -186,7 +183,6 @@ func TestUpdateTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -223,7 +219,6 @@ func TestDeleteTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/calendar_test.go b/radarr/calendar_test.go index ef2390cd..1547b8eb 100644 --- a/radarr/calendar_test.go +++ b/radarr/calendar_test.go @@ -183,7 +183,6 @@ func TestGetCalendar(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/command_test.go b/radarr/command_test.go index 02d334fb..84d97205 100644 --- a/radarr/command_test.go +++ b/radarr/command_test.go @@ -63,7 +63,6 @@ func TestGetCommands(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -137,7 +136,6 @@ func TestSendCommand(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/customformat_test.go b/radarr/customformat_test.go index 62a1b96f..dda98b21 100644 --- a/radarr/customformat_test.go +++ b/radarr/customformat_test.go @@ -168,7 +168,6 @@ func TestGetCustomFormats(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -265,7 +264,6 @@ func TestGetCustomFormat(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -422,7 +420,6 @@ func TestAddCustomFormat(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -581,7 +578,6 @@ func TestUpdateCustomFormat(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -618,7 +614,6 @@ func TestDeleteCustomFormat(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/delayprofile_test.go b/radarr/delayprofile_test.go index 60ac3bfb..22b764a8 100644 --- a/radarr/delayprofile_test.go +++ b/radarr/delayprofile_test.go @@ -86,7 +86,6 @@ func TestGetDelayProfiles(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -135,7 +134,6 @@ func TestGetDelayProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -204,7 +202,6 @@ func TestAddDelayProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -263,7 +260,6 @@ func TestUpdateDelayProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -301,7 +297,6 @@ func TestDeleteDelayProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/downloadclient_test.go b/radarr/downloadclient_test.go index 9c7ee58b..b3ca5ef7 100644 --- a/radarr/downloadclient_test.go +++ b/radarr/downloadclient_test.go @@ -131,7 +131,6 @@ func TestGetDownloadClients(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -208,7 +207,6 @@ func TestGetDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -333,7 +331,6 @@ func TestAddDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -460,7 +457,6 @@ func TestUpdateDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -497,7 +493,6 @@ func TestDeleteDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/downloadclientconfig_test.go b/radarr/downloadclientconfig_test.go index f1ca79d1..80737dcd 100644 --- a/radarr/downloadclientconfig_test.go +++ b/radarr/downloadclientconfig_test.go @@ -53,7 +53,6 @@ func TestGetDownloadClientConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -114,7 +113,6 @@ func TestUpdateDownloadClientConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/indexer_test.go b/radarr/indexer_test.go index c11925e1..c80da5c9 100644 --- a/radarr/indexer_test.go +++ b/radarr/indexer_test.go @@ -122,7 +122,6 @@ func TestGetIndexers(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -195,7 +194,6 @@ func TestGetIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -312,7 +310,6 @@ func TestAddIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -431,7 +428,6 @@ func TestUpdateIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -468,7 +464,6 @@ func TestDeleteIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/indexerconfig_test.go b/radarr/indexerconfig_test.go index 51ccf7ba..56a9ea57 100644 --- a/radarr/indexerconfig_test.go +++ b/radarr/indexerconfig_test.go @@ -62,7 +62,6 @@ func TestGetIndexerConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -132,7 +131,6 @@ func TestUpdateIndexerConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/mediamanagement_test.go b/radarr/mediamanagement_test.go index 89245122..ee4a343b 100644 --- a/radarr/mediamanagement_test.go +++ b/radarr/mediamanagement_test.go @@ -84,7 +84,6 @@ func TestGetMediaManagement(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -152,7 +151,6 @@ func TestUpdateMediaManagement(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/movieeditor_test.go b/radarr/movieeditor_test.go index 8aed5e04..f515a5f3 100644 --- a/radarr/movieeditor_test.go +++ b/radarr/movieeditor_test.go @@ -54,7 +54,6 @@ func TestEditMovies(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -86,7 +85,6 @@ func TestDeleteMovies(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/naming_test.go b/radarr/naming_test.go index 7aa34201..3aacf688 100644 --- a/radarr/naming_test.go +++ b/radarr/naming_test.go @@ -53,7 +53,6 @@ func TestGetNaming(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -109,7 +108,6 @@ func TestUpdateNaming(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/notification_test.go b/radarr/notification_test.go index fd7fae30..5d454ff6 100644 --- a/radarr/notification_test.go +++ b/radarr/notification_test.go @@ -139,7 +139,6 @@ func TestGetNotifications(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -216,7 +215,6 @@ func TestGetNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -317,7 +315,6 @@ func TestAddNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -420,7 +417,6 @@ func TestUpdateNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -457,7 +453,6 @@ func TestDeleteNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/qualityprofile_test.go b/radarr/qualityprofile_test.go index 4bd4790f..12c4e187 100644 --- a/radarr/qualityprofile_test.go +++ b/radarr/qualityprofile_test.go @@ -138,7 +138,6 @@ func TestGetQualityProfiles(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -219,7 +218,6 @@ func TestGetQualityProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -380,7 +378,6 @@ func TestAddQualityProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -543,7 +540,6 @@ func TestUpdateQualityProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -581,7 +577,6 @@ func TestDeleteQualityProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/remotepathmapping_test.go b/radarr/remotepathmapping_test.go index a0646712..c4367cc5 100644 --- a/radarr/remotepathmapping_test.go +++ b/radarr/remotepathmapping_test.go @@ -53,7 +53,6 @@ func TestGetRemotePathMappings(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -97,7 +96,6 @@ func TestGetRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -151,7 +149,6 @@ func TestAddRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -207,7 +204,6 @@ func TestUpdateRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -244,7 +240,6 @@ func TestDeleteRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/restriction_test.go b/radarr/restriction_test.go index a43f82ad..cd7e3b9e 100644 --- a/radarr/restriction_test.go +++ b/radarr/restriction_test.go @@ -51,7 +51,6 @@ func TestGetRestrictions(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -95,7 +94,6 @@ func TestGetRestriction(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -147,7 +145,6 @@ func TestAddRestriction(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -201,7 +198,6 @@ func TestUpdateRestriction(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -238,7 +234,6 @@ func TestDeleteRestriction(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/rootfolder_test.go b/radarr/rootfolder_test.go index 32cbdedf..f0302088 100644 --- a/radarr/rootfolder_test.go +++ b/radarr/rootfolder_test.go @@ -79,7 +79,6 @@ func TestGetRootFolders(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -124,7 +123,6 @@ func TestGetRootFolder(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -180,7 +178,6 @@ func TestAddRootFolder(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -217,7 +214,6 @@ func TestDeleteRootFolder(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/radarr/tag_test.go b/radarr/tag_test.go index bbe6f54d..e7a9345b 100644 --- a/radarr/tag_test.go +++ b/radarr/tag_test.go @@ -46,7 +46,6 @@ func TestGetTags(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -88,7 +87,6 @@ func TestGetTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -136,7 +134,6 @@ func TestAddTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -186,7 +183,6 @@ func TestUpdateTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -223,7 +219,6 @@ func TestDeleteTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/calendar_test.go b/readarr/calendar_test.go index a5988731..9ba9cc1e 100644 --- a/readarr/calendar_test.go +++ b/readarr/calendar_test.go @@ -153,7 +153,6 @@ func TestGetCalendar(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -192,7 +191,6 @@ func TestGetCalendarID(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/command_test.go b/readarr/command_test.go index 8da3cff0..46a0c256 100644 --- a/readarr/command_test.go +++ b/readarr/command_test.go @@ -63,7 +63,6 @@ func TestGetCommands(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -138,7 +137,6 @@ func TestSendCommand(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/downloadclient_test.go b/readarr/downloadclient_test.go index aa37dded..892485f1 100644 --- a/readarr/downloadclient_test.go +++ b/readarr/downloadclient_test.go @@ -123,7 +123,6 @@ func TestGetDownloadClients(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -200,7 +199,6 @@ func TestGetDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -321,7 +319,6 @@ func TestAddDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -444,7 +441,6 @@ func TestUpdateDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -481,7 +477,6 @@ func TestDeleteDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/downloadclientconfig_test.go b/readarr/downloadclientconfig_test.go index 4159b598..aaf81734 100644 --- a/readarr/downloadclientconfig_test.go +++ b/readarr/downloadclientconfig_test.go @@ -58,7 +58,6 @@ func TestGetDownloadClientConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -120,7 +119,6 @@ func TestUpdateDownloadClientConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/indexer_test.go b/readarr/indexer_test.go index e46cd8f3..bad8bcf0 100644 --- a/readarr/indexer_test.go +++ b/readarr/indexer_test.go @@ -121,7 +121,6 @@ func TestGetIndexers(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -194,7 +193,6 @@ func TestGetIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -309,7 +307,6 @@ func TestAddIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -426,7 +423,6 @@ func TestUpdateIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -463,7 +459,6 @@ func TestDeleteIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/indexerconfig_test.go b/readarr/indexerconfig_test.go index 82831d1a..b5d42f09 100644 --- a/readarr/indexerconfig_test.go +++ b/readarr/indexerconfig_test.go @@ -51,7 +51,6 @@ func TestGetIndexerConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -110,7 +109,6 @@ func TestUpdateIndexerConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/notification_test.go b/readarr/notification_test.go index 3ded34b9..cfea0d55 100644 --- a/readarr/notification_test.go +++ b/readarr/notification_test.go @@ -145,7 +145,6 @@ func TestGetNotifications(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -224,7 +223,6 @@ func TestGetNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -327,7 +325,6 @@ func TestAddNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -432,7 +429,6 @@ func TestUpdateNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -469,7 +465,6 @@ func TestDeleteNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/remotepathmapping_test.go b/readarr/remotepathmapping_test.go index d76b27ba..73f9614f 100644 --- a/readarr/remotepathmapping_test.go +++ b/readarr/remotepathmapping_test.go @@ -53,7 +53,6 @@ func TestGetRemotePathMappings(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -97,7 +96,6 @@ func TestGetRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -151,7 +149,6 @@ func TestAddRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -207,7 +204,6 @@ func TestUpdateRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -244,7 +240,6 @@ func TestDeleteRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/readarr/tag_test.go b/readarr/tag_test.go index 7f6ec0a2..8c2f7095 100644 --- a/readarr/tag_test.go +++ b/readarr/tag_test.go @@ -46,7 +46,6 @@ func TestGetTags(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -88,7 +87,6 @@ func TestGetTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -136,7 +134,6 @@ func TestAddTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -186,7 +183,6 @@ func TestUpdateTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -223,7 +219,6 @@ func TestDeleteTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/calendar_test.go b/sonarr/calendar_test.go index 3ecc8c93..f14ca06f 100644 --- a/sonarr/calendar_test.go +++ b/sonarr/calendar_test.go @@ -190,7 +190,6 @@ func TestGetCalendar(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -229,7 +228,6 @@ func TestGetCalendarID(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/command_test.go b/sonarr/command_test.go index 0b0d364a..9fb0e356 100644 --- a/sonarr/command_test.go +++ b/sonarr/command_test.go @@ -64,7 +64,6 @@ func TestGetCommands(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -139,7 +138,6 @@ func TestSendCommand(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -210,7 +208,6 @@ func TestGetCommandStatus(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/customformat_test.go b/sonarr/customformat_test.go index 9fbe5367..343ea13c 100644 --- a/sonarr/customformat_test.go +++ b/sonarr/customformat_test.go @@ -168,7 +168,6 @@ func TestGetCustomFormats(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -265,7 +264,6 @@ func TestGetCustomFormat(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -422,7 +420,6 @@ func TestAddCustomFormat(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -581,7 +578,6 @@ func TestUpdateCustomFormat(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -618,7 +614,6 @@ func TestDeleteCustomFormat(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/delayprofile_test.go b/sonarr/delayprofile_test.go index eadc2225..02f05d9c 100644 --- a/sonarr/delayprofile_test.go +++ b/sonarr/delayprofile_test.go @@ -86,7 +86,6 @@ func TestGetDelayProfiles(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -135,7 +134,6 @@ func TestGetDelayProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -204,7 +202,6 @@ func TestAddDelayProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -263,7 +260,6 @@ func TestUpdateDelayProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -301,7 +297,6 @@ func TestDeleteDelayProfile(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/downloadclient_test.go b/sonarr/downloadclient_test.go index 86fdca04..4d83b86f 100644 --- a/sonarr/downloadclient_test.go +++ b/sonarr/downloadclient_test.go @@ -131,7 +131,6 @@ func TestGetDownloadClients(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -208,7 +207,6 @@ func TestGetDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -333,7 +331,6 @@ func TestAddDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -460,7 +457,6 @@ func TestUpdateDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -497,7 +493,6 @@ func TestDeleteDownloadClient(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/downloadclientconfig_test.go b/sonarr/downloadclientconfig_test.go index 0d5ff3da..55649929 100644 --- a/sonarr/downloadclientconfig_test.go +++ b/sonarr/downloadclientconfig_test.go @@ -51,7 +51,6 @@ func TestGetDownloadClientConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -109,7 +108,6 @@ func TestUpdateDownloadClientConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/episode_test.go b/sonarr/episode_test.go index 1e9d9225..d7579812 100644 --- a/sonarr/episode_test.go +++ b/sonarr/episode_test.go @@ -287,7 +287,6 @@ func TestGetEpisodeByID(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/importlist_test.go b/sonarr/importlist_test.go index 9baaf3e0..6515006f 100644 --- a/sonarr/importlist_test.go +++ b/sonarr/importlist_test.go @@ -117,7 +117,6 @@ func TestGetImportLists(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -191,7 +190,6 @@ func TestGetImportList(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -301,7 +299,6 @@ func TestAddImportList(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -413,7 +410,6 @@ func TestUpdateImportList(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -450,7 +446,6 @@ func TestDeleteImportList(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/indexer_test.go b/sonarr/indexer_test.go index fe43d06c..e2b78a95 100644 --- a/sonarr/indexer_test.go +++ b/sonarr/indexer_test.go @@ -122,7 +122,6 @@ func TestGetIndexers(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -195,7 +194,6 @@ func TestGetIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -312,7 +310,6 @@ func TestAddIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -431,7 +428,6 @@ func TestUpdateIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -468,7 +464,6 @@ func TestDeleteIndexer(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/indexerconfig_test.go b/sonarr/indexerconfig_test.go index 03e0ac0a..17f966b9 100644 --- a/sonarr/indexerconfig_test.go +++ b/sonarr/indexerconfig_test.go @@ -51,7 +51,6 @@ func TestGetIndexerConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -110,7 +109,6 @@ func TestUpdateIndexerConfig(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/mediamanagement_test.go b/sonarr/mediamanagement_test.go index a20f4a20..dca1197a 100644 --- a/sonarr/mediamanagement_test.go +++ b/sonarr/mediamanagement_test.go @@ -82,7 +82,6 @@ func TestGetMediaManagement(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -149,7 +148,6 @@ func TestUpdateMediaManagement(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/naming_test.go b/sonarr/naming_test.go index 9b3e46e3..57174b74 100644 --- a/sonarr/naming_test.go +++ b/sonarr/naming_test.go @@ -64,7 +64,6 @@ func TestGetNaming(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -136,7 +135,6 @@ func TestUpdateNaming(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/notification_test.go b/sonarr/notification_test.go index c3476c70..345d2f0c 100644 --- a/sonarr/notification_test.go +++ b/sonarr/notification_test.go @@ -136,7 +136,6 @@ func TestGetNotifications(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -212,7 +211,6 @@ func TestGetNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -312,7 +310,6 @@ func TestAddNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -414,7 +411,6 @@ func TestUpdateNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -451,7 +447,6 @@ func TestDeleteNotification(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/remotepathmapping_test.go b/sonarr/remotepathmapping_test.go index 95cebbc6..a8d8ecfb 100644 --- a/sonarr/remotepathmapping_test.go +++ b/sonarr/remotepathmapping_test.go @@ -53,7 +53,6 @@ func TestGetRemotePathMappings(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -97,7 +96,6 @@ func TestGetRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -151,7 +149,6 @@ func TestAddRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -207,7 +204,6 @@ func TestUpdateRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -244,7 +240,6 @@ func TestDeleteRemotePathMapping(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/rootfolder_test.go b/sonarr/rootfolder_test.go index bc1924c9..6d47f710 100644 --- a/sonarr/rootfolder_test.go +++ b/sonarr/rootfolder_test.go @@ -79,7 +79,6 @@ func TestGetRootFolders(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -124,7 +123,6 @@ func TestGetRootFolder(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -180,7 +178,6 @@ func TestAddRootFolder(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -217,7 +214,6 @@ func TestDeleteRootFolder(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/series_test.go b/sonarr/series_test.go index 8480d7c5..04a7a1b5 100644 --- a/sonarr/series_test.go +++ b/sonarr/series_test.go @@ -531,7 +531,6 @@ func TestGetAllSeries(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -668,7 +667,6 @@ func TestGetSeries(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -803,7 +801,6 @@ func TestGetSeriesByID(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -1020,7 +1017,6 @@ func TestAddSeries(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -1203,7 +1199,6 @@ func TestUpdateSeries(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -1240,7 +1235,6 @@ func TestDeleteSeries(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -1518,7 +1512,6 @@ func TestLookupName(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -1655,7 +1648,6 @@ func TestLookupID(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/sonarr/tag_test.go b/sonarr/tag_test.go index 7e0e0586..7caa4593 100644 --- a/sonarr/tag_test.go +++ b/sonarr/tag_test.go @@ -46,7 +46,6 @@ func TestGetTags(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -88,7 +87,6 @@ func TestGetTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -136,7 +134,6 @@ func TestAddTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -186,7 +183,6 @@ func TestUpdateTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) @@ -223,7 +219,6 @@ func TestDeleteTag(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.Name, func(t *testing.T) { t.Parallel() mockServer := test.GetMockServer(t) diff --git a/starrcmd/parser.go b/starrcmd/parser.go index a0898605..f3f36b4b 100644 --- a/starrcmd/parser.go +++ b/starrcmd/parser.go @@ -34,7 +34,7 @@ func fillStructFromEnv(dataStruct interface{}) error { } t := field.Type().Elem() - for idx := 0; idx < t.NumField(); idx++ { // Loop each struct member + for idx := range t.NumField() { // Loop each struct member split := strings.SplitN(t.Field(idx).Tag.Get("env"), ",", 2) //nolint:mnd tag := strings.ToLower(split[0]) // lower to protect naming mistakes.