add test for empty patch file

pkg/patch/transformer/patchjson6902json.go

@@ -19,7 +19,7 @@ package transformer
 import (
 	"fmt"
 
-	"github.com/evanphx/json-patch"
+	jsonpatch "github.com/evanphx/json-patch"
 	"github.com/pkg/errors"
 	"sigs.k8s.io/kustomize/pkg/resid"
 	"sigs.k8s.io/kustomize/pkg/resmap"
@@ -42,6 +42,11 @@ func newPatchJson6902JSONTransformer(
 	id resid.ResId, rawOp []byte) (transformers.Transformer, error) {
 	op := rawOp
 	var err error
+
+	if len(op) == 0 {
+		return nil, fmt.Errorf("json patch file is empty %v", id)
+	}
+
 	if !isJsonFormat(op) {
 		// if it isn't JSON, try to parse it as YAML
 		op, err = yaml.YAMLToJSON(rawOp)

pkg/patch/transformer/patchjson6902json_test.go

@@ -163,3 +163,40 @@ func TestJsonPatchJSONTransformer_UnHappyTransform(t *testing.T) {
 		t.Fatalf("expected error didn't happen, but got %v", err)
 	}
 }
+
+func TestJsonPatchJSONTransformer_EmptyPatchFile(t *testing.T) {
+	rf := resource.NewFactory(
+		kunstruct.NewKunstructuredFactoryImpl())
+	id := resid.NewResId(deploy, "deploy1")
+	base := resmap.ResMap{
+		id: rf.FromMap(
+			map[string]interface{}{
+				"apiVersion": "apps/v1",
+				"kind":       "Deployment",
+				"metadata": map[string]interface{}{
+					"name": "deploy1",
+				},
+				"spec": map[string]interface{}{
+					"template": map[string]interface{}{},
+				},
+			}),
+	}
+
+	operations := []byte(``)
+
+	jpt, err := newPatchJson6902JSONTransformer(id, operations)
+
+	if err == nil {
+		t.Fatalf("unexpected error : %v", err)
+
+		if jpt != nil {
+			err = jpt.Transform(base)
+		}
+	}
+
+	if err != nil {
+		if !strings.HasPrefix(err.Error(), "json patch file is empty") {
+			t.Fatalf("expected error didn't happen, but got %v", err)
+		}
+	}
+}