fix: small fixes from self-review

ethersphere · Aug 19, 2021 · 0b481e0 · 0b481e0
1 parent c2c16a3
commit 0b481e0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/pkg/debugapi/postage_test.go b/pkg/debugapi/postage_test.go
@@ -639,7 +639,7 @@ func TestPostageAccessHandler(t *testing.T) {
 		{
 			name:     "dilute batch ok",
 			method:   http.MethodPatch,
-			url:      fmt.Sprintf("/stamps/dilute/%s/10", batchOkStr),
+			url:      fmt.Sprintf("/stamps/dilute/%s/18", batchOkStr),
 			respCode: http.StatusAccepted,
 			resp: &debugapi.PostageCreateResponse{
 				BatchID: batchOk,
@@ -671,7 +671,7 @@ func TestPostageAccessHandler(t *testing.T) {
 		{
 			name:     "dilute batch not ok",
 			method:   http.MethodPatch,
-			url:      fmt.Sprintf("/stamps/dilute/%s/10", batchOkStr),
+			url:      fmt.Sprintf("/stamps/dilute/%s/18", batchOkStr),
 			respCode: http.StatusTooManyRequests,
 			resp: &jsonhttp.StatusResponse{
 				Code:    http.StatusTooManyRequests,

diff --git a/pkg/postage/postagecontract/contract.go b/pkg/postage/postagecontract/contract.go
@@ -232,7 +232,7 @@ func (c *postageContract) CreateBatch(ctx context.Context, initialBalance *big.I
 	}
 
 	for _, ev := range receipt.Logs {
-		if ev.Address == c.postageContractAddress && ev.Topics[0] == batchCreatedTopic {
+		if ev.Address == c.postageContractAddress && len(ev.Topics) > 0 && ev.Topics[0] == batchCreatedTopic {
 			var createdEvent batchCreatedEvent
 			err = transaction.ParseEvent(&postageStampABI, "BatchCreated", &createdEvent, *ev)
 			if err != nil {
@@ -312,7 +312,7 @@ func (c *postageContract) DiluteBatch(ctx context.Context, batchID []byte, newDe
 	}
 
 	for _, ev := range receipt.Logs {
-		if ev.Address == c.postageContractAddress && ev.Topics[0] == batchDiluteTopic {
+		if ev.Address == c.postageContractAddress && len(ev.Topics) > 0 && ev.Topics[0] == batchDiluteTopic {
 			return nil
 		}
 	}