diff --git a/pkg/permission/ruleset.go b/pkg/permission/ruleset.go index 342075e642c41ea29a734245c5ac415c93197e8b..538e1a47b455404f6d4dd6d41ec957cbfc943a66 100644 --- a/pkg/permission/ruleset.go +++ b/pkg/permission/ruleset.go @@ -22,9 +22,9 @@ type Rule struct { Access Access `json:"access" bson:"access,omitempty"` // Поля не передаются API клиенту HiddenFields []string `json:"hiddenFields,omitempty" bson:"hiddenFields,omitempty"` - // Запрещено чтение указанных полей + // Клиент не может сохранять данные поля DenyReadFields []string `json:"denyReadFields,omitempty" bson:"denyReadFields,omitempty"` - // Запрещена запись в указанные поля + // Клиент может сохранить данные поля, но поля не передаются в API DenyWriteFields []string `json:"denyWriteFields,omitempty" bson:"denyWriteFields,omitempty"` // Дополнительный фильтр ReadFilter string `json:"readFilter,omitempty" bson:"readFilter,omitempty"` diff --git a/pkg/setup/setup.go b/pkg/setup/setup.go index 96a6cc1086e9fa7b5bb07aee69f6eb2b84325304..76eec9737154d10078c75e120db84de947568a04 100644 --- a/pkg/setup/setup.go +++ b/pkg/setup/setup.go @@ -33,7 +33,7 @@ type Setup struct { force bool remove bool - waitForSpaceReady bool + waitSpaceAvailable bool //attempts uint //delay time.Duration @@ -76,16 +76,12 @@ func (s *Setup) IsRemove() bool { return s.remove } -func (s *Setup) WithWaitForSpaceReady() *Setup { +func (s *Setup) WithWaitSpaceAvailable() *Setup { setup := *s - setup.waitForSpaceReady = true + setup.waitSpaceAvailable = true return &setup } -func (s *Setup) IsWaitForSpace() bool { - return s.waitForSpaceReady -} - func (s *Setup) HasErrors() bool { return len(s.errors) > 0 } @@ -161,7 +157,7 @@ func (s *Setup) AddItem(item *items.Item, opt ...ItemsOption) *Setup { // Install выполняет установку необходимых требований func (s *Setup) Install(ctx context.Context) error { - if s.waitForSpaceReady { + if s.waitSpaceAvailable { if err := spaces.WaitSpaceAvailable(ctx, s.content.Spaces, s.SpaceID, s.logger); err != nil { return err } @@ -183,7 +179,7 @@ func (s *Setup) Install(ctx context.Context) error { // Check выполняет проверку требований func (s *Setup) Check(ctx context.Context) error { - if s.waitForSpaceReady { + if s.waitSpaceAvailable { if err := spaces.WaitReadAvailable(ctx, s.content.Spaces, s.SpaceID, s.logger); err != nil { return err } @@ -205,7 +201,7 @@ func (s *Setup) Check(ctx context.Context) error { // Uninstall выполняет удаление установленных раннее требований func (s *Setup) Uninstall(ctx context.Context) error { - if s.waitForSpaceReady { + if s.waitSpaceAvailable { if err := spaces.WaitSpaceAvailable(ctx, s.content.Spaces, s.SpaceID, s.logger); err != nil { return err } diff --git a/pkg/setup/setup_test.go b/pkg/setup/setup_test.go index a3729263b0eab70e670658d890f12ccccda49cfa..cebbbde677dcaab305358868398e14d1a37a6ca0 100644 --- a/pkg/setup/setup_test.go +++ b/pkg/setup/setup_test.go @@ -672,7 +672,7 @@ func TestSetupInstall(t *testing.T) { sp.On("Get", mock.Anything, mock.Anything).Return(sps, nil).Once() setup := NewSetup(&content.Content{Spaces: sp}, spaceID, envID, logger) - err := setup.WithWaitForSpaceReady().Install(context.Background()) + err := setup.WithWaitSpaceAvailable().Install(context.Background()) require.NoError(t, err) @@ -685,7 +685,7 @@ func TestSetupInstall(t *testing.T) { sp.On("Get", mock.Anything, mock.Anything).Return(&spaces.Space{ID: spaceID, OrgID: "org", StateInfo: &spaces.StateInfo{State: spaces.StateMigration}}, nil).Twice() sp.On("Get", mock.Anything, mock.Anything).Return(nil, errors.New("some error")).Once() setup := NewSetup(&content.Content{Spaces: sp}, spaceID, envID, logger) - err := setup.WithWaitForSpaceReady().Install(context.Background()) + err := setup.WithWaitSpaceAvailable().Install(context.Background()) require.Error(t, err, "в момент выполнения пришла ошибка отличная от 'space not available'") @@ -1056,7 +1056,7 @@ func TestSetupUninstall(t *testing.T) { setup := NewSetup(&content.Content{Spaces: sp}, spaceID, envID, logger) - err := setup.WithWaitForSpaceReady().Uninstall(context.Background()) + err := setup.WithWaitSpaceAvailable().Uninstall(context.Background()) require.NoError(t, err) @@ -1072,7 +1072,7 @@ func TestSetupUninstall(t *testing.T) { setup := NewSetup(&content.Content{Spaces: sp}, spaceID, envID, logger) - err := setup.WithWaitForSpaceReady().Uninstall(context.Background()) + err := setup.WithWaitSpaceAvailable().Uninstall(context.Background()) require.Error(t, err, "в момент выполнения пришла ошибка отличная от 'space not available'") }) @@ -1280,7 +1280,7 @@ func TestSetupCheck(t *testing.T) { setup := NewSetup(&content.Content{Spaces: sp}, spaceID, envID, logger) - err := setup.WithWaitForSpaceReady().Check(context.Background()) + err := setup.WithWaitSpaceAvailable().Check(context.Background()) require.NoError(t, err) @@ -1296,7 +1296,7 @@ func TestSetupCheck(t *testing.T) { setup := NewSetup(&content.Content{Spaces: sp}, spaceID, envID, logger) - err := setup.WithWaitForSpaceReady().Check(context.Background()) + err := setup.WithWaitSpaceAvailable().Check(context.Background()) require.Error(t, err, "в момент выполнения пришла ошибка отличная от 'Space is migrating'") })