mirrors/mostlymatter
1
0
Форкнуть 0
Код Задачи Запросы на слияние Действия Пакеты Проекты Релизы Вики Активность

fix errcheck issues (#28560)

Просмотр исходного кода 
* fix errcheck issues

* fix errcheck

* fix errcheck issues

* fix errcheck

* fix errcheck
Этот коммит содержится в:
Johnson
2024-10-23 12:30:56 +01:00
коммит произвёл GitHub
родитель 610a756e7f
Коммит 131064596d
2 изменённых файлов: 28 добавлений и 15 удалений
Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

1
server/.golangci.yml
Просмотреть файл

@@ -73,7 +73,6 @@ issues:
channels/api4/config_test.go|\ channels/api4/config_test.go|\
channels/api4/data_retention.go|\ channels/api4/data_retention.go|\
channels/api4/file_test.go|\ channels/api4/file_test.go|\
channels/api4/ip_filtering_test.go|\
channels/api4/job_test.go|\ channels/api4/job_test.go|\
channels/api4/license_local.go|\ channels/api4/license_local.go|\
channels/api4/post_test.go|\ channels/api4/post_test.go|\

42
server/channels/api4/ip_filtering_test.go
Просмотреть файл

@@ -37,9 +37,11 @@ func Test_getIPFilters(t *testing.T) {
ipFiltering := &mocks.IPFilteringInterface{} ipFiltering := &mocks.IPFilteringInterface{}
th.App.Srv().IPFiltering = ipFiltering th.App.Srv().IPFiltering = ipFiltering
th.App.Srv().RemoveLicense() appErr := th.App.Srv().RemoveLicense()
require.Nil(t, appErr)
th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password) _, _, err := th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password)
require.NoError(t, err)
ipFilters, r, err := th.Client.GetIPFilters(context.Background()) ipFilters, r, err := th.Client.GetIPFilters(context.Background())
require.Error(t, err) require.Error(t, err)
@@ -58,7 +60,8 @@ func Test_getIPFilters(t *testing.T) {
th.App.Srv().SetLicense(lic) th.App.Srv().SetLicense(lic)
th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password) _, _, err := th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password)
require.NoError(t, err)
ipFilters, r, err := th.Client.GetIPFilters(context.Background()) ipFilters, r, err := th.Client.GetIPFilters(context.Background())
require.Error(t, err) require.Error(t, err)
@@ -76,7 +79,8 @@ func Test_getIPFilters(t *testing.T) {
th.App.Srv().SetLicense(lic) th.App.Srv().SetLicense(lic)
th.Client.Login(context.Background(), th.BasicUser2.Email, th.BasicUser2.Password) _, _, err := th.Client.Login(context.Background(), th.BasicUser2.Email, th.BasicUser2.Password)
require.NoError(t, err)
ipFilters, r, err := th.Client.GetIPFilters(context.Background()) ipFilters, r, err := th.Client.GetIPFilters(context.Background())
require.Error(t, err) require.Error(t, err)
@@ -100,7 +104,8 @@ func Test_getIPFilters(t *testing.T) {
th.App.Srv().SetLicense(lic) th.App.Srv().SetLicense(lic)
th.Client.Login(context.Background(), th.SystemAdminUser.Email, th.SystemAdminUser.Password) _, _, err := th.Client.Login(context.Background(), th.SystemAdminUser.Email, th.SystemAdminUser.Password)
require.NoError(t, err)
ipFilters, r, err := th.Client.GetIPFilters(context.Background()) ipFilters, r, err := th.Client.GetIPFilters(context.Background())
require.NoError(t, err) require.NoError(t, err)
@@ -126,7 +131,8 @@ func Test_getIPFilters(t *testing.T) {
th.App.Srv().SetLicense(lic) th.App.Srv().SetLicense(lic)
th.Client.Login(context.Background(), th.SystemAdminUser.Email, th.SystemAdminUser.Password) _, _, err := th.Client.Login(context.Background(), th.SystemAdminUser.Email, th.SystemAdminUser.Password)
require.NoError(t, err)
ipFilters, r, err := th.Client.GetIPFilters(context.Background()) ipFilters, r, err := th.Client.GetIPFilters(context.Background())
require.Error(t, err) require.Error(t, err)
@@ -155,9 +161,11 @@ func Test_applyIPFilters(t *testing.T) {
ipFiltering := &mocks.IPFilteringInterface{} ipFiltering := &mocks.IPFilteringInterface{}
th.App.Srv().IPFiltering = ipFiltering th.App.Srv().IPFiltering = ipFiltering
th.App.Srv().RemoveLicense() appErr := th.App.Srv().RemoveLicense()
require.Nil(t, appErr)
th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password) _, _, err := th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password)
require.NoError(t, err)
ipFilters, r, err := th.Client.ApplyIPFilters(context.Background(), allowedRanges) ipFilters, r, err := th.Client.ApplyIPFilters(context.Background(), allowedRanges)
require.Error(t, err) require.Error(t, err)
@@ -175,7 +183,8 @@ func Test_applyIPFilters(t *testing.T) {
th.App.Srv().IPFiltering = ipFiltering th.App.Srv().IPFiltering = ipFiltering
th.App.Srv().SetLicense(lic) th.App.Srv().SetLicense(lic)
th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password) _, _, err := th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password)
require.NoError(t, err)
ipFilters, r, err := th.Client.ApplyIPFilters(context.Background(), allowedRanges) ipFilters, r, err := th.Client.ApplyIPFilters(context.Background(), allowedRanges)
require.Error(t, err) require.Error(t, err)
@@ -188,7 +197,8 @@ func Test_applyIPFilters(t *testing.T) {
th := Setup(t).InitBasic() th := Setup(t).InitBasic()
defer th.TearDown() defer th.TearDown()
th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password) _, _, err := th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password)
require.NoError(t, err)
ipFiltering := &mocks.IPFilteringInterface{} ipFiltering := &mocks.IPFilteringInterface{}
th.App.Srv().IPFiltering = ipFiltering th.App.Srv().IPFiltering = ipFiltering
@@ -223,7 +233,8 @@ func Test_applyIPFilters(t *testing.T) {
th.App.Srv().Cloud = cloud th.App.Srv().Cloud = cloud
th.Client.Login(context.Background(), th.SystemAdminUser.Email, th.SystemAdminUser.Password) _, _, err := th.Client.Login(context.Background(), th.SystemAdminUser.Email, th.SystemAdminUser.Password)
require.NoError(t, err)
ipFilters, r, err := th.Client.ApplyIPFilters(context.Background(), allowedRanges) ipFilters, r, err := th.Client.ApplyIPFilters(context.Background(), allowedRanges)
require.NoError(t, err) require.NoError(t, err)
@@ -255,9 +266,11 @@ func Test_getMyIP(t *testing.T) {
ipFiltering := &mocks.IPFilteringInterface{} ipFiltering := &mocks.IPFilteringInterface{}
th.App.Srv().IPFiltering = ipFiltering th.App.Srv().IPFiltering = ipFiltering
th.App.Srv().RemoveLicense() appErr := th.App.Srv().RemoveLicense()
require.Nil(t, appErr)
th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password) _, _, err := th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password)
require.NoError(t, err)
myIP, r, err := th.Client.GetMyIP(context.Background()) myIP, r, err := th.Client.GetMyIP(context.Background())
require.Error(t, err) require.Error(t, err)
@@ -271,7 +284,8 @@ func Test_getMyIP(t *testing.T) {
th := Setup(t).InitBasic() th := Setup(t).InitBasic()
defer th.TearDown() defer th.TearDown()
th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password) _, _, err := th.Client.Login(context.Background(), th.BasicUser.Email, th.BasicUser.Password)
require.NoError(t, err)
ipFiltering := &mocks.IPFilteringInterface{} ipFiltering := &mocks.IPFilteringInterface{}
th.App.Srv().IPFiltering = ipFiltering th.App.Srv().IPFiltering = ipFiltering