fixes #23

Author: jph00

apswutils/db.py

@@ -397,11 +397,16 @@ def attach(self, alias: str, filepath: Union[str, pathlib.Path]):
         ).strip()
         self.execute(attach_sql)
 
-    def fetchone(self, sql:str, where_args: Optional[Union[Iterable, dict]] = None):
+    def item(self, sql:str, where_args: Optional[Union[Iterable, dict]] = None):
         """
-        Execute ``sql`` and return a single row result
+        Execute ``sql`` and return a single field from a single row
         """
-        return self.execute(sql, where_args or []).fetchone()[0]
+        row = self.execute(sql, (where_args or [])).fetchall()
+        if len(row)==0: raise NotFoundError
+        elif len(row) > 1: raise ValueError(f"Not unique: {len(row)} results")
+        res = row[0]
+        if len(res) > 1: raise ValueError(f"Too many fields: {len(res)} fields")
+        return res[0]
 
     def query(
         self, sql: str, params: Optional[Union[Iterable, dict]] = None

tests/test_constructor.py

@@ -13,11 +13,15 @@ def test_recursive_triggers_off():
     assert not db.execute("PRAGMA recursive_triggers").fetchone()[0]
 
 
+# This test is currently disabled since shared mem for sqlite is not recommended,
+#   and it requires a special build.
+"""
 def test_memory_name():
     db1 = Database(memory_name="shared")
     db2 = Database(memory_name="shared")
     db1["dogs"].insert({"name": "Cleo"})
     assert list(db2["dogs"].rows) == [{"name": "Cleo"}]
+"""
 
 
 def test_sqlite_version():