diff options
| author | Dan McGee <dan@archlinux.org> | 2011-01-22 23:11:33 +0100 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-01-22 23:29:32 +0100 |
| commit | a99e7272b8703e54b3c96ac0fdb7fe7eacfabbe8 (patch) | |
| tree | 65d3bf3f105c73627cb34331505ea0d6b9e5cf25 | |
| parent | b3d71bf7d0ef8f577114bc39abecdc7ade0395dd (diff) | |
| download | pacman-a99e7272b8703e54b3c96ac0fdb7fe7eacfabbe8.tar.gz pacman-a99e7272b8703e54b3c96ac0fdb7fe7eacfabbe8.tar.xz | |
pactest: sort repos by alpha order in config file
The order was non-deterministic before, and just happened to work for
sync023.py as it was written. Ensure there is some sort of predictable
ordering.
Signed-off-by: Dan McGee <dan@archlinux.org>
| -rw-r--r-- | test/pacman/tests/sync023.py | 3 | ||||
| -rwxr-xr-x | test/pacman/util.py | 5 |
2 files changed, 6 insertions, 2 deletions
diff --git a/test/pacman/tests/sync023.py b/test/pacman/tests/sync023.py index 8233ab73..3644c9de 100644 --- a/test/pacman/tests/sync023.py +++ b/test/pacman/tests/sync023.py @@ -15,7 +15,8 @@ for p in lp1, lp2, lp3, sp1, sp2, sp3, newp1: for p in lp1, lp2, lp3: self.addpkg2db("local", p) -self.addpkg2db("testing", newp1); +# repos are sorted in alpha order +self.addpkg2db("atesting", newp1); for p in sp1, sp2, sp3: self.addpkg2db("sync", p); diff --git a/test/pacman/util.py b/test/pacman/util.py index 652467ae..359b42bf 100755 --- a/test/pacman/util.py +++ b/test/pacman/util.py @@ -125,8 +125,11 @@ def mkcfgfile(filename, root, option, db): data.extend(["%s = %s" % (key, j) for j in value]) # Repositories - for key, value in db.iteritems(): + # sort by repo name so tests can predict repo order, rather than be + # subjects to the whims of python dict() ordering + for key in sorted(db.iterkeys()): if key != "local": + value = db[key] data.append("[%s]\n" \ "Server = file://%s" \ % (value.treename, |