Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(336)

Unified Diff: state/apiserver/errors_test.go

Issue 10044043: state/apiserver: Split Machiner into subpackage (Closed)
Patch Set: state/apiserver: Split Machiner into subpackage Created 11 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « state/apiserver/common/interfaces.go ('k') | state/apiserver/export_test.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: state/apiserver/errors_test.go
=== modified file 'state/apiserver/errors_test.go'
--- state/apiserver/errors_test.go 2013-06-05 09:07:49 +0000
+++ state/apiserver/errors_test.go 2013-06-06 17:09:49 +0000
@@ -9,7 +9,7 @@
"launchpad.net/juju-core/errors"
"launchpad.net/juju-core/state"
"launchpad.net/juju-core/state/api"
- "launchpad.net/juju-core/state/apiserver"
+ "launchpad.net/juju-core/state/apiserver/common"
"launchpad.net/juju-core/testing"
)
@@ -41,25 +41,25 @@
err: state.ErrUnitHasSubordinates,
code: api.CodeUnitHasSubordinates,
}, {
- err: apiserver.ErrBadId,
+ err: common.ErrBadId,
code: api.CodeNotFound,
}, {
- err: apiserver.ErrBadCreds,
- code: api.CodeUnauthorized,
-}, {
- err: apiserver.ErrPerm,
- code: api.CodeUnauthorized,
-}, {
- err: apiserver.ErrNotLoggedIn,
- code: api.CodeUnauthorized,
-}, {
- err: apiserver.ErrUnknownWatcher,
+ err: common.ErrBadCreds,
+ code: api.CodeUnauthorized,
+}, {
+ err: common.ErrPerm,
+ code: api.CodeUnauthorized,
+}, {
+ err: common.ErrNotLoggedIn,
+ code: api.CodeUnauthorized,
+}, {
+ err: common.ErrUnknownWatcher,
code: api.CodeNotFound,
}, {
err: &state.NotAssignedError{&state.Unit{}}, // too sleazy?! nah..
code: api.CodeNotAssigned,
}, {
- err: apiserver.ErrStoppedWatcher,
+ err: common.ErrStoppedWatcher,
code: api.CodeStopped,
}, {
err: &state.HasAssignedUnitsError{"42", []string{"a"}},
@@ -67,16 +67,19 @@
}, {
err: stderrors.New("an error"),
code: "",
+}, {
+ err: nil,
+ code: "",
}}
func (s *errorsSuite) TestErrorTransform(c *C) {
for _, t := range errorTransformTests {
- err1 := apiserver.ServerError(t.err)
- c.Assert(err1.Error(), Equals, t.err.Error())
- if t.code != "" {
- c.Assert(api.ErrCode(err1), Equals, t.code)
+ err1 := common.ServerError(t.err)
+ if t.err == nil {
+ c.Assert(err1, IsNil)
} else {
- c.Assert(err1, Equals, t.err)
+ c.Assert(err1.Message, Equals, t.err.Error())
+ c.Assert(err1.Code, Equals, t.code)
}
}
}
« no previous file with comments | « state/apiserver/common/interfaces.go ('k') | state/apiserver/export_test.go » ('j') | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b