Merge branch 'master' into fix/PRXS-2779-ConfigAppend

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)
 	})
 }