Bug 18736: (QA follow-up) Change to signed, add large int test
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 28 Sep 2018 09:20:49 +0000 (11:20 +0200)
committerNick Clemens <nick@bywatersolutions.com>
Thu, 21 Mar 2019 16:29:06 +0000 (16:29 +0000)
commit2f5e839bf780955d228c9672abce9d6e5ad1aedf
tree8e029489de2c7084f0555fa2f918c81cdef6e454
parent9922d2bf40e2b7429f300d4d73134f9fef3b8c65
Bug 18736: (QA follow-up) Change to signed, add large int test

[1] Add trivial subtest in t/Number/Price.t
Checking a negative number and a large number for the MAX_INT change.
Note: Confusing to have t/Prices.t too.

[2] Change UNSIGNED to SIGNED in get_rounding_sql. Although I did not spot
problems with negative prices, we theoretically could while casting.
cast(-2 as unsigned) == 18446744073709551614

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
C4/Acquisition.pm
t/Number/Price.t