Merge pull request #137 from q384566678/value-test

cmd: Modify the assignment of parameters

Author: coolljt0725

cmd/oci-image-tool/create.go

@@ -39,17 +39,10 @@ func createHandle(context *cli.Context) error {
 		return fmt.Errorf("both src and dest must be provided")
 	}
 
-	var v bundleCmd
-	if context.IsSet("type") {
-		v.typ = context.String("type")
-	}
-	if context.IsSet("ref") {
-		v.ref = context.String("ref")
-	}
-	if context.IsSet("rootfs") {
-		v.root = context.String("rootfs")
-	} else {
-		v.root = "rootfs"
+	v := bundleCmd{
+		typ:  context.String("type"),
+		ref:  context.String("ref"),
+		root: context.String("rootfs"),
 	}
 
 	if v.typ == "" {

cmd/oci-image-tool/unpack.go

@@ -38,12 +38,9 @@ func unpackHandle(context *cli.Context) error {
 		return fmt.Errorf("both src and dest must be provided")
 	}
 
-	var v unpackCmd
-	if context.IsSet("type") {
-		v.typ = context.String("type")
-	}
-	if context.IsSet("ref") {
-		v.ref = context.String("ref")
+	v := unpackCmd{
+		typ: context.String("type"),
+		ref: context.String("ref"),
 	}
 
 	if v.typ == "" {

cmd/oci-image-tool/validate.go

@@ -48,12 +48,9 @@ func validateHandler(context *cli.Context) error {
 		return fmt.Errorf("no files specified")
 	}
 
-	if context.IsSet("type") {
-		v.typ = context.String("type")
-	}
-
-	if context.IsSet("ref") {
-		v.refs = context.StringSlice("ref")
+	v = validateCmd{
+		typ:  context.String("type"),
+		refs: context.StringSlice("ref"),
 	}
 
 	var errs []string