From c09efe32154170649ba0a4d53d80aa51f974a4bd Mon Sep 17 00:00:00 2001 From: ko_oler <kooler89@gmail.com> Date: Wed, 20 Dec 2023 12:57:36 +0300 Subject: [PATCH] =?UTF-8?q?=D0=BF=D1=80=D0=B0=D0=B2=D0=BA=D0=B0=20=D0=BF?= =?UTF-8?q?=D0=BE=20=D0=9F=D0=A0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- pkg/permission/ruleset.go | 4 ++-- pkg/setup/setup.go | 16 ++++++---------- pkg/setup/setup_test.go | 12 ++++++------ 3 files changed, 14 insertions(+), 18 deletions(-) diff --git a/pkg/permission/ruleset.go b/pkg/permission/ruleset.go index 342075e6..538e1a47 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 96a6cc10..76eec973 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 a3729263..cebbbde6 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'") }) -- GitLab