/testing

testing: cleanup MgoInstance #91

Merged
merged 1 commit into from Mar 7, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@davecheney @jujubot
@davecheney
Contributor

davecheney commented Mar 7, 2016

Remove MustDial, it was only called in one place where an error would be
more appropriate.

Reset now returns an error, rather than panicing.

(Review request: http://reviews.vapour.ws/r/4071/)
@davecheney
testing: cleanup MgoInstance 
Remove MustDial, it was only called in one place where an error would be
more appropriate.

Reset now returns an error, rather than panicing.
685c684
@davecheney
Contributor

davecheney commented Mar 7, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Mar 7, 2016

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

jujubot added a commit that referenced this pull request Mar 7, 2016

@jujubot
Merge pull request #91 from davecheney/cleanup-mgoinstance 
testing: cleanup MgoInstance

Remove MustDial, it was only called in one place where an error would be
more appropriate.

Reset now returns an error, rather than panicing.

(Review request: http://reviews.vapour.ws/r/4071/)
4d5a7d6

@jujubot jujubot merged commit 4d5a7d6 into juju:master Mar 7, 2016

@davecheney davecheney deleted the davecheney:cleanup-mgoinstance branch Mar 7, 2016

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