summaryrefslogtreecommitdiffhomepage
path: root/runsc/cmd/delete_test.go
diff options
context:
space:
mode:
authorFabricio Voznika <fvoznika@google.com>2018-05-29 17:57:26 -0700
committerShentubot <shentubot@google.com>2018-05-29 17:58:12 -0700
commit812e83d3bbb99d4fa1ece4712a1ac85e84fe6ec3 (patch)
tree559b9a5a5fc2b37899bf2ffb915e1ed4eadf6bfa /runsc/cmd/delete_test.go
parentc5dc873e441706e8aaff7389e26c862f1386c6a8 (diff)
Supress error when deleting non-existing container with --force
This addresses the first issue reported in #59. CRI-O expects runsc to return success to delete when --force is used with a non-existing container. PiperOrigin-RevId: 198487418 Change-Id: If7660e8fdab1eb29549d0a7a45ea82e20a1d4f4a
Diffstat (limited to 'runsc/cmd/delete_test.go')
-rw-r--r--runsc/cmd/delete_test.go41
1 files changed, 41 insertions, 0 deletions
diff --git a/runsc/cmd/delete_test.go b/runsc/cmd/delete_test.go
new file mode 100644
index 000000000..928e9ee2c
--- /dev/null
+++ b/runsc/cmd/delete_test.go
@@ -0,0 +1,41 @@
+// Copyright 2018 Google Inc.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+package cmd
+
+import (
+ "io/ioutil"
+ "testing"
+
+ "gvisor.googlesource.com/gvisor/runsc/boot"
+)
+
+func TestNotFound(t *testing.T) {
+ ids := []string{"123"}
+ dir, err := ioutil.TempDir("", "metadata")
+ if err != nil {
+ t.Fatalf("error creating dir: %v", err)
+ }
+ conf := &boot.Config{RootDir: dir}
+
+ d := Delete{}
+ if err := d.execute(ids, conf); err == nil {
+ t.Error("Deleting non-existend container should have failed")
+ }
+
+ d = Delete{force: true}
+ if err := d.execute(ids, conf); err != nil {
+ t.Errorf("Deleting non-existend container with --force should NOT have failed: %v", err)
+ }
+}