Merge lp:~osomon/notes-app/manually-create-db into lp:notes-app

[Olivier Tilloy]

Manually create the notes database before starting tests,
instead of launching the app, quitting it, and launching it again.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:219
http://jenkins.qa.ubuntu.com/job/notes-app-ci/222/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/159
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/153/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-amd64-ci/4
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/4
        deb: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/4/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-i386-ci/4
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/150
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/159
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/159/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/153
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/153/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2777/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2827/console
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/594
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/593

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/notes-app-ci/222/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:219
http://jenkins.qa.ubuntu.com/job/notes-app-ci/223/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/161
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/155/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-amd64-ci/5
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/5
        deb: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/5/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-i386-ci/5
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/152
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/161
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/161/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/155
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/155/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2779/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2830/console
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/598
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/597

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/notes-app-ci/223/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:219
http://jenkins.qa.ubuntu.com/job/notes-app-ci/224/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/163
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/157/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-amd64-ci/6
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/6
        deb: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/6/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-i386-ci/6
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/154
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/163
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/163/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/157
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/157/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2781/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2832/console
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/602
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/601

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/notes-app-ci/224/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:219
http://jenkins.qa.ubuntu.com/job/notes-app-ci/225/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/170
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/164/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-amd64-ci/7
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/7
        deb: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/7/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-i386-ci/7
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/160
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/170
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/170/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/164
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/164/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2786/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2838/console
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/611
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/613

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/notes-app-ci/225/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:219
http://jenkins.qa.ubuntu.com/job/notes-app-ci/226/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/172
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/166/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-amd64-ci/8
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/8
        deb: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/8/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-i386-ci/8
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/162
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/172
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/172/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/166
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/166/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2789/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2839/console
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/618
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/616

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/notes-app-ci/226/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:219
http://jenkins.qa.ubuntu.com/job/notes-app-ci/227/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/176
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/170/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-amd64-ci/9
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/9
        deb: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-armhf-ci/9/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/notes-app-trusty-i386-ci/9
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/165
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/176
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/176/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/170
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/170/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2793/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2844/console
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/626
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/625

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/notes-app-ci/227/rebuild

review: Needs Fixing (continuous-integration)

[Olivier Tilloy (osomon)]

Superseded by https://code.launchpad.net/~osomon/notes-app/simplify-db-lookup/+merge/193750.

review: Disapprove

Unmerged revisions

219. By Olivier Tilloy on 2013-10-29

Remove the DatabaseMixin helper class.

218. By Olivier Tilloy on 2013-10-29

Manually create the notes database before starting tests,
instead of launching the app, quitting it, and launching it again.

=== modified file 'tests/autopilot/notes_app/tests/__init__.py'
--- tests/autopilot/notes_app/tests/__init__.py	2013-10-08 21:50:29 +0000
+++ tests/autopilot/notes_app/tests/__init__.py	2013-10-29 16:46:19 +0000
@@ -8,10 +8,13 @@
 """notes-app autopilot tests."""
 
 import BaseHTTPServer
+import base64
+import glob
+import md5
+import os
 import os.path
-import glob
+import sqlite3
 import threading
-import base64
 
 from autopilot.input import Mouse, Touch, Pointer
 from autopilot.matchers import Eventually
@@ -82,17 +85,29 @@
             "com.ubuntu.notes",
             emulator_base=UbuntuUIToolkitEmulatorBase)
 
-    def launch_and_quit_app(self):
-        self.launch_app()
-        self.main_window.visible.wait_for(True)
-
-        # When calling launch_app an instance of the spawned process
-        # control object will be stored in self.app.process, and a cleanup
-        # handler will be registered that essentially kills the process.
-        # Therefore, by triggering the cleanup handler here we're killing the
-        # process and removing the handler, which allows a clean launch of
-        # the process during regular test setup.
-        self.doCleanups()
+    def ensure_db(self):
+        path = os.path.expanduser("~/.local/share/com.ubuntu.notes/Databases")
+        if not os.path.exists(path):
+            os.makedirs(path)
+        filepath = os.path.join(path, md5.md5("notes").hexdigest())
+        ini = filepath + ".ini"
+        if not os.path.exists(ini):
+            with open(ini, "w") as f:
+                f.write("""[General]
+Description=Notes database
+Driver=QSQLITE
+EstimatedSize=3145728
+Name=notes
+Version=1
+""")
+        sql = filepath + ".sqlite"
+        connection = sqlite3.connect(sql)
+        cursor = connection.cursor()
+        cursor.execute("CREATE TABLE IF NOT EXISTS notes "
+                       "(id INTEGER PRIMARY KEY, date DATETIME, note TEXT)")
+        connection.commit()
+        connection.close()
+        return sql
 
     def assert_osk_eventually_shown(self):
         if model() != 'Desktop':
@@ -176,40 +191,3 @@
     def tearDown(self):
         super(NotesTestCaseBaseWithHTTPServer, self).tearDown()
         self.server.shutdown()
