From 5044978accdb91b9e35a32f0dedb1873b1c7331d Mon Sep 17 00:00:00 2001 From: Helmut Merz Date: Mon, 15 Apr 2024 12:39:41 +0200 Subject: [PATCH] fix tracks.remove(): use track as parameter, not trackId --- scopes/storage/tracking.py | 4 ++-- tests/tlib_storage.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/scopes/storage/tracking.py b/scopes/storage/tracking.py index 108227e..e0b747f 100644 --- a/scopes/storage/tracking.py +++ b/scopes/storage/tracking.py @@ -171,8 +171,8 @@ class Container(object): return track.trackId return self.insert(track, withTrackId=True) - def remove(self, trackId): - stmt = self.table.delete().where(self.table.c.trackid == trackId) + def remove(self, track): + stmt = self.table.delete().where(self.table.c.trackid == track.trackId) n = self.session.execute(stmt).rowcount if n > 0: self.storage.mark_changed() diff --git a/tests/tlib_storage.py b/tests/tlib_storage.py index 1912bc6..cd9a9bd 100644 --- a/tests/tlib_storage.py +++ b/tests/tlib_storage.py @@ -50,7 +50,7 @@ def test_tracking(self, config): tr03 = storage.getItem('rec-31') self.assertEqual(tr03.trackId, 31) - n = tracks.remove(31) + n = tracks.remove(tr03) self.assertEqual(n, 1) self.assertEqual(tracks.get(31), None)