Mercurial > hg
changeset 23157:0e80564a6f3c
test-revert: put content, not keys, into 'combination'
By putting the file content rather than keys in the 'combination'
list, we restrict the knowledge of 'ctxcontent' and 'wccontent' to the
loop generating the combinations. That will make it easier to replace
the generation code.
| author | Martin von Zweigbergk <martinvonz@google.com> |
|---|---|
| date | Sat, 18 Oct 2014 22:23:19 -0700 |
| parents | e630c176ceda |
| children | 33a67fa048bf |
| files | tests/test-revert.t |
| diffstat | 1 files changed, 10 insertions(+), 9 deletions(-) [+] |
line wrap: on
line diff
--- a/tests/test-revert.t Fri Oct 17 09:02:30 2014 -0700 +++ b/tests/test-revert.t Sat Oct 18 22:23:19 2014 -0700 @@ -467,14 +467,16 @@ > combination = [] > for ctxkey, ctxvalue in ctxcontent.iteritems(): > for wckey in wccontent: - > if (ctxvalue[0] == ctxvalue[1] and 'revert' in wckey): + > base, parent = ctxvalue + > if (base == parent and 'revert' in wckey): > continue - > if not ctxvalue[0] and 'revert' in wckey: + > if not base and 'revert' in wckey: > continue - > if not ctxvalue[1] and 'deleted' in wckey: + > if not parent and 'deleted' in wckey: > continue > filename = "%s_%s" % (ctxkey, wckey) - > combination.append((filename, ctxkey, wckey)) + > combination.append((filename, base, parent, + > wccontent[wckey](ctxvalue))) > > # make sure we have stable output > combination.sort() @@ -484,16 +486,15 @@ > > # compute file content > content = [] - > for filename, ctxkey, wckey in combination: - > cc = ctxcontent[ctxkey] + > for filename, base, parent, wcc in combination: > if target == 'filelist': > print filename > elif target == 'base': - > content.append((filename, cc[0])) + > content.append((filename, base)) > elif target == 'parent': - > content.append((filename, cc[1])) + > content.append((filename, parent)) > elif target == 'wc': - > content.append((filename, wccontent[wckey](cc))) + > content.append((filename, wcc)) > else: > print >> sys.stderr, "unknown target:", target > sys.exit(1)