Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fix the comment deletion from table.

  • Loading branch information...
commit 8fa6ac1d757dd7e2a3e83f160fa5d4749dd109d8 1 parent 4a3de91
@mjessome authored
View
BIN  scripts/autoland/test/autoland.sqlite
Binary file not shown
View
2  scripts/autoland/test/test_dbhandler.py
@@ -112,7 +112,7 @@ def testPatchSetQuery(self):
self.db.PatchSetDelete(ps2)
self.db.PatchSetDelete(ps3)
# Add tests for querying on other params than just branch
-
+
def testPatchSetGetNext(self):
ps1 = PatchSet(bug_id=12577, patches='534442', branch='try',
try_run=1, author='lsblakk@mozilla.com', retries=None, try_syntax=None)
View
4 scripts/autoland/utils/db_handler.py
@@ -402,12 +402,12 @@ def CommentUpdate(self, cmnt):
connection.execute(q)
return True
- def CommentDelete(self, cmnt_id):
+ def CommentDelete(self, cmnt):
"""
Delete the corresponding comment.
"""
r = self.scheduler_db_meta.tables['comments']
- q = r.delete(r.c.id == cmnt_id)
+ q = r.delete(r.c.id == cmnt.id)
connection = self.engine.connect()
connection.execute(q)
Please sign in to comment.
Something went wrong with that request. Please try again.