Merge pull request #113 from tvincentNuoDB/master

Updated ScaledCount2 handling of negative values

Author: mjmichaels

pynuodb/encodedsession.py

@@ -658,9 +658,10 @@ def putScaledCount2(self, value):
         :type value: decimal.Decimal
         """
         scale = abs(value.as_tuple()[2])
-        sign = "1" if value.as_tuple()[0] == 0 else "-1"
-        value = toSignedByteString(int(value * decimal.Decimal(10**scale)))
-        packed = chr(protocol.SCALEDCOUNT2) + chr(scale) + sign + chr(len(value)) + value
+        sign = 1 if value.as_tuple()[0] == 0 else -1
+        sign = toSignedByteString(sign)
+        value = toByteString(int(abs(value) * decimal.Decimal(10**scale)))
+        packed = chr(protocol.SCALEDCOUNT2) + toByteString(scale) + sign + chr(len(value)) + value
         self.__output += packed
         return self
 
@@ -1076,5 +1077,3 @@ def getCommitInfo(self, nodeID):
         """ Currently does not support last commit """
 
         return 0
-
-

tests/nuodb_basic_test.py

@@ -155,6 +155,7 @@ def _test_decimal_fixture(self, value, precision, scale):
     #
     def test_many_significant_digits(self):
         self._test_decimal_fixture(decimal.Decimal("31943874831932418390.01"), 38, 12)
+        self._test_decimal_fixture(decimal.Decimal("-31943874831932418390.01"), 38, 12)
 
     # This test demonstrates the broken implementation of getScaledInt
     # which results in: