summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorumohnani8 <umohnani@redhat.com>2017-12-13 11:23:01 -0500
committerAtomic Bot <atomic-devel@projectatomic.io>2017-12-14 14:14:34 +0000
commitd8f099bb5a38b85bdea2bf6d99e0dbd0457ab666 (patch)
treee4056e7a08729ee82d4e01e169ee4706d1f5f5c7
parent05f4dd9f41707959ce801f9851c79232a1b79dca (diff)
downloadpodman-d8f099bb5a38b85bdea2bf6d99e0dbd0457ab666.tar.gz
podman-d8f099bb5a38b85bdea2bf6d99e0dbd0457ab666.tar.bz2
podman-d8f099bb5a38b85bdea2bf6d99e0dbd0457ab666.zip
Check for mutually exclusive flags
Error out if more than one mutually exclusive flags are passed in to kpod ps Signed-off-by: umohnani8 <umohnani@redhat.com> Closes: #128 Approved by: rhatdan
-rw-r--r--cmd/kpod/ps.go31
-rw-r--r--test/kpod_ps.bats129
2 files changed, 122 insertions, 38 deletions
diff --git a/cmd/kpod/ps.go b/cmd/kpod/ps.go
index 4148edf68..89fed991f 100644
--- a/cmd/kpod/ps.go
+++ b/cmd/kpod/ps.go
@@ -143,9 +143,8 @@ func psCmd(c *cli.Context) error {
return err
}
- // latest, and last are mutually exclusive.
- if c.Int("last") >= 0 && c.Bool("latest") {
- return errors.Errorf("last and latest are mutually exclusive")
+ if err := checkFlagsPassed(c); err != nil {
+ return errors.Wrapf(err, "error with flags passed")
}
runtime, err := getRuntime(c)
@@ -212,6 +211,32 @@ func psCmd(c *cli.Context) error {
return generatePsOutput(outputContainers, opts)
}
+// checkFlagsPassed checks if mutually exclusive flags are passed together
+func checkFlagsPassed(c *cli.Context) error {
+ // latest, and last are mutually exclusive.
+ if c.Int("last") >= 0 && c.Bool("latest") {
+ return errors.Errorf("last and latest are mutually exclusive")
+ }
+ // quiet, size, namespace, and format with Go template are mutually exclusive
+ flags := 0
+ if c.Bool("quiet") {
+ flags++
+ }
+ if c.Bool("size") {
+ flags++
+ }
+ if c.Bool("namespace") {
+ flags++
+ }
+ if c.IsSet("format") && c.String("format") != formats.JSONString {
+ flags++
+ }
+ if flags > 1 {
+ return errors.Errorf("quiet, size, namespace, and format with Go template are mutually exclusive")
+ }
+ return nil
+}
+
func generateContainerFilterFuncs(filter, filterValue string, runtime *libpod.Runtime) (func(container *libpod.Container) bool, error) {
switch filter {
case "id":
diff --git a/test/kpod_ps.bats b/test/kpod_ps.bats
index 3a5d94f59..977266f59 100644
--- a/test/kpod_ps.bats
+++ b/test/kpod_ps.bats
@@ -12,120 +12,179 @@ function teardown() {
}
@test "kpod ps with no containers" {
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps default" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps all flag" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -a
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps size flag" {
- skip "Need size flag implemented in container backend"
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --size
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --size"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps quiet flag" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --quiet
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ ctr_id="$output"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --quiet"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps latest flag" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps --latest
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps --latest"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps last flag" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${BB} ls
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls -s
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps --last 2
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${BB} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls -s"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps --last 2"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps no-trunc flag" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --no-trunc
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --no-trunc"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps namespace flag" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps --all --namespace
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps --all --namespace"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps namespace flag and format flag = json" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --ns --format json | python -m json.tool | grep namespace"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps without namespace flag and format flag = json" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --format json | python -m json.tool | grep namespace"
echo "$output"
[ "$status" -eq 1 ]
}
@test "kpod ps format flag = go template" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --format "table {{.ID}} {{.Image}} {{.Labels}}"
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --format 'table {{.ID}} {{.Image}} {{.Labels}}'"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps filter flag - ancestor" {
- ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --filter ancestor=${ALPINE}
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --filter ancestor=${ALPINE}"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps filter flag - id" {
- run ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls
- ctr_id="$output"
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --filter id="$ctr_id"
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} ls"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --filter id=$ctr_id"
echo "$output"
[ "$status" -eq 0 ]
}
@test "kpod ps filter flag - status" {
- run ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} sleep 99
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} sleep 99"
ctr_id="$output"
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --filter status=running
echo "$output"
[ "$status" -eq 0 ]
- run ${KPOD_BINARY} ${KPOD_OPTIONS} stop $ctr_id
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --filter status=running"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} stop $ctr_id"
+ echo "$output"
+ [ "$status" -eq 0 ]
}
@test "kpod ps short options" {
- run ${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} sleep 99
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} sleep 99"
+ ctr_id="$output"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -aq"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} stop $ctr_id"
+ echo "$output"
+ [ "$status" -eq 0 ]
+}
+
+@test "kpod ps with mutually exclusive flags" {
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} run -d ${ALPINE} sleep 99"
ctr_id="$output"
- run bash -c ${KPOD_BINARY} ${KPOD_OPTIONS} ps -aqs
echo "$output"
[ "$status" -eq 0 ]
- run ${KPOD_BINARY} ${KPOD_OPTIONS} stop $ctr_id
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -aqs"
+ echo "$output"
+ [ "$status" -ne 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --ns -s"
+ echo "$output"
+ [ "$status" -ne 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --ns format {{.ID}}"
+ echo "$output"
+ [ "$status" -ne 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} ps -a --ns --format json"
+ echo "$output"
+ [ "$status" -eq 0 ]
+ run bash -c "${KPOD_BINARY} ${KPOD_OPTIONS} stop $ctr_id"
+ echo "$output"
+ [ "$status" -eq 0 ]
}