Rev 5778: Mention poolie's point about focusing tests. in file:///home/vila/src/bzr/experimental/config/

[Vincent Ladeuil]

At file:///home/vila/src/bzr/experimental/config/

------------------------------------------------------------
revno: 5778
revision-id: v.ladeuil+lp at free.fr-20110503092215-50y0xylk73fimj6h
parent: v.ladeuil+lp at free.fr-20110503091905-n4z9ev9y52xtp827
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: config-concrete-stores
timestamp: Tue 2011-05-03 11:22:15 +0200
message:
  Mention poolie's point about focusing tests.
-------------- next part --------------
=== modified file 'bzrlib/config.py'
--- a/bzrlib/config.py	2011-05-03 08:47:37 +0000
+++ b/bzrlib/config.py	2011-05-03 09:22:15 +0000
@@ -2337,6 +2337,11 @@
 # FIXME: global, bazaar, shouldn't that be 'user' instead or even
 # 'user_defaults' as opposed to 'user_overrides', 'system_defaults'
 # (/etc/bzr/bazaar.conf) and 'system_overrides' ? -- vila 2011-04-05
+
+# FIXME: Moreover, we shouldn't need classes for these stores either, factory
+# functions or a registry will make it easier and clearer for tests, focusing
+# on the relevant parts of the API that needs testing -- vila 20110503 (based
+# on a poolie's remark)
 class GlobalStore(LockableIniFileStore):
 
     def __init__(self, possible_transports=None):