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

Side by Side Diff: tests/test_packagerWebExt.py

Issue 29600577: Issue 5997 - Avoid including qunit files in release builds (Closed)
Patch Set: Make use of base_extension_path variable Created Nov. 7, 2017, 3:27 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « templates/testIndex.html.tmpl ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # This Source Code Form is subject to the terms of the Mozilla Public 1 # This Source Code Form is subject to the terms of the Mozilla Public
2 # License, v. 2.0. If a copy of the MPL was not distributed with this 2 # License, v. 2.0. If a copy of the MPL was not distributed with this
3 # file, You can obtain one at http://mozilla.org/MPL/2.0/. 3 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
4 4
5 import difflib 5 import difflib
6 import json 6 import json
7 import os 7 import os
8 import re 8 import re
9 import shutil 9 import shutil
10 import zipfile 10 import zipfile
(...skipping 213 matching lines...) Expand 10 before | Expand all | Expand 10 after
224 """Test-privatekey for signing chrome release-package.""" 224 """Test-privatekey for signing chrome release-package."""
225 return os.path.join(os.path.dirname(__file__), 'chrome_rsa.pem') 225 return os.path.join(os.path.dirname(__file__), 'chrome_rsa.pem')
226 226
227 227
228 @pytest.fixture 228 @pytest.fixture
229 def lib_files(tmpdir): 229 def lib_files(tmpdir):
230 files = packager.Files(['lib'], set()) 230 files = packager.Files(['lib'], set())
231 files['ext/a.js'] = 'require("./c.js");\nvar bar;' 231 files['ext/a.js'] = 'require("./c.js");\nvar bar;'
232 files['lib/b.js'] = 'var foo;' 232 files['lib/b.js'] = 'var foo;'
233 files['ext/c.js'] = 'var this_is_c;' 233 files['ext/c.js'] = 'var this_is_c;'
234 files['qunit/common.js'] = 'var qunit = {};'
235 files['qunit/tests/some_test.js'] = 'var passed = true;'
234 236
235 tmpdir.mkdir('lib').join('b.js').write(files['lib/b.js']) 237 tmpdir.mkdir('lib').join('b.js').write(files['lib/b.js'])
236 ext_dir = tmpdir.mkdir('ext') 238 ext_dir = tmpdir.mkdir('ext')
237 ext_dir.join('a.js').write(files['ext/a.js']) 239 ext_dir.join('a.js').write(files['ext/a.js'])
238 ext_dir.join('c.js').write(files['ext/c.js']) 240 ext_dir.join('c.js').write(files['ext/c.js'])
239 241 qunit_dir = tmpdir.mkdir('qunit')
242 qunit_dir.join('common.js').write(files['qunit/common.js'])
243 qunit_tests_dir = qunit_dir.mkdir('tests')
244 qunit_tests_dir.join('some_test.js').write(
245 files['qunit/tests/some_test.js']
246 )
240 return files 247 return files
241 248
242 249
243 def comparable_xml(xml): 250 def comparable_xml(xml):
244 """Create a nonambiguous representation of a given XML tree. 251 """Create a nonambiguous representation of a given XML tree.
245 252
246 Note that this function is not safe against ambiguous tags 253 Note that this function is not safe against ambiguous tags
247 containing differently distributed children, e.g.: 254 containing differently distributed children, e.g.:
248 255
249 '<a><b><c/></b><b><d/></b></a>' 256 '<a><b><c/></b><b><d/></b></a>'
(...skipping 226 matching lines...) Expand 10 before | Expand all | Expand 10 after
476 expected = os.path.join( 483 expected = os.path.join(
477 os.path.dirname(__file__), 484 os.path.dirname(__file__),
478 'expecteddata', 485 'expecteddata',
479 filename.format(name, ext), 486 filename.format(name, ext),
480 ) 487 )
481 488
482 assert_manifest_content( 489 assert_manifest_content(
483 package.read(os.path.join(folder, '{}.{}'.format(name, ext))), 490 package.read(os.path.join(folder, '{}.{}'.format(name, ext))),
484 expected, 491 expected,
485 ) 492 )
OLDNEW
« no previous file with comments | « templates/testIndex.html.tmpl ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld