Merge lp:~mbruzek/charms/precise/limesurvey/tests into lp:charms/limesurvey
Proposed by
Matt Bruzek
Status: | Merged |
---|---|
Merged at revision: | 31 |
Proposed branch: | lp:~mbruzek/charms/precise/limesurvey/tests |
Merge into: | lp:charms/limesurvey |
Diff against target: |
70 lines (+60/-0) 2 files modified
tests/00-setup (+5/-0) tests/99-autogen (+55/-0) |
To merge this branch: | bzr merge lp:~mbruzek/charms/precise/limesurvey/tests |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
José Antonio Rey (community) | Approve | ||
Adam Israel (community) | Approve | ||
Review via email: mp+240950@code.launchpad.net |
Commit message
Adding tests for this charm
To post a comment you must log in.
Hi Matt,
I had the opportunity to review this MP today. Your tests are a welcome addition to this charm. The tests actually failed when I ran them through bundletester, but due to a problem with the charm unrelated to your additions, so you still have a +1 for me. I've filed a bug to address the broken install hook.