test: fix compatibility with Tarantool master

New Tarantool 3.0 patch introduces a breaking change [1]. This patch
adds a compatibility layer to test instance.

It was decided to explicitly set SEQSCAN in our tests and change session
options for DBAPI tests.

1. tarantool/tarantool#8602

Author: DifferentialOrange

test/suites/box.lua

@@ -1,11 +1,17 @@
 #!/usr/bin/env tarantool
 
+local is_compat, compat = pcall(require, "compat")
 local os = require('os')
 
 local admin_listen = os.getenv("ADMIN")
 local primary_listen = os.getenv("LISTEN")
 local auth_type = os.getenv("AUTH_TYPE")
 
+local SQL_SEQ_SCAN_DEFAULT = os.getenv("SQL_SEQ_SCAN_DEFAULT")
+if is_compat and SQL_SEQ_SCAN_DEFAULT then
+    compat.sql_seq_scan_default = SQL_SEQ_SCAN_DEFAULT
+end
+
 require('console').listen(admin_listen)
 box.cfg{
     listen       = primary_listen,

test/suites/lib/tarantool_server.py

@@ -189,7 +189,8 @@ def __new__(cls,
                 ssl_password=None,
                 ssl_password_file=None,
                 create_unix_socket=False,
-                auth_type=None):
+                auth_type=None,
+                sql_seq_scan_default=None):
         # pylint: disable=unused-argument
 
         if os.name == 'nt':
@@ -205,7 +206,8 @@ def __init__(self,
                  ssl_password=None,
                  ssl_password_file=None,
                  create_unix_socket=False,
-                 auth_type=None):
+                 auth_type=None,
+                 sql_seq_scan_default=None):
         # pylint: disable=consider-using-with
 
         os.popen('ulimit -c unlimited').close()
@@ -235,6 +237,7 @@ def __init__(self,
         self.ssl_password = ssl_password
         self.ssl_password_file = ssl_password_file
         self.auth_type = auth_type
+        self.sql_seq_scan_default = sql_seq_scan_default
         self._binary = None
         self._log_des = None
 
@@ -289,6 +292,8 @@ def generate_configuration(self):
             os.putenv("AUTH_TYPE", self.auth_type)
         else:
             os.putenv("AUTH_TYPE", "")
+        if self.sql_seq_scan_default is not None:
+            os.putenv("SQL_SEQ_SCAN_DEFAULT", self.sql_seq_scan_default)
 
     def prepare_args(self):
         """

test/suites/test_dbapi.py

@@ -27,7 +27,9 @@ class TestSuiteDBAPI(dbapi20.DatabaseAPI20Test):
     def setUpClass(cls):
         print(' DBAPI '.center(70, '='), file=sys.stderr)
         print('-' * 70, file=sys.stderr)
-        cls.srv = TarantoolServer()
+        # Select scans are not allowed with compat.sql_seq_scan_default = "new",
+        # but tests create cursors with fullscan.
+        cls.srv = TarantoolServer(sql_seq_scan_default="old")
         cls.srv.script = 'test/suites/box.lua'
         cls.srv.start()
         cls.con = tarantool.Connection(cls.srv.host, cls.srv.args['primary'])

test/suites/test_schema.py

@@ -454,10 +454,21 @@ def test_07_schema_version_update(self):
     def _run_test_schema_fetch_disable(self, con, mode=None):
         # Enable SQL test case for tarantool 2.* and higher.
         if int(str(self.srv.admin.tnt_version)[0]) > 1:
-            self.testing_methods['available']['execute'] = {
-                'input': ['SELECT * FROM "tester"'],
-                'output': [[1, None]],
-            }
+            if self.srv.admin.tnt_version >= pkg_resources.parse_version('2.11.0'):
+                # SEQSCAN keyword is explicitly allowing to use seqscan
+                # https://github.com/tarantool/tarantool/commit/77648827326ad268ec0ffbcd620c2371b65ef2b4
+                # It was introduced in 2.11.0-rc1. If compat.sql_seq_scan_default
+                # set to "new" (default value since 3.0), returns error
+                # if trying to scan without keyword.
+                self.testing_methods['available']['execute'] = {
+                    'input': ['SELECT * FROM SEQSCAN "tester"'],
+                    'output': [[1, None]],
+                }
+            else:
+                self.testing_methods['available']['execute'] = {
+                    'input': ['SELECT * FROM "tester"'],
+                    'output': [[1, None]],
+                }
 
         # Testing the schemaless connection with methods
         # that should NOT be available.