From cd0b18c24f9835e8bb258b906800f0975fbe9e2f Mon Sep 17 00:00:00 2001
From: Valentin Rothberg <rothberg@redhat.com>
Date: Tue, 23 Jun 2020 11:17:42 +0200
Subject: image load: no args required

Disable the args requirement of `image load`.  Instead of requiring a
lower bound, we really need an upper one with at most 1 argument.
Extend the system tests to prevent future regressions.

Fixes: #6718
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
---
 cmd/podman/images/load.go |  2 +-
 test/system/120-load.bats | 18 +++++++++++++++++-
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/cmd/podman/images/load.go b/cmd/podman/images/load.go
index a984ad81f..115e9a070 100644
--- a/cmd/podman/images/load.go
+++ b/cmd/podman/images/load.go
@@ -30,7 +30,7 @@ var (
 	}
 
 	imageLoadCommand = &cobra.Command{
-		Args:  cobra.MinimumNArgs(1),
+		Args:  loadCommand.Args,
 		Use:   loadCommand.Use,
 		Short: loadCommand.Short,
 		Long:  loadCommand.Long,
diff --git a/test/system/120-load.bats b/test/system/120-load.bats
index 15df6adec..f290c1888 100644
--- a/test/system/120-load.bats
+++ b/test/system/120-load.bats
@@ -44,6 +44,11 @@ verify_iid_and_name() {
     run_podman load < $archive
     verify_iid_and_name "<none>:<none>"
 
+    # Same as above, using stdin but with `podman image load`
+    run_podman rmi $iid
+    run_podman image load < $archive
+    verify_iid_and_name "<none>:<none>"
+
     # Cleanup: since load-by-iid doesn't preserve name, re-tag it;
     # otherwise our global teardown will rmi and re-pull our standard image.
     run_podman tag $iid $img_name
@@ -57,9 +62,14 @@ verify_iid_and_name() {
     # Load using -i; this time the image should be tagged.
     run_podman load -i $archive
     verify_iid_and_name $img_name
+    run_podman rmi $iid
 
-    # Same as above, using stdin
+    # Also make sure that `image load` behaves the same.
+    run_podman image load -i $archive
+    verify_iid_and_name $img_name
     run_podman rmi $iid
+
+    # Same as above, using stdin
     run_podman load < $archive
     verify_iid_and_name $img_name
 }
@@ -97,4 +107,10 @@ verify_iid_and_name() {
        "Diagnostic from 'podman load' without redirection or -i"
 }
 
+@test "podman load - at most 1 arg(s)" {
+    run_podman 125 load 1 2 3
+    is "$output" \
+       "Error: accepts at most 1 arg(s), received 3"
+}
+
 # vim: filetype=sh
-- 
cgit v1.2.3-54-g00ecf