Reflect Oracle NUMBER(NULL, 0) as INTEGER

The Oracle NUMBER datatype is reflected as INTEGER if the precision is NULL
and the scale is zero, as this is how INTEGER values come back when
reflected from Oracle's tables.  Pull request courtesy Kent Bower.

Change-Id: I4627febd46cab7085299c0a5700ee0f0bdca513c
Pull-request: https://github.com/zzzeek/sqlalchemy/pull/437

1 files changed, 46 insertions, 2 deletions

diff --git a/test/dialect/oracle/test_reflection.py b/test/dialect/oracle/test_reflection.py
index d09f12e60..190fd9f38 100644
--- a/test/dialect/oracle/test_reflection.py
+++ b/test/dialect/oracle/test_reflection.py
@@ -1,7 +1,7 @@
 # coding: utf-8
 
 
-from sqlalchemy.testing import eq_
+from sqlalchemy.testing import eq_, is_
 from sqlalchemy import exc
 from sqlalchemy.sql import table
 from sqlalchemy.testing import fixtures, AssertsCompiledSQL
@@ -12,7 +12,8 @@ from sqlalchemy import Integer, Text, LargeBinary, Unicode, UniqueConstraint,\
     literal_column, VARCHAR, create_engine, Date, NVARCHAR, \
     ForeignKeyConstraint, Sequence, Float, DateTime, cast, UnicodeText, \
     union, except_, type_coerce, or_, outerjoin, DATE, NCHAR, outparam, \
-    PrimaryKeyConstraint, FLOAT
+    PrimaryKeyConstraint, FLOAT, INTEGER
+from sqlalchemy.dialects.oracle.base import NUMBER
 from sqlalchemy.testing import assert_raises
 from sqlalchemy.testing.engines import testing_engine
 from sqlalchemy.testing.schema import Table, Column
@@ -534,3 +535,46 @@ class DBLinkReflectionTest(fixtures.TestBase):
         eq_(list(t.primary_key), [t.c.id])
 
 
+class TypeReflectionTest(fixtures.TestBase):
+    __only_on__ = 'oracle'
+    __backend__ = True
+
+    @testing.provide_metadata
+    def _run_test(self, specs, attributes):
+        columns = [Column('c%i' % (i + 1), t[0]) for i, t in enumerate(specs)]
+        m = self.metadata
+        Table('oracle_types', m, *columns)
+        m.create_all()
+        m2 = MetaData(testing.db)
+        table = Table('oracle_types', m2, autoload=True)
+        for i, (reflected_col, spec) in enumerate(zip(table.c, specs)):
+            expected_spec = spec[1]
+            reflected_type = reflected_col.type
+            is_(type(reflected_type), type(expected_spec))
+            for attr in attributes:
+                eq_(
+                    getattr(reflected_type, attr),
+                    getattr(expected_spec, attr),
+                    "Column %s: Attribute %s value of %s does not "
+                    "match %s for type %s" % (
+                        "c%i" % (i + 1),
+                        attr,
+                        getattr(reflected_type, attr),
+                        getattr(expected_spec, attr),
+                        spec[0]
+                    )
+                )
+
+    def test_integer_types(self):
+        specs = [
+            (Integer, INTEGER(),),
+            (Numeric, INTEGER(),),
+        ]
+        self._run_test(specs, [])
+
+    def test_number_types(self):
+        specs = [
+            (Numeric(5, 2), NUMBER(5, 2),),
+            (NUMBER, NUMBER(),),
+        ]
+        self._run_test(specs, ['precision', 'scale'])