From 4e590b3a9f7214c19bd0addb88893fa781400ca0 Mon Sep 17 00:00:00 2001 From: evpost Date: Thu, 9 Nov 2023 18:11:00 +0300 Subject: [PATCH] fix linter --- internal/client/config/config.go | 5 +++++ internal/client/config/config_test.go | 1 + internal/client/usecase/create.go | 2 +- internal/client/usecase/update.go | 2 +- 4 files changed, 8 insertions(+), 2 deletions(-) diff --git a/internal/client/config/config.go b/internal/client/config/config.go index 106cad3..b1f4e21 100644 --- a/internal/client/config/config.go +++ b/internal/client/config/config.go @@ -16,7 +16,10 @@ const ( defaultServerAddress = "127.0.0.1:50051" ) +//nolint:gochecknoglobals var flags Flags + +//nolint:gochecknoglobals var once sync.Once func GetConfig() (Config, error) { @@ -32,6 +35,7 @@ func GetConfig() (Config, error) { if _, err := os.Stat(configPath); os.IsNotExist(err) { // Возвращаем ошибку, если файл конфигурации не существует + //nolint:goerr113 return Config{}, fmt.Errorf("config file not found: %s", configPath) } else if err != nil { // Возвращаем ошибку, если есть другая ошибка при проверке файла @@ -41,6 +45,7 @@ func GetConfig() (Config, error) { // Попытка парсинга файла конфигурации if err := parseConfig(configPath, &fileCfg); err != nil { fmt.Printf("failed to read yaml config file: %v\n", err) + return Config{}, err } diff --git a/internal/client/config/config_test.go b/internal/client/config/config_test.go index a31dfb6..2c50c8c 100644 --- a/internal/client/config/config_test.go +++ b/internal/client/config/config_test.go @@ -83,6 +83,7 @@ server_address: "127.0.0.1:8080" t.Run(tt.name, func(t *testing.T) { // Если указаны данные конфигурации, записываем их в файл if len(tt.configFileData) > 0 { + //nolint:gosec if err := os.WriteFile(tt.configFile, tt.configFileData, 0644); err != nil { t.Fatalf("Failed to write to config file: %v", err) } diff --git a/internal/client/usecase/create.go b/internal/client/usecase/create.go index 3076308..00623d1 100644 --- a/internal/client/usecase/create.go +++ b/internal/client/usecase/create.go @@ -1,4 +1,3 @@ -//nolint:ireturn package usecase import ( @@ -37,6 +36,7 @@ func (c *ClientUsecase) createSecret(secretType int, success bool) { } } +//nolint:ireturn,nolintlint func (c *ClientUsecase) getSecretrData(secretType int) (filemanager.Data, models.DataInfo, error) { var data filemanager.Data var filePath string diff --git a/internal/client/usecase/update.go b/internal/client/usecase/update.go index 154f803..5679c39 100644 --- a/internal/client/usecase/update.go +++ b/internal/client/usecase/update.go @@ -1,4 +1,3 @@ -//nolint:ireturn package usecase import ( @@ -54,6 +53,7 @@ func (c *ClientUsecase) updateMetaInfo(secretName string, metaInfo models.DataIn return nil } +//nolint:ireturn,nolintlint func (c *ClientUsecase) getUpdatedData(secretName string, dataType int) (filemanager.Data, error) { var data filemanager.Data var err error