Closed Bug 641037 Opened 13 years ago Closed 13 years ago

Validation fails with KeyError: 'path'

Categories

(addons.mozilla.org Graveyard :: Public Pages, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: krupa.mozbugs, Assigned: basta)

References

Details

(Whiteboard: [validator][post-freeze+])

Attachments

(1 file)

243.30 KB, application/x-xpinstall
Details
Attached file test file
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'
Since it's a crash, let's get it into this release.  -> Basta
Assignee: nobody → mbasta
Whiteboard: [validator] → [validator][post-freeze+]
Fixed:

https://github.com/mozilla/amo-validator/pull/6
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
verified fixed @ https://addons.allizom.org/en-US/developers/upload/9129418f5f044be7a5172640ff6c0864
Status: RESOLVED → VERIFIED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: