Merge lp:~edwin-grubbs/launchpad/bug-462622-series-caching-test into lp:launchpad/db-devel

[Edwin Grubbs]

[Edwin Grubbs (edwin-grubbs)]

Summary
-------

Added test for populating and clearing the cache for
the distribution.serieses attribute.

Tests
-----

./bin/test -vvm lp.registry.tests.test_distribution

[Abel Deuring (adeuring)]

review: Approve (code)

=== modified file 'lib/lp/registry/tests/test_distribution.py'
--- lib/lp/registry/tests/test_distribution.py	2009-06-25 04:06:00 +0000
+++ lib/lp/registry/tests/test_distribution.py	2009-11-06 04:15:24 +0000
@@ -7,6 +7,8 @@
 
 import unittest
 
+from zope.security.proxy import removeSecurityProxy
+
 from lp.registry.tests.test_distroseries import (
     TestDistroSeriesCurrentSourceReleases)
 from lp.soyuz.interfaces.distributionsourcepackagerelease import (
@@ -45,6 +47,32 @@
             version='1.1', distroseries=self.current_series)
         self.assertCurrentVersion('1.1')
 
+    def test_distribution_series_cache(self):
+        distribution = removeSecurityProxy(
+            self.factory.makeDistribution('foo'))
+
+        # Not yet cached.
+        missing = object()
+        cached_series = getattr(distribution, '_cached_serieses', missing)
+        self.assertEqual(missing, cached_series)
+
+        # Now cached.
+        series = distribution.serieses
+        self.assertTrue(series is distribution._cached_serieses)
+
+        # Cache cleared.
+        distribution.newSeries(
+            name='bar', displayname='Bar', title='Bar', summary='',
+            description='', version='1', parent_series=None,
+            owner=self.factory.makePerson())
+        cached_series = getattr(distribution, '_cached_serieses', missing)
+        self.assertEqual(missing, cached_series)
+
+        # New cached value.
+        series = distribution.serieses
+        self.assertEqual(1, len(series))
+        self.assertTrue(series is distribution._cached_serieses)
+
 
 def test_suite():
     suite = unittest.TestSuite()