-
-
-class DatabaseMixin(object):
-
-    """
-    Helper functions for dealing with sqlite databases
-    """
-
-    def _get_db_path(self):
-        db_path_list = [
-            "~/.local/share/com.ubuntu.notes/Databases",
-            "~/.local/share/Qt Project/QtQmlViewer/QML/OfflineStorage/"
-            "Databases/"]
-        for path in db_path_list:
-            path = os.path.expanduser(path)
-            if os.path.exists(path):
-                return path
-        return None
-
-    def find_db(self):
-        dbs_path = self._get_db_path()
-        if not dbs_path:
-            return None
-        files = [f for f in os.listdir(dbs_path)
-                 if os.path.splitext(f)[1] == ".ini"]
-        for f in files:
-            ini_path = os.path.join(dbs_path, f)
-            with open(ini_path) as ini:
-                for line in ini:
-                    if "=" in line:
-                        key, val = line.strip().split("=")
-                        if key == "Name" and val == "notes":
-                            try:
-                                return ini_path.replace(".ini", ".sqlite")
-                            except OSError:
-                                pass
-        return None

=== modified file 'tests/autopilot/notes_app/tests/test_create.py'
--- tests/autopilot/notes_app/tests/test_create.py	2013-09-27 14:42:07 +0000
+++ tests/autopilot/notes_app/tests/test_create.py	2013-10-29 16:46:19 +0000
@@ -12,12 +12,12 @@
 from testtools.matchers import Equals
 from autopilot.matchers import Eventually
 
-from notes_app.tests import NotesAppTestCase, DatabaseMixin
+from notes_app.tests import NotesAppTestCase
 
 import os
 
 
-class TestCreate(NotesAppTestCase, DatabaseMixin):
+class TestCreate(NotesAppTestCase):
     """Tests creating notes"""
 
     """ This is needed to wait for the application to start.
@@ -29,13 +29,7 @@
         super(TestCreate, self).setUp()
 
     def clean_db(self):
-        path = self.find_db()
-        if path is None:
-            self.launch_and_quit_app()
-            path = self.find_db()
-            if path is None:
-                self.assertNotEquals(path, None)
-
+        path = self.ensure_db()
         try:
             os.remove(path)
         except OSError:

=== modified file 'tests/autopilot/notes_app/tests/test_delete.py'
--- tests/autopilot/notes_app/tests/test_delete.py	2013-10-10 09:30:43 +0000
+++ tests/autopilot/notes_app/tests/test_delete.py	2013-10-29 16:46:19 +0000
@@ -12,12 +12,12 @@
 from testtools.matchers import Equals
 from autopilot.matchers import Eventually
 
-from notes_app.tests import NotesAppTestCase, DatabaseMixin
+from notes_app.tests import NotesAppTestCase
 
 import sqlite3
 
 
-class TestDelete(NotesAppTestCase, DatabaseMixin):
+class TestDelete(NotesAppTestCase):
     """Tests deleting notes"""
 
     def setUp(self):
@@ -30,13 +30,7 @@
             {"content":"This is a note.","type":"text"}
         ]}"""
 
-        path = self.find_db()
-        if path is None:
-            self.launch_and_quit_app()
-            path = self.find_db()
-            if path is None:
-                self.assertNotEquals(path, None)
-
+        path = self.ensure_db()
         conn = sqlite3.connect(path)
         cursor = conn.cursor()
         cursor.execute("DELETE FROM notes")

=== modified file 'tests/autopilot/notes_app/tests/test_expand_collapse.py'
--- tests/autopilot/notes_app/tests/test_expand_collapse.py	2013-10-01 07:42:44 +0000
+++ tests/autopilot/notes_app/tests/test_expand_collapse.py	2013-10-29 16:46:19 +0000
@@ -12,12 +12,12 @@
 from testtools.matchers import Equals
 from autopilot.matchers import Eventually
 
-from notes_app.tests import NotesAppTestCase, DatabaseMixin
+from notes_app.tests import NotesAppTestCase
 
 import sqlite3
 
 
-class TestExpandCollapse(NotesAppTestCase, DatabaseMixin):
+class TestExpandCollapse(NotesAppTestCase):
     """Tests deleting notes"""
 
     def setUp(self):
