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

Unified Diff: tests/test_packagerEdge.py

Issue 29756673: Noissue - Adapt best practices for trailing commas (buildtools) (Closed)
Patch Set: Re-run script on Python 2, added flake8-commas extension Created April 19, 2018, 2:09 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
« no previous file with comments | « releaseAutomation.py ('k') | tests/test_packagerWebExt.py » ('j') | tox.ini » ('J')
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
@@ -25,7 +25,7 @@
file.get('Name'): {
'size': file.get('Size'),
'lfhsize': file.get('LfhSize'),
- 'blocks': [b.get('Hash') for b in file]
+ 'blocks': [b.get('Hash') for b in file],
}
for file in ET.fromstring(xml_data)
}
@@ -36,7 +36,7 @@
assert blockmap['Extension\\foo.xml'] == {
'size': '500',
'lfhsize': '47',
- 'blocks': ['Vhwfmzss1Ney+j/ssR2QVISvFyMNBQeS2P+UjeE/di0=']
+ 'blocks': ['Vhwfmzss1Ney+j/ssR2QVISvFyMNBQeS2P+UjeE/di0='],
}
assert blockmap['Extension\\bar.png'] == {
'size': '100000',
@@ -44,7 +44,7 @@
'blocks': [
'KPW2SxeEikUEGhoKmKxruUSexKun0bGXMppOqUFrX5E=',
'KQHnov1SZ1z34ttdDUjX2leYtpIIGndUVoUteieS2cw=',
- ]
+ ],
}
@@ -79,9 +79,9 @@
'json': 'application/json',
'otf': 'application/octet-stream',
'png': 'image/png',
- 'xml': 'application/xml'
+ 'xml': 'application/xml',
}
assert ctm_dict['overrides'] == {
'/AppxBlockMap.xml': 'application/vnd.ms-appx.blockmap+xml',
- '/AppxManifest.xml': 'application/vnd.ms-appx.manifest+xml'
+ '/AppxManifest.xml': 'application/vnd.ms-appx.manifest+xml',
}
« no previous file with comments | « releaseAutomation.py ('k') | tests/test_packagerWebExt.py » ('j') | tox.ini » ('J')

Powered by Google App Engine
This is Rietveld