Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: tests/test_packagerEdge.py

Issue 29542744: Issue 5666 - Replace 4th component of version in AppxManifest.xml with 0 (Closed)
Patch Set: Created Sept. 12, 2017, 7:49 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« packagerEdge.py ('K') | « packagerEdge.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/test_packagerEdge.py
===================================================================
--- a/tests/test_packagerEdge.py
+++ b/tests/test_packagerEdge.py
@@ -213,7 +213,7 @@
for release_build, pairs in [(False, devbuild), (True, release)]:
manifest = ET.fromstring(packagerEdge.create_appx_manifest(
- {'metadata': metadata},
+ {'metadata': metadata, 'version': '1.2.3.4'},
Sebastian Noack 2017/09/12 19:52:33 The previous change broke this test already, since
tlucas 2017/09/12 21:24:18 Ack, tox succeeds happily now. (I also added "tox"
files,
release_build=release_build))
for expression, modifiers, value in pairs:
« packagerEdge.py ('K') | « packagerEdge.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld