lib/ldb-samba: Add comment dicouraging use of schemaUpgradeInProgress

[obnox/samba/samba-obnox.git] / lib / ldb-samba / tests / match_rules.py

diff --git a/lib/ldb-samba/tests/match_rules.py b/lib/ldb-samba/tests/match_rules.py
index cb18248f169363f6f33f636c675bfb12c69aa7aa..6b31392df1dcb676ae96832f8cfae27e60ec2f13 100755 (executable)
--- a/lib/ldb-samba/tests/match_rules.py
+++ b/lib/ldb-samba/tests/match_rules.py
@@ -209,6 +209,16 @@ class MatchRulesTests(samba.tests.TestCase):
         # |-> c1
         # |   |->c2
         # |   |  |->u1
+
+        #
+        # While appropriate for this test, this is NOT a good practice
+        # in general.  This is only done here because the alternative
+        # is to make a schema modification.
+        #
+        # IF/WHEN Samba protects this attribute better, this
+        # particular part of the test can be removed, as the same code
+        # is covered by the addressBookRoots2 case well enough.
+        #
         m = Message()
         m.dn = Dn(self.ldb, "")
         m["e1"] = MessageElement("1", FLAG_MOD_REPLACE, "schemaUpgradeInProgress")