common: improve GraphQL error messages (#20354)

pull/20359/head
Felix Lange 5 years ago committed by Péter Szilágyi
parent 0754100464
commit 0ec5ab4175
  1. 52
      common/bytes_test.go
  2. 25
      common/main_test.go
  3. 9
      common/types.go

@ -19,41 +19,43 @@ package common
import ( import (
"bytes" "bytes"
"testing" "testing"
checker "gopkg.in/check.v1"
) )
type BytesSuite struct{} func TestCopyBytes(t *testing.T) {
input := []byte{1, 2, 3, 4}
var _ = checker.Suite(&BytesSuite{})
func (s *BytesSuite) TestCopyBytes(c *checker.C) { v := CopyBytes(input)
data1 := []byte{1, 2, 3, 4} if !bytes.Equal(v, []byte{1, 2, 3, 4}) {
exp1 := []byte{1, 2, 3, 4} t.Fatal("not equal after copy")
res1 := CopyBytes(data1) }
c.Assert(res1, checker.DeepEquals, exp1) v[0] = 99
if bytes.Equal(v, input) {
t.Fatal("result is not a copy")
}
} }
func (s *BytesSuite) TestLeftPadBytes(c *checker.C) { func TestLeftPadBytes(t *testing.T) {
val1 := []byte{1, 2, 3, 4} val := []byte{1, 2, 3, 4}
exp1 := []byte{0, 0, 0, 0, 1, 2, 3, 4} padded := []byte{0, 0, 0, 0, 1, 2, 3, 4}
res1 := LeftPadBytes(val1, 8)
res2 := LeftPadBytes(val1, 2)
c.Assert(res1, checker.DeepEquals, exp1) if r := LeftPadBytes(val, 8); !bytes.Equal(r, padded) {
c.Assert(res2, checker.DeepEquals, val1) t.Fatalf("LeftPadBytes(%v, 8) == %v", val, r)
}
if r := LeftPadBytes(val, 2); !bytes.Equal(r, val) {
t.Fatalf("LeftPadBytes(%v, 2) == %v", val, r)
}
} }
func (s *BytesSuite) TestRightPadBytes(c *checker.C) { func TestRightPadBytes(t *testing.T) {
val := []byte{1, 2, 3, 4} val := []byte{1, 2, 3, 4}
exp := []byte{1, 2, 3, 4, 0, 0, 0, 0} padded := []byte{1, 2, 3, 4, 0, 0, 0, 0}
resstd := RightPadBytes(val, 8)
resshrt := RightPadBytes(val, 2)
c.Assert(resstd, checker.DeepEquals, exp) if r := RightPadBytes(val, 8); !bytes.Equal(r, padded) {
c.Assert(resshrt, checker.DeepEquals, val) t.Fatalf("RightPadBytes(%v, 8) == %v", val, r)
}
if r := RightPadBytes(val, 2); !bytes.Equal(r, val) {
t.Fatalf("RightPadBytes(%v, 2) == %v", val, r)
}
} }
func TestFromHex(t *testing.T) { func TestFromHex(t *testing.T) {

@ -1,25 +0,0 @@
// Copyright 2014 The go-ethereum Authors
// This file is part of the go-ethereum library.
//
// The go-ethereum library is free software: you can redistribute it and/or modify
// it under the terms of the GNU Lesser General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// The go-ethereum library is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU Lesser General Public License for more details.
//
// You should have received a copy of the GNU Lesser General Public License
// along with the go-ethereum library. If not, see <http://www.gnu.org/licenses/>.
package common
import (
"testing"
checker "gopkg.in/check.v1"
)
func Test(t *testing.T) { checker.TestingT(t) }

@ -20,6 +20,7 @@ import (
"database/sql/driver" "database/sql/driver"
"encoding/hex" "encoding/hex"
"encoding/json" "encoding/json"
"errors"
"fmt" "fmt"
"math/big" "math/big"
"math/rand" "math/rand"
@ -142,7 +143,7 @@ func (h Hash) Value() (driver.Value, error) {
} }
// ImplementsGraphQLType returns true if Hash implements the specified GraphQL type. // ImplementsGraphQLType returns true if Hash implements the specified GraphQL type.
func (_ Hash) ImplementsGraphQLType(name string) bool { return name == "Bytes32" } func (Hash) ImplementsGraphQLType(name string) bool { return name == "Bytes32" }
// UnmarshalGraphQL unmarshals the provided GraphQL query data. // UnmarshalGraphQL unmarshals the provided GraphQL query data.
func (h *Hash) UnmarshalGraphQL(input interface{}) error { func (h *Hash) UnmarshalGraphQL(input interface{}) error {
@ -151,7 +152,7 @@ func (h *Hash) UnmarshalGraphQL(input interface{}) error {
case string: case string:
err = h.UnmarshalText([]byte(input)) err = h.UnmarshalText([]byte(input))
default: default:
err = fmt.Errorf("Unexpected type for Bytes32: %v", input) err = fmt.Errorf("unexpected type %T for Hash", input)
} }
return err return err
} }
@ -290,7 +291,7 @@ func (a *Address) UnmarshalGraphQL(input interface{}) error {
case string: case string:
err = a.UnmarshalText([]byte(input)) err = a.UnmarshalText([]byte(input))
default: default:
err = fmt.Errorf("Unexpected type for Address: %v", input) err = fmt.Errorf("unexpected type %T for Address", input)
} }
return err return err
} }
@ -323,7 +324,7 @@ func NewMixedcaseAddress(addr Address) MixedcaseAddress {
// NewMixedcaseAddressFromString is mainly meant for unit-testing // NewMixedcaseAddressFromString is mainly meant for unit-testing
func NewMixedcaseAddressFromString(hexaddr string) (*MixedcaseAddress, error) { func NewMixedcaseAddressFromString(hexaddr string) (*MixedcaseAddress, error) {
if !IsHexAddress(hexaddr) { if !IsHexAddress(hexaddr) {
return nil, fmt.Errorf("Invalid address") return nil, errors.New("invalid address")
} }
a := FromHex(hexaddr) a := FromHex(hexaddr)
return &MixedcaseAddress{addr: BytesToAddress(a), original: hexaddr}, nil return &MixedcaseAddress{addr: BytesToAddress(a), original: hexaddr}, nil

Loading…
Cancel
Save