Merge lp:~jameinel/launchpadlib/py27-config-ordering-1020667 into lp:launchpadlib

[John A Meinel]

Don't rely on ConfigParser's ordering of content.

It appears that launchpadlib itself doesn't run most of the doctests. So we only run them when running the Launchpad test suite.

It turns out that python-2.7 changed the ordering of config file entries for ConfigParser.SafeConfigParser, which a test case was relying on. (It shoudn't have, as in 2.6 it was strictly 'iter(dict)' ordering, which only happened to be stable enough.)

The new ordering is actually stable (OrderedDict), but to be compatible with 2.6 and 2.7, I just switched to adding a sorted() call around it.

[John A Meinel (jameinel)]

Note, if this is approved, we'll want a new release of launchpadlib, so that we can update the Launchpad dependencies, so that we can get a clean test suite run of Launchpad for python-2.7.

[Martin Packman (gz)]

Looks reasonable to me.

review: Approve

=== modified file 'src/launchpadlib/docs/introduction.txt'
--- src/launchpadlib/docs/introduction.txt	2011-01-06 13:37:16 +0000
+++ src/launchpadlib/docs/introduction.txt	2012-07-04 11:44:19 +0000
@@ -126,14 +126,14 @@
     >>> encrypted_file = StringIO()
     >>> stream = EncodedFile(encrypted_file, 'rot_13', 'ascii')
     >>> credentials.save(stream)
-    >>> print encrypted_file.getvalue()
+    >>> print ''.join(sorted(encrypted_file))
     [1]
+    <BLANKLINE>
+    <BLANKLINE>
+    npprff_frperg = grfg
+    npprff_gbxra = fnytnqb-punatr-nalguvat
     pbafhzre_frperg =
-    npprff_gbxra = fnytnqb-punatr-nalguvat
     pbafhzre_xrl = ynhapucnq-yvoenel
-    npprff_frperg = grfg
-    <BLANKLINE>
-    <BLANKLINE>
 
     >>> stream.seek(0)
     >>> credentials = Credentials()