Don't pickle registry flattened bases in the 'ro' attribute.

This avoids pickle references that are redundant with the '__bases__'
pickling.

1 files changed, 10 insertions, 1 deletions

diff --git a/src/zope/component/tests/test_persistentregistry.py b/src/zope/component/tests/test_persistentregistry.py
index e98277a..e39f537 100644
--- a/src/zope/component/tests/test_persistentregistry.py
+++ b/src/zope/component/tests/test_persistentregistry.py
@@ -96,7 +96,7 @@ class PersistentAdapterRegistryTests(unittest.TestCase):
         self.assertEqual(state['_provided'], {})
         self.assertEqual(state['_adapters'], [])
         self.assertEqual(state['_subscribers'], [])
-        self.assertEqual(state['ro'], [registry] + list(bases))
+        self.assertNotIn('ro', state)
 
     def test___getstate___skips_delegated_names(self):
         registry, jar, OID = self._makeOneWithJar()
@@ -112,6 +112,15 @@ class PersistentAdapterRegistryTests(unittest.TestCase):
         registry.__setstate__(state)
         self.assertTrue('_v_lookup' in registry.__dict__)
 
+    def test___setstate___rebuilds__ro(self):
+        from zope.component import globalSiteManager
+        bases = (globalSiteManager.adapters, globalSiteManager.utilities)
+        registry, jar, OID = self._makeOneWithJar(bases=bases)
+        state = registry.__getstate__()
+        registry.__setstate__(state)
+        self.assertEqual(registry.__bases__, bases)
+        self.assertEqual(registry.ro, [registry] + list(bases))
+
 
 class PersistentComponentsTests(unittest.TestCase):