Make interval decoding logic match that of psycopg2.

asyncpg will now treat 12-month interval increments as 365-day spans,
instead of 12*30 day spans.  This matches psycopg2 interval decoding
logic.

Fixes: #150.

Author: elprans

asyncpg/protocol/codecs/datetime.pyx

@@ -248,15 +248,23 @@ cdef interval_decode(ConnectionSettings settings, FastReadBuffer buf):
     cdef:
         int32_t days
         int32_t months
+        int32_t years
         int64_t seconds = 0
         uint32_t microseconds = 0
 
     _decode_time(buf, &seconds, &microseconds)
     days = hton.unpack_int32(buf.read(4))
     months = hton.unpack_int32(buf.read(4))
 
-    return datetime.timedelta(days=days + months * 30, seconds=seconds,
-                              microseconds=microseconds)
+    if months < 0:
+        years = -<int32_t>(-months // 12)
+        months = -<int32_t>(-months % 12)
+    else:
+        years = <int32_t>(months // 12)
+        months = <int32_t>(months % 12)
+
+    return datetime.timedelta(days=days + months * 30 + years * 365,
+                              seconds=seconds, microseconds=microseconds)
 
 
 cdef init_datetime_codecs():

tests/test_codecs.py

@@ -439,6 +439,19 @@ def test_bitstring(self):
 
         self.assertEqual(len(bits.bytes), expected_bytelen)
 
+    async def test_interval(self):
+        res = await self.con.fetchval("SELECT '5 years'::interval")
+        self.assertEqual(res, datetime.timedelta(days=1825))
+
+        res = await self.con.fetchval("SELECT '5 years 1 month'::interval")
+        self.assertEqual(res, datetime.timedelta(days=1855))
+
+        res = await self.con.fetchval("SELECT '-5 years'::interval")
+        self.assertEqual(res, datetime.timedelta(days=-1825))
+
+        res = await self.con.fetchval("SELECT '-5 years -1 month'::interval")
+        self.assertEqual(res, datetime.timedelta(days=-1855))
+
     async def test_unhandled_type_fallback(self):
         await self.con.execute('''
             CREATE EXTENSION IF NOT EXISTS isn