elebedeva/frostfs-node
1
0
Fork 0
forked from TrueCloudLab/frostfs-node
Code Pull requests Activity

[#438] metabase: Write unit tests for checking the absence of payloads

Browse source 
Metabase should not store payloads of objects. Set payload in generated test
object. Ascertain that objects returned by Get method have no payload.

Signed-off-by: Leonard Lyubich <leonard@nspcc.ru>
This commit is contained in:
Leonard Lyubich 2021-03-22 19:44:57 +03:00 committed by Alex Vanin
parent 8bdffa2dc5
commit e3f8470d7a
2 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
pkg/local_object_storage/metabase/db_test.go
View file

@ -95,6 +95,7 @@ func generateRawObjectWithCID(t *testing.T, cid *container.ID) *object.RawObject
obj.SetVersion(version) obj.SetVersion(version)
obj.SetPayloadChecksum(csum) obj.SetPayloadChecksum(csum)
obj.SetPayloadHomomorphicHash(csumTZ) obj.SetPayloadHomomorphicHash(csumTZ)
obj.SetPayload([]byte{1, 2, 3, 4, 5})
return obj return obj
} }

10
pkg/local_object_storage/metabase/get_test.go
View file

@ -31,7 +31,7 @@ func TestDB_Get(t *testing.T) {
newObj, err := meta.Get(db, raw.Object().Address()) newObj, err := meta.Get(db, raw.Object().Address())
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, raw.Object(), newObj) require.Equal(t, raw.CutPayload().Object(), newObj)
}) })
t.Run("put tombstone object", func(t *testing.T) { t.Run("put tombstone object", func(t *testing.T) {
@ -43,7 +43,7 @@ func TestDB_Get(t *testing.T) {
newObj, err := meta.Get(db, raw.Object().Address()) newObj, err := meta.Get(db, raw.Object().Address())
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, raw.Object(), newObj) require.Equal(t, raw.CutPayload().Object(), newObj)
}) })
t.Run("put storage group object", func(t *testing.T) { t.Run("put storage group object", func(t *testing.T) {
@ -55,7 +55,7 @@ func TestDB_Get(t *testing.T) {
newObj, err := meta.Get(db, raw.Object().Address()) newObj, err := meta.Get(db, raw.Object().Address())
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, raw.Object(), newObj) require.Equal(t, raw.CutPayload().Object(), newObj)
}) })
t.Run("put virtual object", func(t *testing.T) { t.Run("put virtual object", func(t *testing.T) {
@ -87,11 +87,11 @@ func TestDB_Get(t *testing.T) {
newParent, err := meta.GetRaw(db, parent.Object().Address(), false) newParent, err := meta.GetRaw(db, parent.Object().Address(), false)
require.NoError(t, err) require.NoError(t, err)
require.True(t, binaryEqual(parent.Object(), newParent)) require.True(t, binaryEqual(parent.CutPayload().Object(), newParent))
newChild, err := meta.GetRaw(db, child.Object().Address(), true) newChild, err := meta.GetRaw(db, child.Object().Address(), true)
require.NoError(t, err) require.NoError(t, err)
require.True(t, binaryEqual(child.Object(), newChild)) require.True(t, binaryEqual(child.CutPayload().Object(), newChild))
}) })
} }