Nate Finch <nate.fi...@canonical.com> writes:

Thanks for highlighting these, Nate!

> Here's some especially bad low hanging fruit (note these are the times
> for single tests, not full suites):

If anyone is so inclined, I have modified the deploy command so that it is unit 
testable, and written some unit tests[1] for it to help guide future 
development; they are 100% in-memory (aside from some suite set up which likely 
doesn't need to be there). If you have some free time (ha!), take one of the 
tests from `DeploySuite` and convert it over to a true in-memory unit test.

[1] - 
https://github.com/juju/juju/blob/staging/cmd/juju/application/deploy_test.go#L1243

-- 
Katherine

-- 
Juju-dev mailing list
Juju-dev@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/juju-dev

Reply via email to