comments and cosmetics

Author: David Erb

src/dls_servbase_lib/datafaces/normsql.py

@@ -21,7 +21,9 @@ class Normsql(Thing):
     def __init__(self, specification=None):
         Thing.__init__(self, thing_type, specification)
 
+        # For testing, caller might want to drop the database on connection.
         self.__should_drop_database = specification.get("should_drop_database")
+
         self.__database = None
 
     # ----------------------------------------------------------------------------------------
@@ -43,6 +45,8 @@ async def establish_database_connection(self):
                 self.specification()["database"],
                 DatabaseDefinition(),
             )
+
+            # For testing, caller might want to drop the database on connection.
             await self.__database.connect(
                 should_drop_database=self.__should_drop_database
             )

tests/test_database.py

@@ -21,11 +21,11 @@ def test(self, constants, logging_setup, output_directory):
 
         database_specification = {
             "type": ClassTypes.AIOSQLITE,
-            "filename": "%s/dls_servbase_scheduler.sqlite" % (output_directory),
+            "filename": "%s/dls_servbase_pytest.sqlite" % (output_directory),
         }
 
         # Test direct SQL access to the database.
-        DatabaseTesterImage().main(
+        DatabaseTester().main(
             constants,
             database_specification,
             output_directory,
@@ -53,15 +53,15 @@ def test(self, constants, logging_setup, output_directory):
         }
 
         # Test direct SQL access to the database.
-        DatabaseTesterImage().main(
+        DatabaseTester().main(
             constants,
             database_specification,
             output_directory,
         )
 
 
 # ----------------------------------------------------------------------------------------
-class DatabaseTesterImage(BaseTester2):
+class DatabaseTester(BaseTester2):
     """
     Test direct SQL access to the database.
     """