diff --git a/pkg/container/fmt.go b/pkg/container/fmt.go deleted file mode 100644 index ad1cadab..00000000 --- a/pkg/container/fmt.go +++ /dev/null @@ -1,28 +0,0 @@ -package container - -import ( - "fmt" - - "github.com/nspcc-dev/neofs-api-go/pkg" - "github.com/nspcc-dev/neofs-api-go/v2/container" -) - -// NewVerifiedFromV2 constructs Container from NeoFS API V2 Container message. -// Returns error if message does not meet NeoFS API V2 specification. -// -// Additionally checks if message carries supported version. -func NewVerifiedFromV2(cnrV2 *container.Container) (*Container, error) { - cnr := NewContainerFromV2(cnrV2) - - // check version support - if err := pkg.IsSupportedVersion(cnr.Version()); err != nil { - return nil, err - } - - // check nonce format - if _, err := cnr.NonceUUID(); err != nil { - return nil, fmt.Errorf("invalid nonce: %w", err) - } - - return cnr, nil -} diff --git a/pkg/container/fmt_test.go b/pkg/container/fmt_test.go deleted file mode 100644 index c27bdde4..00000000 --- a/pkg/container/fmt_test.go +++ /dev/null @@ -1,49 +0,0 @@ -package container_test - -import ( - "testing" - - "github.com/google/uuid" - "github.com/nspcc-dev/neofs-api-go/pkg" - "github.com/nspcc-dev/neofs-api-go/pkg/container" - containerV2 "github.com/nspcc-dev/neofs-api-go/v2/container" - "github.com/stretchr/testify/require" -) - -func TestNewVerifiedFromV2(t *testing.T) { - cnrV2 := new(containerV2.Container) - - errAssert := func() { - _, err := container.NewVerifiedFromV2(cnrV2) - require.Error(t, err) - } - - // set unsupported version - v := pkg.SDKVersion() - v.SetMajor(0) - require.Error(t, pkg.IsSupportedVersion(v)) - cnrV2.SetVersion(v.ToV2()) - - errAssert() - - // set supported version - v.SetMajor(2) - require.NoError(t, pkg.IsSupportedVersion(v)) - cnrV2.SetVersion(v.ToV2()) - - errAssert() - - // set invalid nonce - nonce := []byte{1, 2, 3} - cnrV2.SetNonce(nonce) - - errAssert() - - // set valid nonce - uid := uuid.New() - data, _ := uid.MarshalBinary() - cnrV2.SetNonce(data) - - _, err := container.NewVerifiedFromV2(cnrV2) - require.NoError(t, err) -} diff --git a/pkg/version.go b/pkg/version.go index a8513870..928566c0 100644 --- a/pkg/version.go +++ b/pkg/version.go @@ -74,17 +74,6 @@ func (v *Version) String() string { return fmt.Sprintf("v%d.%d", v.Major(), v.Minor()) } -// IsSupportedVersion returns error if v is not supported by current SDK. -func IsSupportedVersion(v *Version) error { - mjr, mnr := v.Major(), v.Minor() - - if mjr != 2 || mnr > sdkMnr { - return fmt.Errorf("unsupported version %d.%d", mjr, mnr) - } - - return nil -} - // Marshal marshals Version into a protobuf binary form. // // Buffer is allocated when the argument is empty. diff --git a/pkg/version_test.go b/pkg/version_test.go index 4586492e..622e4b7e 100644 --- a/pkg/version_test.go +++ b/pkg/version_test.go @@ -47,39 +47,6 @@ func TestSDKVersion(t *testing.T) { require.Equal(t, uint32(sdkMnr), v.Minor()) } -func TestIsSupportedVersion(t *testing.T) { - require.Error(t, IsSupportedVersion(nil)) - - v := NewVersion() - - v.SetMajor(1) - require.Error(t, IsSupportedVersion(v)) - - v.SetMajor(3) - require.Error(t, IsSupportedVersion(v)) - - for _, item := range []struct { - mjr, maxMnr uint32 - }{ - { - mjr: 2, - maxMnr: sdkMnr, - }, - } { - v.SetMajor(item.mjr) - - for i := uint32(0); i <= item.maxMnr; i++ { - v.SetMinor(i) - - require.NoError(t, IsSupportedVersion(v)) - } - - v.SetMinor(item.maxMnr + 1) - - require.Error(t, IsSupportedVersion(v)) - } -} - func TestVersionEncoding(t *testing.T) { v := NewVersion() v.SetMajor(1)