configglue team mailing list archive

[Ricardo Kirkner <ricardo.kirkner@xxxxxxxxxxxxx>]

Ricardo Kirkner has proposed merging lp:~ricardokirkner/configglue/659178-main-invalid-section into lp:configglue.

Requested reviews:
  Configglue developers (configglue)
Related bugs:
  #659178 __main__ shows up as invalid section
  https://bugs.launchpad.net/bugs/659178

For more details, see:
https://code.launchpad.net/~ricardokirkner/configglue/659178-main-invalid-section/+merge/50533

added test to show that bug is fixed.

__main__ section is not always included when validation reports errors
-- 
https://code.launchpad.net/~ricardokirkner/configglue/659178-main-invalid-section/+merge/50533
Your team Configglue developers is requested to review the proposed merge of lp:~ricardokirkner/configglue/659178-main-invalid-section into lp:configglue.

=== modified file 'tests/pyschema/test_parser.py'
--- tests/pyschema/test_parser.py	2010-12-18 22:32:43 +0000
+++ tests/pyschema/test_parser.py	2011-02-20 22:40:14 +0000
@@ -825,6 +825,15 @@
 
         self.assertFalse(self.parser.is_valid())
 
+    def test_parse_invalid_section_with_report(self):
+        config = StringIO("[bar]\nbaz=foo")
+        self.parser.readfp(config)
+
+        valid, errors = self.parser.is_valid(report=True)
+        self.assertFalse(valid)
+        self.assertEqual(errors,
+            [u'Sections in configuration do not match schema: bar'])
+
     def test_different_sections(self):
         config = StringIO("[__main__]\nfoo=1\n[bar]\nbaz=2")
         self.parser.readfp(config)