Add unit tests for dbapi module

Used dbapi-compliance [1] package to test module according to pep-249
specification. Not implemented features are skipped in the tests.

Added dbapi-compliance package to test.sh requirements and appveyor.yml

[1] https://github.com/baztian/dbapi-compliance/

Author: artembo

appveyor.yml

@@ -23,7 +23,7 @@ install:
   # install runtime dependencies
   - "%PYTHON%\\python.exe -m pip install -r requirements.txt"
   # install testing dependencies
-  - "%PYTHON%\\python.exe -m pip install pyyaml%PYYAML%"
+  - "%PYTHON%\\python.exe -m pip install pyyaml%PYYAML% dbapi-compliance==1.15.0"
 
 build: off
 

test.sh

@@ -11,7 +11,7 @@ sudo apt-get -q -y install tarantool
 
 # Install testing dependencies.
 pip install -r requirements.txt
-pip install pyyaml
+pip install pyyaml dbapi-compliance==1.15.0
 
 # Run tests.
 python setup.py test

unit/suites/__init__.py

@@ -9,9 +9,10 @@
 from .test_protocol import TestSuite_Protocol
 from .test_reconnect import TestSuite_Reconnect
 from .test_mesh import TestSuite_Mesh
+from .test_dbapi import TestSuite_DBAPI
 
 test_cases = (TestSuite_Schema, TestSuite_Request, TestSuite_Protocol,
-              TestSuite_Reconnect, TestSuite_Mesh)
+              TestSuite_Reconnect, TestSuite_Mesh, TestSuite_DBAPI)
 
 def load_tests(loader, tests, pattern):
     suite = unittest.TestSuite()

unit/suites/test_dbapi.py

@@ -0,0 +1,130 @@
+# -*- coding: utf-8 -*-
+
+from __future__ import print_function
+
+import sys
+import unittest
+
+import dbapi20
+
+import tarantool
+from tarantool import dbapi
+from .lib.tarantool_server import TarantoolServer
+
+
+class TestSuite_DBAPI(dbapi20.DatabaseAPI20Test):
+    table_prefix = 'dbapi20test_'  # If you need to specify a prefix for tables
+
+    ddl1 = 'create table %sbooze (name varchar(20) primary key)' % table_prefix
+    ddl2 = 'create table %sbarflys (name varchar(20) primary key, ' \
+           'drink varchar(30))' % table_prefix
+
+    @classmethod
+    def setUpClass(self):
+        print(' DBAPI '.center(70, '='), file=sys.stderr)
+        print('-' * 70, file=sys.stderr)
+        self.srv = TarantoolServer()
+        self.srv.script = 'unit/suites/box.lua'
+        self.srv.start()
+        self.con = tarantool.Connection(self.srv.host, self.srv.args['primary'])
+        self.driver = dbapi
+        self.connect_kw_args = dict(
+            host=self.srv.host,
+            port=self.srv.args['primary'])
+        print("DRIVER +++++++++++++++", self.driver)
+
+    def setUp(self):
+        # prevent a remote tarantool from clean our session
+        if self.srv.is_started():
+            self.srv.touch_lock()
+        self.con.flush_schema()
+
+        # grant full access to guest
+        self.srv.admin("box.schema.user.grant('guest', 'create,read,write,"
+                       "execute', 'universe')")
+
+    @classmethod
+    def tearDownClass(self):
+        self.con.close()
+        self.srv.stop()
+        self.srv.clean()
+
+    def test_rowcount(self):
+        con = self._connect()
+        try:
+            cur = con.cursor()
+            self.executeDDL1(cur)
+            dbapi20._failUnless(self,cur.rowcount in (-1,1),   # Bug #543885
+                'cursor.rowcount should be -1 or 0 after executing no-result '
+                'statements' + str(cur.rowcount)
+                )
+            cur.execute("%s into %sbooze values ('Victoria Bitter')" % (
+                self.insert, self.table_prefix
+                ))
+            dbapi20._failUnless(self,cur.rowcount in (-1,1),
+                'cursor.rowcount should == number or rows inserted, or '
+                'set to -1 after executing an insert statement'
+                )
+            cur.execute("select name from %sbooze" % self.table_prefix)
+            dbapi20._failUnless(self,cur.rowcount in (-1,1),
+                'cursor.rowcount should == number of rows returned, or '
+                'set to -1 after executing a select statement'
+                )
+            self.executeDDL2(cur)
+            dbapi20._failUnless(self,cur.rowcount in (-1,1),   # Bug #543885
+                'cursor.rowcount should be -1 or 0 after executing no-result '
+                'statements'
+                )
+        finally:
+            con.close()
+
+    @unittest.skip('Not implemented')
+    def test_Binary(self):
+        pass
+
+    @unittest.skip('Not implemented')
+    def test_STRING(self):
+        pass
+
+    @unittest.skip('Not implemented')
+    def test_BINARY(self):
+        pass
+
+    @unittest.skip('Not implemented')
+    def test_NUMBER(self):
+        pass
+
+    @unittest.skip('Not implemented')
+    def test_DATETIME(self):
+        pass
+
+    @unittest.skip('Not implemented')
+    def test_ROWID(self):
+        pass
+
+    @unittest.skip('Not implemented')
+    def test_Date(self):
+        pass
+
+    @unittest.skip('Not implemented')
+    def test_Time(self):
+        pass
+
+    @unittest.skip('Not implemented')
+    def test_Timestamp(self):
+        pass
+
+    @unittest.skip('Not implemented as optional.')
+    def test_nextset(self):
+        pass
+
+    @unittest.skip('To do')
+    def test_callproc(self):
+        pass
+
+    def test_setoutputsize(self):  # Do nothing
+        pass
+
+    @unittest.skip('To do')
+    def test_description(self):
+        pass