@@ -35,13 +35,7 @@
             ]}"""
         ]
 
-        path = self.find_db()
-        if path is None:
-            self.launch_and_quit_app()
-            path = self.find_db()
-            if path is None:
-                self.assertNotEquals(path, None)
-
+        path = self.ensure_db()
         conn = sqlite3.connect(path)
         cursor = conn.cursor()
         cursor.execute("DELETE FROM notes")

=== modified file 'tests/autopilot/notes_app/tests/test_images.py'
--- tests/autopilot/notes_app/tests/test_images.py	2013-10-07 10:10:40 +0000
+++ tests/autopilot/notes_app/tests/test_images.py	2013-10-29 16:46:19 +0000
@@ -12,12 +12,12 @@
 from testtools.matchers import Equals
 from autopilot.matchers import Eventually
 
-from notes_app.tests import NotesTestCaseBaseWithHTTPServer, DatabaseMixin
+from notes_app.tests import NotesTestCaseBaseWithHTTPServer
 
 import sqlite3
 
 
-class BaseForTestImages(NotesTestCaseBaseWithHTTPServer, DatabaseMixin):
+class BaseForTestImages(NotesTestCaseBaseWithHTTPServer):
     def setUp(self):
         self.setup_db(self.IMAGE)
         super(BaseForTestImages, self).setUp()
@@ -27,13 +27,7 @@
         {"content":"http://localhost:8129/""" + image + """","type":"image"}
         ]}"""
 
-        path = self.find_db()
-        if path is None:
-            self.launch_and_quit_app()
-            path = self.find_db()
-            if path is None:
-                self.assertNotEquals(path, None)
-
+        path = self.ensure_db()
         conn = sqlite3.connect(path)
         cursor = conn.cursor()
         cursor.execute("DELETE FROM notes")

=== modified file 'tests/autopilot/notes_app/tests/test_parts.py'
--- tests/autopilot/notes_app/tests/test_parts.py	2013-10-07 10:21:27 +0000
+++ tests/autopilot/notes_app/tests/test_parts.py	2013-10-29 16:46:19 +0000
@@ -13,12 +13,12 @@
 from autopilot.matchers import Eventually
 from autopilot.platform import model
 
-from notes_app.tests import NotesTestCaseBaseWithHTTPServer, DatabaseMixin
+from notes_app.tests import NotesTestCaseBaseWithHTTPServer
 
 import sqlite3
 
 
-class TestFocus(NotesTestCaseBaseWithHTTPServer, DatabaseMixin):
+class TestFocus(NotesTestCaseBaseWithHTTPServer):
     """Tests focusing notes"""
 
     """ This is needed to wait for the application to start.
@@ -34,7 +34,7 @@
             {"content":"http://localhost:8129/image.png","type":"image"}
         ]}"""
 
-        path = self.find_db()
+        path = self.ensure_db()
         conn = sqlite3.connect(path)
         cursor = conn.cursor()
         cursor.execute("DELETE FROM notes")

=== modified file 'tests/autopilot/notes_app/tests/test_quit.py'
--- tests/autopilot/notes_app/tests/test_quit.py	2013-10-01 07:42:44 +0000
+++ tests/autopilot/notes_app/tests/test_quit.py	2013-10-29 16:46:19 +0000
@@ -13,7 +13,7 @@
 from autopilot.matchers import Eventually
 from autopilot.introspection.dbus import StateNotFoundError
 
-from notes_app.tests import NotesAppTestCase, DatabaseMixin
+from notes_app.tests import NotesAppTestCase
 
 import sqlite3
 import json
@@ -47,17 +47,8 @@
             return True
         return False
 
-    def get_db_path(self):
-        path = self.find_db()
-        if path is None:
-            self.launch_and_quit_app()
-            path = self.find_db()
-            if path is None:
-                self.assertNotEquals(path, None)
-        return path
-
-
-class TestQuit(NotesAppTestCase, DatabaseMixin, UtilsMixin):
+
+class TestQuit(NotesAppTestCase, UtilsMixin):
     """Tests save on quit"""
 
     """ This is needed to wait for the application to start.
@@ -72,7 +63,7 @@
             {"content":"foobar","type":"text"}
         ]}"""
 
-        path = self.get_db_path()
+        path = self.ensure_db()
         conn = sqlite3.connect(path)
         cursor = conn.cursor()
         cursor.execute("DELETE FROM notes")
@@ -92,7 +83,7 @@
 
         self.assertThat(self.app_has_quit, Eventually(Equals(True)))
 
-        path = self.find_db()
+        path = self.ensure_db()
         conn = sqlite3.connect(path)
         cursor = conn.cursor()
 
@@ -105,7 +96,7 @@
         self.assertEquals(data["elements"][0]["content"], "foobarxxx")
 
 
-class TestQuitNoNote(NotesAppTestCase, DatabaseMixin, UtilsMixin):
+class TestQuitNoNote(NotesAppTestCase, UtilsMixin):
     """Tests save on quit with no notes"""
 
     """ This is needed to wait for the application to start.
@@ -116,7 +107,7 @@
         super(TestQuitNoNote, self).setUp()
 
     def setup_db(self):
-        path = self.get_db_path()
+        path = self.ensure_db()
         conn = sqlite3.connect(path)
         cursor = conn.cursor()
         cursor.execute("DELETE FROM notes")
@@ -134,7 +125,7 @@
 
         self.assertThat(self.app_has_quit, Eventually(Equals(True)))
 
-        path = self.find_db()
+        path = self.ensure_db()
         conn = sqlite3.connect(path)
         cursor = conn.cursor()