/juju

cmd/jujud/agent: fix test snafu #3828

Merged
merged 1 commit into from Nov 26, 2015

Conversation

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

davecheney commented Nov 26, 2015

6209795 accidentally excluded Upgrade test because this package
uses both internal and external tests.

(Review request: http://reviews.vapour.ws/r/3245/)
cmd/jujud/agent/package_test.go
@@ -1,7 +1,7 @@
// Copyright 2015 Canonical Ltd.
// Licensed under the AGPLv3, see LICENCE file for details.
-package agent_test
+package agent // not agent_test for unknown reasons
@mjs

mjs Nov 26, 2015

Contributor

The reasons aren't unknown, just bad :)

@davecheney
cmd/jujud/agent: fix test snafu 
6209795 accidentally excluded Upgrade test because this package
uses both internal and external tests.
c830946
@davecheney
Contributor

davecheney commented Nov 26, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Nov 26, 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 Nov 26, 2015

@jujubot
Merge pull request #3828 from davecheney/fix-agent-test-snafu 
cmd/jujud/agent: fix test snafu

6209795 accidentally excluded Upgrade test because this package
uses both internal and external tests.

(Review request: http://reviews.vapour.ws/r/3245/)
3956dec

@jujubot jujubot merged commit 3956dec into juju:master Nov 26, 2015

@davecheney davecheney deleted the davecheney:fix-agent-test-snafu branch Nov 26, 2015

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