aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTing-Wei Lan <tingwei.lan@cobinhood.com>2019-03-26 18:41:37 +0800
committerJhih-Ming Huang <jm.huang@cobinhood.com>2019-04-11 10:39:59 +0800
commit8cfc216347678a9cf8905cde4dbcf3fb4362a716 (patch)
treeadb7d3c29edabf9f1351add98523f04ad9dcb638
parentfe7c333a045267e6647e7b5f7fd926158a6f8464 (diff)
downloaddexon-8cfc216347678a9cf8905cde4dbcf3fb4362a716.tar
dexon-8cfc216347678a9cf8905cde4dbcf3fb4362a716.tar.gz
dexon-8cfc216347678a9cf8905cde4dbcf3fb4362a716.tar.bz2
dexon-8cfc216347678a9cf8905cde4dbcf3fb4362a716.tar.lz
dexon-8cfc216347678a9cf8905cde4dbcf3fb4362a716.tar.xz
dexon-8cfc216347678a9cf8905cde4dbcf3fb4362a716.tar.zst
dexon-8cfc216347678a9cf8905cde4dbcf3fb4362a716.zip
core: vm: sqlvm: ast: rename tests for DataType encoding or decoding
Since we have both DataType and Decimal functions in this file, we should avoid using generic names such as TestEncode and TestDecode without mentioning what types are going to be tested.
-rw-r--r--core/vm/sqlvm/ast/types_test.go62
1 files changed, 31 insertions, 31 deletions
diff --git a/core/vm/sqlvm/ast/types_test.go b/core/vm/sqlvm/ast/types_test.go
index 15e9cdec0..2107cc206 100644
--- a/core/vm/sqlvm/ast/types_test.go
+++ b/core/vm/sqlvm/ast/types_test.go
@@ -23,7 +23,7 @@ func (s *TypesTestSuite) requireEncodeAndDecodeDecimalNoError(
s.Require().Equal(t.String(), decode.String())
}
-func (s *TypesTestSuite) requireEncodeAndDecodeNoError(
+func (s *TypesTestSuite) requireEncodeAndDecodeDataTypeNoError(
d DataType, t TypeNode) {
encode, code, message := t.GetType()
s.Require().Zero(code)
@@ -34,65 +34,65 @@ func (s *TypesTestSuite) requireEncodeAndDecodeNoError(
s.Require().Equal(t, decode)
}
-func (s *TypesTestSuite) requireEncodeError(input TypeNode) {
+func (s *TypesTestSuite) requireEncodeDataTypeError(input TypeNode) {
_, code, message := input.GetType()
s.Require().NotZero(code)
s.Require().NotEmpty(message)
}
-func (s *TypesTestSuite) requireDecodeError(input DataType) {
+func (s *TypesTestSuite) requireDecodeDataTypeError(input DataType) {
_, err := DataTypeDecode(input)
s.Require().Error(err)
}
-func (s *TypesTestSuite) TestEncodeAndDecode() {
- s.requireEncodeAndDecodeNoError(
+func (s *TypesTestSuite) TestEncodeAndDecodeDataType() {
+ s.requireEncodeAndDecodeDataTypeNoError(
ComposeDataType(DataTypeMajorBool, 0),
&BoolTypeNode{})
- s.requireEncodeAndDecodeNoError(
+ s.requireEncodeAndDecodeDataTypeNoError(
ComposeDataType(DataTypeMajorAddress, 0),
&AddressTypeNode{})
- s.requireEncodeAndDecodeNoError(
+ s.requireEncodeAndDecodeDataTypeNoError(
ComposeDataType(DataTypeMajorInt, 1),
&IntTypeNode{Size: 16})
- s.requireEncodeAndDecodeNoError(
+ s.requireEncodeAndDecodeDataTypeNoError(
ComposeDataType(DataTypeMajorUint, 2),
&IntTypeNode{Unsigned: true, Size: 24})
- s.requireEncodeAndDecodeNoError(
+ s.requireEncodeAndDecodeDataTypeNoError(
ComposeDataType(DataTypeMajorFixedBytes, 3),
&FixedBytesTypeNode{Size: 4})
- s.requireEncodeAndDecodeNoError(
+ s.requireEncodeAndDecodeDataTypeNoError(
ComposeDataType(DataTypeMajorDynamicBytes, 0),
&DynamicBytesTypeNode{})
- s.requireEncodeAndDecodeNoError(
+ s.requireEncodeAndDecodeDataTypeNoError(
ComposeDataType(DataTypeMajorFixed, 1),
&FixedTypeNode{Size: 8, FractionalDigits: 1})
- s.requireEncodeAndDecodeNoError(
+ s.requireEncodeAndDecodeDataTypeNoError(
ComposeDataType(DataTypeMajorUfixed+1, 2),
&FixedTypeNode{Unsigned: true, Size: 16, FractionalDigits: 2})
}
-func (s *TypesTestSuite) TestEncodeError() {
- s.requireEncodeError(&IntTypeNode{Size: 1})
- s.requireEncodeError(&IntTypeNode{Size: 257})
- s.requireEncodeError(&FixedBytesTypeNode{Size: 0})
- s.requireEncodeError(&FixedBytesTypeNode{Size: 257})
- s.requireEncodeError(&FixedTypeNode{Size: 1, FractionalDigits: 0})
- s.requireEncodeError(&FixedTypeNode{Size: 257, FractionalDigits: 0})
- s.requireEncodeError(&FixedTypeNode{Size: 8, FractionalDigits: 81})
+func (s *TypesTestSuite) TestEncodeDataTypeError() {
+ s.requireEncodeDataTypeError(&IntTypeNode{Size: 1})
+ s.requireEncodeDataTypeError(&IntTypeNode{Size: 257})
+ s.requireEncodeDataTypeError(&FixedBytesTypeNode{Size: 0})
+ s.requireEncodeDataTypeError(&FixedBytesTypeNode{Size: 257})
+ s.requireEncodeDataTypeError(&FixedTypeNode{Size: 1, FractionalDigits: 0})
+ s.requireEncodeDataTypeError(&FixedTypeNode{Size: 257, FractionalDigits: 0})
+ s.requireEncodeDataTypeError(&FixedTypeNode{Size: 8, FractionalDigits: 81})
}
-func (s *TypesTestSuite) TestDecodeError() {
- s.requireDecodeError(DataTypeUnknown)
- s.requireDecodeError(ComposeDataType(DataTypeMajorBool, 1))
- s.requireDecodeError(ComposeDataType(DataTypeMajorAddress, 1))
- s.requireDecodeError(ComposeDataType(DataTypeMajorInt, 0x20))
- s.requireDecodeError(ComposeDataType(DataTypeMajorUint, 0x20))
- s.requireDecodeError(ComposeDataType(DataTypeMajorFixedBytes, 0x20))
- s.requireDecodeError(ComposeDataType(DataTypeMajorDynamicBytes, 1))
- s.requireDecodeError(ComposeDataType(DataTypeMajorFixed, 81))
- s.requireDecodeError(ComposeDataType(DataTypeMajorUfixed, 81))
- s.requireDecodeError(ComposeDataType(DataTypeMajorUfixed+0x20, 80))
+func (s *TypesTestSuite) TestDecodeDataTypeError() {
+ s.requireDecodeDataTypeError(DataTypeUnknown)
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorBool, 1))
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorAddress, 1))
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorInt, 0x20))
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorUint, 0x20))
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorFixedBytes, 0x20))
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorDynamicBytes, 1))
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorFixed, 81))
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorUfixed, 81))
+ s.requireDecodeDataTypeError(ComposeDataType(DataTypeMajorUfixed+0x20, 80))
}
func (s *TypesTestSuite) TestEncodeAndDecodeDecimal() {