acid-ant/frostfs-contract
1
0
Fork 0
forked from TrueCloudLab/frostfs-contract
Code Issues Pull requests Projects Releases Packages Wiki Activity

[#142] container: do not fail when deleting missing container

Browse source 
Signed-off-by: Evgenii Stratonikov <evgeniy@nspcc.ru>
This commit is contained in:
Evgenii Stratonikov 2021-10-25 15:43:57 +03:00 committed by Alex Vanin
parent 6bd088aa9f
commit 2cb10edfd2
2 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
container/container_contract.go
View file

@ -303,7 +303,7 @@ func Delete(containerID []byte, signature interop.Signature, token []byte) {
ownerID := getOwnerByID(ctx, containerID)
if ownerID == nil {
panic(NotFoundError)
return
}
if notaryDisabled {

7
tests/container_test.go
View file

@ -185,6 +185,13 @@ func TestContainerDelete(t *testing.T) {
tx = PrepareInvoke(t, bc, CommitteeAcc, h, "delete", c.id[:], c.sig, c.token)
AddBlockCheckHalt(t, bc, tx)
t.Run("missing container", func(t *testing.T) {
id := c.id
id[0] ^= 0xFF
tx = PrepareInvoke(t, bc, CommitteeAcc, h, "delete", id[:], c.sig, c.token)
AddBlockCheckHalt(t, bc, tx)
})
tx = PrepareInvoke(t, bc, acc, h, "get", c.id[:])
_, err := TestInvoke(bc, tx)
require.Error(t, err)