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

Unified Diff: packagerEdge.py

Issue 29600577: Issue 5997 - Avoid including qunit files in release builds (Closed)
Patch Set: Avoid bundling tests if testScripts option is missing Created Nov. 7, 2017, 4:04 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
Index: packagerEdge.py
diff --git a/packagerEdge.py b/packagerEdge.py
index 8006a9ef5ef96f5702307ffb72f83e9f2ef304e8..766baae91807072a50a5bfa7aa91b723071f99dc 100644
--- a/packagerEdge.py
+++ b/packagerEdge.py
@@ -178,7 +178,8 @@ def createBuild(baseDir, type='edge', outFile=None, # noqa: preserve API.
files.read(baseDir)
if metadata.has_section('bundles'):
- packagerChrome.create_bundles(params, files)
+ bundle_tests = devenv and metadata.has_option('general', 'testScripts')
+ packagerChrome.create_bundles(params, files, bundle_tests)
if metadata.has_section('preprocess'):
files.preprocess(metadata.options('preprocess'), {'needsExt': True})

Powered by Google App Engine
This is Rietveld