From d48392a35b157612d28c48eb8e4fad1272fa1442 Mon Sep 17 00:00:00 2001 From: Sebastiaan van Stijn Date: Mon, 5 Mar 2018 23:39:07 +0100 Subject: [PATCH] Fix definition of ErrSystem type Signed-off-by: Sebastiaan van Stijn --- errdefs/defs.go | 2 +- errdefs/helpers_test.go | 13 +++++++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/errdefs/defs.go b/errdefs/defs.go index 615ff3c798..e6a2275b2d 100644 --- a/errdefs/defs.go +++ b/errdefs/defs.go @@ -35,7 +35,7 @@ type ErrForbidden interface { // ErrSystem signals that some internal error occurred. // An example of this would be a failed mount request. type ErrSystem interface { - ErrSystem() + System() } // ErrNotModified signals that an action can't be performed because it's already in the desired state diff --git a/errdefs/helpers_test.go b/errdefs/helpers_test.go index 46c8f18718..f1c88704ca 100644 --- a/errdefs/helpers_test.go +++ b/errdefs/helpers_test.go @@ -179,3 +179,16 @@ func TestUnavailable(t *testing.T) { t.Fatalf("causual should be errTest, got: %v", cause) } } + +func TestSystem(t *testing.T) { + if IsSystem(errTest) { + t.Fatalf("did not expect system error, got %T", errTest) + } + e := System(errTest) + if !IsSystem(e) { + t.Fatalf("expected system error, got %T", e) + } + if cause := e.(causal).Cause(); cause != errTest { + t.Fatalf("causual should be errTest, got: %v", cause) + } +}