/juju

cmd/juju/space: Supercommand and create, 100% tests coverage #1976

Merged
merged 6 commits into from Mar 30, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@dimitern @jameinel @dooferlad @jujubot
@dimitern
Contributor

dimitern commented Mar 30, 2015

Based on dooferlad's branch implementing juju space create, I've
experimented a bit to see how to simplify the tests, make it easier to
write new subcommands and their tests, using a stub API.
@jameinel
Owner

jameinel commented Mar 30, 2015

Looks nice to me
@dooferlad
Contributor

dooferlad commented Mar 30, 2015

Like it!
@dimitern
Contributor

dimitern commented Mar 30, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Mar 30, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request Mar 30, 2015

@jujubot
Merge pull request #1976 from dimitern/net-cli-spaces 
cmd/juju/space: Supercommand and create, 100% tests coverage

Based on dooferlad's branch implementing juju space create, I've
experimented a bit to see how to simplify the tests, make it easier to
write new subcommands and their tests, using a stub API.
dc2f9bd

@jujubot jujubot merged commit dc2f9bd into juju:net-cli Mar 30, 2015

@dimitern dimitern deleted the dimitern:net-cli-spaces branch Mar 30, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment