Closed
Bug 641037
Opened 14 years ago
Closed 14 years ago
Validation fails with KeyError: 'path'
Categories
(addons.mozilla.org Graveyard :: Public Pages, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
6.0.1
People
(Reporter: krupa.mozbugs, Assigned: basta)
References
Details
(Whiteboard: [validator][post-freeze+])
Attachments
(1 file)
243.30 KB,
application/x-xpinstall
|
Details |
steps to reproduce:
1. Upload the attached test file
traceback:
"Traceback (most recent call last):
File "/data/amo_python/www/preview/zamboni/apps/devhub/tasks.py", line 27, in validator
result = _validator(upload.path)
File "/data/amo_python/www/preview/zamboni/apps/devhub/tasks.py", line 70, in _validator
spidermonkey=settings.SPIDERMONKEY)
File "/data/amo_python/www/preview/zamboni/vendor/src/amo-validator/validator/validate.py", line 37, in validate
validator.submain.prepare_package(bundle, path, expectation)
File "/data/amo_python/www/preview/zamboni/vendor/src/amo-validator/validator/submain.py", line 53, in prepare_package
output = test_package(err, package, path, expectation)
File "/data/amo_python/www/preview/zamboni/vendor/src/amo-validator/validator/submain.py", line 139, in test_package
return test_inner_package(err, package_contents, package)
File "/data/amo_python/www/preview/zamboni/vendor/src/amo-validator/validator/submain.py", line 227, in test_inner_package
test_func(err, package_contents, xpi_package)
File "/data/amo_python/www/preview/zamboni/vendor/src/amo-validator/validator/testcases/l10ncompleteness.py", line 180, in test_xpi
ref_name.startswith(split_target[0]))
File "/data/amo_python/www/preview/zamboni/vendor/src/amo-validator/validator/testcases/l10ncompleteness.py", line 420, in _aggregate_results
(locale["path"],
KeyError: 'path'
Comment 1•14 years ago
|
||
Since it's a crash, let's get it into this release. -> Basta
Assignee: nobody → mbasta
Whiteboard: [validator] → [validator][post-freeze+]
Assignee | ||
Comment 2•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 3•14 years ago
|
||
Oops, wrong link:
https://github.com/mozilla/amo-validator/pull/7
Reporter | ||
Comment 4•14 years ago
|
||
verified fixed @ https://addons.allizom.org/en-US/developers/upload/9129418f5f044be7a5172640ff6c0864
Status: RESOLVED → VERIFIED
Updated•9 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•