diff --git a/pkg/setup/config_test.go b/pkg/setup/config_test.go index 60f039164268fa8c17a870372abcf1940d6976f1..4934575be5672a878c7ee6daebb1a8dbd4739207 100644 --- a/pkg/setup/config_test.go +++ b/pkg/setup/config_test.go @@ -12,7 +12,7 @@ import ( ) func TestConfig_WithCollectionOptions(t *testing.T) { - t.Run("Option set in setup config with filter", func(t *testing.T) { + t.Run("With filter", func(t *testing.T) { // создаем конфигурацию config := &Config{Collections: []CollectionConfig{ {collection: &collections.Collection{ID: "1", SpaceID: "sp", EnvID: "env"}}, @@ -21,13 +21,13 @@ func TestConfig_WithCollectionOptions(t *testing.T) { // применяем опцию OverwriteCollection к конфигурации и дополнительно добавляем фильтр (опция применится только к коллекции с id 1) config.WithCollectionOptions(func(c *collections.Collection) bool { return c.ID == "1" }, OverwriteCollection()) - assert.NotNil(t, config.Collections[0].UpdateFn, "должна быть установлена опция OverwriteCollection с учетом переданного фильтра, до правок конфигурация оставалась прежней") - assert.Nil(t, config.Collections[1].UpdateFn, "опция не установлена, так как не удовлетворяет фильтру") + assert.NotNil(t, config.Collections[0].UpdateFn, "должна быть выполнена OverwriteCollection и установлена UpdateFn для коллекции ID:1") + assert.Nil(t, config.Collections[1].UpdateFn) }) } func TestConfig_WithItemsOptions(t *testing.T) { - t.Run("Option set in setup config with filter", func(t *testing.T) { + t.Run("With filter", func(t *testing.T) { // создаем конфигурацию config := &Config{Items: []ItemConfig{ {item: &items.Item{ID: "1", SpaceID: "sp", EnvID: "env", CollectionID: "coll"}}, @@ -36,13 +36,13 @@ func TestConfig_WithItemsOptions(t *testing.T) { // применяем опцию OverwriteItem к конфигурации и дополнительно добавляем фильтр (опция применится только к коллекции с id 1) config.WithItemsOptions(func(c *items.Item) bool { return c.ID == "1" }, OverwriteItem()) - assert.NotNil(t, config.Items[0].UpdateFn, "должна быть установлена опция OverwriteItem с учетом переданного фильтра, до правок конфигурация оставалась прежней") - assert.Nil(t, config.Items[1].UpdateFn, "опция не установлена, так как не удовлетворяет фильтру") + assert.NotNil(t, config.Items[0].UpdateFn, "должна быть выполнена OverwriteItem и установлена UpdateFn для элемента ID: 1") + assert.Nil(t, config.Items[1].UpdateFn) }) } func TestConfig_WithRoleOptions(t *testing.T) { - t.Run("Option set in setup config with filter", func(t *testing.T) { + t.Run("With filter", func(t *testing.T) { // создаем конфигурацию config := &Config{Roles: []RoleConfig{ {role: &roles.Role{ID: "1", SpaceID: "sp"}}, @@ -51,13 +51,13 @@ func TestConfig_WithRoleOptions(t *testing.T) { // применяем опцию OverwriteRole к конфигурации и дополнительно добавляем фильтр (опция применится только к коллекции с id 1) config.WithRolesOptions(func(c *roles.Role) bool { return c.ID == "1" }, OverwriteRole()) - assert.NotNil(t, config.Roles[0].UpdateFn, "должна быть установлена опция OverwriteRole с учетом переданного фильтра, до правок конфигурация оставалась прежней") - assert.Nil(t, config.Roles[1].UpdateFn, "опция не установлена, так как не удовлетворяет фильтру") + assert.NotNil(t, config.Roles[0].UpdateFn, "должна быть выполнена опция OverwriteRole") + assert.Nil(t, config.Roles[1].UpdateFn) }) } func TestConfig_WithClientOptions(t *testing.T) { - t.Run("Option set in setup config with filter", func(t *testing.T) { + t.Run("With filter", func(t *testing.T) { // создаем конфигурацию config := &Config{Clients: []ClientConfig{ {client: &clients.Client{ID: "1", SpaceID: "sp"}}, @@ -66,8 +66,8 @@ func TestConfig_WithClientOptions(t *testing.T) { // применяем опцию OverwriteClient к конфигурации и дополнительно добавляем фильтр (опция применится только к коллекции с id 1) config.WithClientsOptions(func(c *clients.Client) bool { return c.ID == "1" }, OverwriteClient()) - assert.NotNil(t, config.Clients[0].UpdateFn, "должна быть установлена опция OverwriteClient с учетом переданного фильтра, до правок конфигурация оставалась прежней") - assert.Nil(t, config.Clients[1].UpdateFn, "опция не установлена, так как не удовлетворяет фильтру") + assert.NotNil(t, config.Clients[0].UpdateFn, "должна выполнена опция OverwriteClient") + assert.Nil(t, config.Clients[1].UpdateFn) }) }