From: Torsten Werner Date: Wed, 19 Jan 2011 09:35:03 +0000 (+0100) Subject: Extend test_poolfiles(). X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=319aa4d75191d281e64332a846ac9ca20cce5396;p=dak Extend test_poolfiles(). Signed-off-by: Torsten Werner --- diff --git a/tests/dbtest_packages.py b/tests/dbtest_packages.py index c06d9442..439c9a2e 100755 --- a/tests/dbtest_packages.py +++ b/tests/dbtest_packages.py @@ -109,6 +109,17 @@ class PackageTestCase(DBDakTestCase): self.session.add_all(self.file.values()) def test_poolfiles(self): + ''' + Test the relation of the classes PoolFile and Location. + + The code needs some explaination. The property Location.files is not a + list as in other relations because such a list would become rather + huge. It is a query object that can be queried, filtered, and iterated + as usual. But list like methods like append() and remove() are + supported as well which allows code like: + + somelocation.files.append(somefile) + ''' self.setup_locations() self.setup_poolfiles() location = self.session.query(Location)[0] @@ -118,6 +129,17 @@ class PackageTestCase(DBDakTestCase): filter(PoolFile.filename.like('%/hello/hello%')).one() self.assertEqual('main/h/hello/hello_2.2-2.dsc', poolfile.filename) self.assertEqual(location, poolfile.location) + location.files.remove(self.file['sl']) + # TODO: deletion should cascade automatically + self.session.delete(self.file['sl']) + self.session.refresh(location) + self.assertEqual(1, location.files.count()) + # please note that we intentionally do not specify 'location' here + self.file['sl'] = PoolFile(filename = 'main/s/sl/sl_3.03-16.dsc', \ + filesize = 0, md5sum = '') + location.files.append(self.file['sl']) + self.session.refresh(location) + self.assertEqual(2, location.files.count()) def setup_maintainers(self): 'create some Maintainer objects'