[#275] client: Return status from all methods

Since status is checked first in handleError method, it should be returned from client methods

Signed-off-by: Marina Biryukova <m.biryukova@yadro.com>

client/netmap.go

@@ -239,14 +239,10 @@ func (c *Client) NetMapSnapshot(ctx context.Context, _ PrmNetMapSnapshot) (*ResN
 
 	var res ResNetMapSnapshot
 	res.st, err = c.processResponse(resp)
-	if err != nil {
+	if err != nil || !apistatus.IsSuccessful(res.st) {
 		return &res, err
 	}
 
-	if !apistatus.IsSuccessful(res.st) {
-		return &res, nil
-	}
-
 	const fieldNetMap = "network map"
 
 	netMapV2 := resp.GetBody().NetMap()

client/object_delete.go

@@ -148,14 +148,10 @@ func (c *Client) ObjectDelete(ctx context.Context, prm PrmObjectDelete) (*ResObj
 
 	var res ResObjectDelete
 	res.st, err = c.processResponse(resp)
-	if err != nil {
+	if err != nil || !apistatus.IsSuccessful(res.st) {
 		return &res, err
 	}
 
-	if !apistatus.IsSuccessful(res.st) {
-		return &res, nil
-	}
-
 	const fieldTombstone = "tombstone"
 
 	idTombV2 := resp.GetBody().GetTombstone().GetObjectID()

client/object_get.go

@@ -492,14 +492,10 @@ func (c *Client) ObjectHead(ctx context.Context, prm PrmObjectHead) (*ResObjectH
 
 	var res ResObjectHead
 	res.st, err = c.processResponse(resp)
-	if err != nil {
+	if err != nil || !apistatus.IsSuccessful(res.st) {
 		return &res, err
 	}
 
-	if !apistatus.IsSuccessful(res.st) {
-		return &res, nil
-	}
-
 	res.idObj = *prm.ObjectID
 
 	switch v := resp.GetBody().GetHeaderPart().(type) {

client/object_hash.go

@@ -189,14 +189,10 @@ func (c *Client) ObjectHash(ctx context.Context, prm PrmObjectHash) (*ResObjectH
 
 	var res ResObjectHash
 	res.st, err = c.processResponse(resp)
-	if err != nil {
+	if err != nil || !apistatus.IsSuccessful(res.st) {
 		return &res, err
 	}
 
-	if !apistatus.IsSuccessful(res.st) {
-		return &res, nil
-	}
-
 	res.checksums = resp.GetBody().GetHashList()
 	if len(res.checksums) == 0 {
 		return nil, newErrMissingResponseField("hash list")

client/object_patch.go

@@ -246,14 +246,10 @@ func (x *objectPatcher) Close(_ context.Context) (*ResObjectPatch, error) {
 	}
 
 	x.res.st, x.err = x.client.processResponse(&x.respV2)
-	if x.err != nil {
+	if x.err != nil || !apistatus.IsSuccessful(x.res.st) {
 		return &x.res, x.err
 	}
 
-	if !apistatus.IsSuccessful(x.res.st) {
-		return &x.res, nil
-	}
-
 	const fieldID = "ID"
 
 	idV2 := x.respV2.Body.ObjectID

client/object_put_raw.go

@@ -156,14 +156,10 @@ func (x *objectWriterRaw) Close(_ context.Context) (*ResObjectPut, error) {
 	}
 
 	x.res.st, x.err = x.client.processResponse(&x.respV2)
-	if x.err != nil {
+	if x.err != nil || !apistatus.IsSuccessful(x.res.st) {
 		return &x.res, x.err
 	}
 
-	if !apistatus.IsSuccessful(x.res.st) {
-		return &x.res, nil
-	}
-
 	const fieldID = "ID"
 
 	idV2 := x.respV2.GetBody().GetObjectID()