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

Unified Diff: tests/test_packagerWebExt.py

Issue 29609559: Issue 6021 - Refactoring build.py (Closed) Base URL: https://hg.adblockplus.org/buildtools/file/79688f4a4aff
Patch Set: Only construct subcommands if they would actually be available Created Nov. 17, 2017, 1:30 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 | « build.py ('k') | tox.ini » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/test_packagerWebExt.py
diff --git a/tests/test_packagerWebExt.py b/tests/test_packagerWebExt.py
index 1b22d1162bcde2c00fb716e25d968c2be8535cbb..cef155e087e19c051f467f7b64773c6e0f9bff7b 100644
--- a/tests/test_packagerWebExt.py
+++ b/tests/test_packagerWebExt.py
@@ -18,7 +18,7 @@ from Crypto.Hash import SHA
from buildtools import packager
from buildtools.packagerChrome import defaultLocale
-from buildtools.build import processArgs
+from buildtools.build import process_args
LOCALES_MODULE = {
'test.Foobar': {
@@ -134,12 +134,12 @@ def run_webext_build(ext_type, build_opt, srcdir, keyfile=None):
'release_build': ['build', '-r'],
}
- args = ['build.py', '-t', ext_type] + cmd_mapping[build_opt]
+ args = cmd_mapping[build_opt] + ['-t', ext_type]
if keyfile:
args += ['-k', keyfile]
- processArgs(str(srcdir), args)
+ process_args(str(srcdir), *args)
def locale_files(languages, rootfolder, srcdir):
« no previous file with comments | « build.py ('k') | tox.ini » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld