<div dir="ltr"><div>Testing additions for logstash-agent and squid-reverseproxy</div><div><br></div><a href="https://code.launchpad.net/~nicopace/charms/trusty/squid-reverseproxy/all-tests/+merge/251999">https://code.launchpad.net/~nicopace/charms/trusty/squid-reverseproxy/all-tests/+merge/251999</a><br clear="all"><div><br></div><div><a href="https://code.launchpad.net/~nicopace/charms/trusty/logstash-agent/all-tests/+merge/251311d">https://code.launchpad.net/~nicopace/charms/trusty/logstash-agent/all-tests/+merge/251311d</a><br></div><div><br></div><div>Both nacked due to errors in testing.  </div><div><br></div><div>I took the time to investigate the errors and failures and attempted to fix the errors with logstash-agent and contacted the maintainers wrt dependency skew issues wrt squid-reverseproxy. </div><div><br></div><div>I would like to state while adding tests to illuminate issues is good, adding tests to charms who's existing tests have errors makes for more and I believe pointless work for reviewers.  I believe if you find a bug and want to commit a test, commit them to a personal namespace branch, not the charmstore.</div><div><br></div><div>For basic stuff we can generate tests for based on charm metadata, we should automate tests and provide them (much like we provide charm proof), rather than adding additional burden to the review process in LOC added to charms.  </div><div><br></div><div>-w</div><div><br></div><div><br></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div><span style="font-family:'courier new',monospace">---------------<br>D. Whit Morriss<br></span></div><div><span style="font-family:'courier new',monospace">Developer, Juju Ecosystem<br></span></div></div><span style="font-family:'courier new',monospace">Canonical USA</span><br></div></div>
</div>