Add unit tests for dbapi module

used dbapi-compliance package to test module according to pep-249 specification
removed EOL python 3.4 from appveyor test
used installer.sh from tarantool.io to set up Tarantool 2.4 version in test.sh

Author: artembo

Diff for: appveyor.yml

@@ -21,7 +21,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
 

Diff for: test.sh

@@ -3,15 +3,11 @@
 set -exu # Strict shell (w/o -o pipefail)
 
 # Install tarantool.
-curl http://download.tarantool.org/tarantool/2x/gpgkey | sudo apt-key add -
-release=`lsb_release -c -s`
-echo "deb http://download.tarantool.org/tarantool/2x/ubuntu/ ${release} main" | sudo tee /etc/apt/sources.list.d/tarantool_2x.list
-sudo apt-get update > /dev/null
-sudo apt-get -q -y install tarantool
+curl -L https://tarantool.io/installer.sh | VER=2.4 sudo -E bash
 
 # 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

Diff for: 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()

Diff for: unit/suites/test_dbapi.py

@@ -0,0 +1,126 @@
+# -*- coding: utf-8 -*-
+
+from __future__ import print_function
+
+import sys
+import unittest
+
+import dbapi20
+
+import tarantool
+from tarantool.dbapi import Connection
+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 = Connection(self.srv.host, self.srv.args['primary'])
+
+    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