Readd SQL tests.

Author: janiversen

pymodbus/datastore/database/sql_datastore.py

@@ -29,7 +29,7 @@ def __init__(self, *args, **kwargs):  # pylint: disable=unused-argument
         self._table = None
         self._connection = None
         self.table = kwargs.get("table", "pymodbus")
-        self.database = kwargs.get("database", "sqlite:///pymodbus.db")
+        self.database = kwargs.get("database", "sqlite:///:memory:")
         self._db_create(self.table, self.database)
 
     def __str__(self):
@@ -93,6 +93,8 @@ def _db_create(self, table, database):
         :param database: The database uri to use
         """
         self._engine = sqlalchemy.create_engine(database, echo=False)
+        self._connection = self._engine.connect()
+        self._connection.execute('ATTACH DATABASE \':memory:\' AS pymodbus_db;')
         self._metadata = sqlalchemy.MetaData(self._engine)
         self._table = sqlalchemy.Table(
             table,
@@ -102,8 +104,7 @@ def _db_create(self, table, database):
             sqlalchemy.Column("value", sqltypes.Integer),
             UniqueConstraint("type", "index", name="key"),
         )
-        self._table.create(checkfirst=True)
-        self._connection = self._engine.connect()
+        self._table.create(self._engine)
 
     def _get(self, type, offset, count):  # pylint: disable=redefined-builtin
         """Get.

pymodbus/payload.py

@@ -356,7 +356,7 @@ def _unpack_words(self, fstring, handle):
 
         # Repack as unsigned Integer
         handle = [pack(self._byteorder + "H", p) for p in handle]
-        Log.debug(handle)
+        Log.debug("handle: {}", handle)
         handle = b"".join(handle)
         return handle
 

test/test_datastore.py

@@ -337,7 +337,6 @@ def setUp(self):
     def tearDown(self):
         """Clean up the test environment"""
 
-    @pytest.mark.skip
     @pytest.mark.xdist_group(name="sql")
     def test_str(self):
         """Test string."""