Merge lp:~mfisch/ubuntu-accomplishments-daemon/ubuntu-accomplishments-daemon-parse-fix into lp:ubuntu-accomplishments-daemon
|Merged at revision:||117|
|Diff against target:||239 lines (+51/-49) 2 files modified|
|To merge this branch:||bzr merge lp:~mfisch/ubuntu-accomplishments-daemon/ubuntu-accomplishments-daemon-parse-fix|
|Related bugs and blueprints:||
Description of the Change
Fixes the broken unit tests, which behaved differently depending on whether you had a daemon running on your system. (LP: 1030208
Fixes the daemon crash when there is a parse error in the .accomplishment file. I've modified the unit tests to account for this. You can see where I wrapped the Parse exceptions with a try/except block. Ideally I think we could move large chunks of this piece of code to separate functions.
If you test this, please run the unit tests with and without a daemon running on your system.