// Copyright 2019 The Kubernetes Authors.
// SPDX-License-Identifier: Apache-2.0

package filters_test

import (
	"bytes"
	"io"
	"os"
	"path/filepath"
	"strings"
	"testing"

	"github.com/stretchr/testify/assert"
	"github.com/stretchr/testify/require"
	"sigs.k8s.io/kustomize/kyaml/kio"
	. "sigs.k8s.io/kustomize/kyaml/kio/filters"
	"sigs.k8s.io/kustomize/kyaml/kio/filters/testyaml"
	"sigs.k8s.io/kustomize/kyaml/yaml"
)

func TestFormatInput_FixYaml1_1Compatibility(t *testing.T) {
	y := `
apiVersion: apps/v1
kind: Deployment
metadata:
  name: foo
  labels:
    foo: on
    foo2: hello1
  annotations:
    bar: 1
    bar2: hello2
spec:
  template:
    spec:
      containers:
      - name: nginx
        image: nginx:1.0.0
        args:
        - on
        - 1
        - hello
        ports:
        - name: http
          targetPort: 80
          containerPort: 80
`

	// keep the style on values that parse as non-string types
	expected := `apiVersion: apps/v1
kind: Deployment
metadata:
  name: foo
  labels:
    foo: "on"
    foo2: hello1
  annotations:
    bar: "1"
    bar2: hello2
spec:
  template:
    spec:
      containers:
      - name: nginx
        image: nginx:1.0.0
        args:
        - "on"
        - "1"
        - hello
        ports:
        - name: http
          targetPort: 80
          containerPort: 80
`

	buff := &bytes.Buffer{}
	err := kio.Pipeline{
		Inputs: []kio.Reader{&kio.ByteReader{Reader: strings.NewReader(y)}},
		Filters: []kio.Filter{FormatFilter{
			UseSchema: true,
		}},
		Outputs: []kio.Writer{kio.ByteWriter{Writer: buff}},
	}.Execute()
	require.NoError(t, err)
	assert.Equal(t, expected, buff.String())
}

func TestFormat_UnsortedInput_No_Schema(t *testing.T) {
	y := `
apiVersion: apps/v1
spec:
  template:
    spec:
      containers:
      - name: nginx
        image: nginx:1.0.0
        args:
        - on
        - 1
        - hello
        ports:
        - name: http
          targetPort: 80
          containerPort: 80
kind: Deployment
metadata:
  name: foo
  labels:
    foo: on
    foo2: hello1
  annotations:
    bar: 1
    bar2: hello2
`

	// keep the style on values that parse as non-string types
	expected := `apiVersion: apps/v1
kind: Deployment
metadata:
  name: foo
  labels:
    foo: on
    foo2: hello1
  annotations:
    bar: 1
    bar2: hello2
spec:
  template:
    spec:
      containers:
      - name: nginx
        image: nginx:1.0.0
        args:
        - on
        - 1
        - hello
        ports:
        - name: http
          targetPort: 80
          containerPort: 80
`

	buff := &bytes.Buffer{}
	err := kio.Pipeline{
		Inputs:  []kio.Reader{&kio.ByteReader{Reader: strings.NewReader(y)}},
		Filters: []kio.Filter{FormatFilter{}},
		Outputs: []kio.Writer{kio.ByteWriter{Writer: buff}},
	}.Execute()
	require.NoError(t, err)
	assert.Equal(t, expected, buff.String())
}

func TestFormatInput_PostprocessStyle(t *testing.T) {
	y := `
apiVersion: v1
kind: Foo
metadata:
  name: foo
spec:
  notBoolean: "true"
  notBoolean2: "on"
  isBoolean: on
  isBoolean2: true
  notInt: "12345"
  isInt: 12345
  isString1: hello world
  isString2: "hello world"
`

	// keep the style on values that parse as non-string types
	expected := `apiVersion: v1
kind: Foo
metadata:
  name: foo
spec:
  isBoolean: on
  isBoolean2: true
  isInt: 12345
  isString1: hello world
  isString2: hello world
  notBoolean: "true"
  notBoolean2: "on"
  notInt: "12345"
`

	buff := &bytes.Buffer{}
	err := kio.Pipeline{
		Inputs: []kio.Reader{&kio.ByteReader{Reader: strings.NewReader(y)}},
		Filters: []kio.Filter{FormatFilter{
			UseSchema: true,
			Process: func(n *yaml.Node) error {
				if yaml.IsYaml1_1NonString(n) {
					// don't change these styles, they are important for backwards compatibility
					// e.g. "on" must remain quoted, on must remain unquoted
					return nil
				}
				// style does not have semantic meaning
				n.Style = 0
				return nil
			}}},
		Outputs: []kio.Writer{kio.ByteWriter{Writer: buff}},
	}.Execute()
	require.NoError(t, err)
	assert.Equal(t, expected, buff.String())

	y = `
apiVersion: v1
kind: Foo
metadata:
  name: 'foo'
spec:
  notBoolean: "true"
  notBoolean2: "on"
  notBoolean3: y is yes
  isBoolean: on
  isBoolean2: true
  isBoolean3: y
  notInt2: 1234 five
  notInt3: one 2345
  notInt: "12345"
  isInt1: 12345
  isInt2: -12345
  isFloat1: 1.1234
  isFloat2: 1.1234
  isString1: hello world
  isString2: "hello world"
  isString3: 'hello world'
`

	// keep the style on values that parse as non-string types
	expected = `apiVersion: 'v1'
kind: 'Foo'
metadata:
  name: 'foo'
spec:
  isBoolean: on
  isBoolean2: true
  isBoolean3: y
  isFloat1: 1.1234
  isFloat2: 1.1234
  isInt1: 12345
  isInt2: -12345
  isString1: 'hello world'
  isString2: 'hello world'
  isString3: 'hello world'
  notBoolean: "true"
  notBoolean2: "on"
  notBoolean3: 'y is yes'
  notInt: "12345"
  notInt2: '1234 five'
  notInt3: 'one 2345'
`

	buff = &bytes.Buffer{}
	err = kio.Pipeline{
		Inputs: []kio.Reader{&kio.ByteReader{Reader: strings.NewReader(y)}},
		Filters: []kio.Filter{FormatFilter{
			UseSchema: true,
			Process: func(n *yaml.Node) error {
				if yaml.IsYaml1_1NonString(n) {
					// don't change these styles, they are important for backwards compatibility
					// e.g. "on" must remain quoted, on must remain unquoted
					return nil
				}
				// style does not have semantic meaning
				n.Style = yaml.SingleQuotedStyle
				return nil
			}}},
		Outputs: []kio.Writer{kio.ByteWriter{Writer: buff}},
	}.Execute()
	require.NoError(t, err)
	assert.Equal(t, expected, buff.String())
}

func TestFormatInput_Style(t *testing.T) {
	y := `
apiVersion: v1
kind: Foo
metadata:
  name: foo
spec:
  notBoolean: "true"
  notBoolean2: "on"
  isBoolean: on
  isBoolean2: true
`

	expected := `apiVersion: v1
kind: Foo
metadata:
  name: foo
spec:
  isBoolean: on
  isBoolean2: true
  notBoolean: "true"
  notBoolean2: "on"
`

	s, err := FormatInput(strings.NewReader(y))
	require.NoError(t, err)
	assert.Equal(t, expected, s.String())
}

// TestFormatInput_configMap verifies a ConfigMap yaml is formatted correctly
func TestFormatInput_configMap(t *testing.T) {
	y := `


# this formatting is intentionally weird

apiVersion: v1
# this is data
data:
  # this is color
  color: purple
  # that was color

  # this is textmode
  textmode: "true"
  # this is how
  how: fairlyNice



kind: ConfigMap


metadata:
  selfLink: /api/v1/namespaces/default/configmaps/config-multi-env-files
  namespace: default
  creationTimestamp: 2017-12-27T18:38:34Z
  name: config-multi-env-files
  resourceVersion: "810136"
  uid: 252c4572-eb35-11e7-887b-42010a8002b8  # keep no trailing linefeed`

	expected := `# this formatting is intentionally weird

apiVersion: v1
kind: ConfigMap
metadata:
  name: config-multi-env-files
  namespace: default
  creationTimestamp: 2017-12-27T18:38:34Z
  resourceVersion: "810136"
  selfLink: /api/v1/namespaces/default/configmaps/config-multi-env-files
  uid: 252c4572-eb35-11e7-887b-42010a8002b8 # keep no trailing linefeed
# this is data
data:
  # this is color
  color: purple
  # that was color

  # this is how
  how: fairlyNice
  # this is textmode
  textmode: "true"
`

	s, err := FormatInput(strings.NewReader(y))
	require.NoError(t, err)
	assert.Equal(t, expected, s.String())
}

// TestFormatInput_deployment verifies a Deployment yaml is formatted correctly
func TestFormatInput_deployment(t *testing.T) {
	y := `
apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  labels:
    app: nginx
spec:
  selector:
    matchLabels:
      app: nginx
  replicas: 3
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      # this is a container
      - ports:
        # this is a port
        - containerPort: 80
        name: b-nginx
        image: nginx:1.7.9
      # this is another container
      - name: a-nginx
        image: nginx:1.7.9
        ports:
        - containerPort: 80
`
	expected := `apiVersion: apps/v1
kind: Deployment
metadata:
  name: nginx-deployment
  labels:
    app: nginx
spec:
  replicas: 3
  selector:
    matchLabels:
      app: nginx
  template:
    metadata:
      labels:
        app: nginx
    spec:
      containers:
      # this is another container
      - name: a-nginx
        image: nginx:1.7.9
        ports:
        - containerPort: 80
      # this is a container
      - name: b-nginx
        image: nginx:1.7.9
        ports:
        # this is a port
        - containerPort: 80
`
	s, err := FormatInput(strings.NewReader(y))
	require.NoError(t, err)
	assert.Equal(t, expected, s.String())
}

// TestFormatInput_service verifies a Service yaml is formatted correctly
func TestFormatInput_service(t *testing.T) {
	y := `
apiVersion: v1
kind: Service
metadata:
  name: my-service
spec:
  selector:
    app: MyApp
  ports:
    - protocol: TCP
      port: 80
      targetPort: 9376
`
	expected := `apiVersion: v1
kind: Service
metadata:
  name: my-service
spec:
  selector:
    app: MyApp
  ports:
  - protocol: TCP
    port: 80
    targetPort: 9376
`
	s, err := FormatInput(strings.NewReader(y))
	require.NoError(t, err)
	assert.Equal(t, expected, s.String())
}

// TestFormatInput_service verifies a Service yaml is formatted correctly
func TestFormatInput_validatingWebhookConfiguration(t *testing.T) {
	y := `
apiVersion: admissionregistration.k8s.io/v1
kind: ValidatingWebhookConfiguration
metadata:
  name: <name of this configuration object>
webhooks:
- name: <webhook name, e.g., pod-policy.example.io>
  rules:
  - apiGroups:
    - ""
    apiVersions:
    - v1
    operations:
      - UPDATE # this list is indented by 2
      - CREATE
      - CONNECT
    resources:
    - pods # this list is not indented by 2
    scope: "Namespaced"
  clientConfig:
    service:
      namespace: <namespace of the front-end service>
      name: <name of the front-end service>
    caBundle: <pem encoded ca cert that signs the server cert used by the webhook>
  admissionReviewVersions:
  - v1beta1
  timeoutSeconds: 1
`
	expected := `apiVersion: admissionregistration.k8s.io/v1
kind: ValidatingWebhookConfiguration
metadata:
  name: <name of this configuration object>
webhooks:
- name: <webhook name, e.g., pod-policy.example.io>
  admissionReviewVersions:
  - v1beta1
  clientConfig:
    service:
      name: <name of the front-end service>
      namespace: <namespace of the front-end service>
    caBundle: <pem encoded ca cert that signs the server cert used by the webhook>
  rules:
  - resources:
    - pods # this list is not indented by 2
    apiGroups:
    - ""
    apiVersions:
    - v1
    operations:
    - CONNECT
    - CREATE
    - UPDATE # this list is indented by 2
    scope: "Namespaced"
  timeoutSeconds: 1
`
	s, err := FormatInput(strings.NewReader(y))
	require.NoError(t, err)
	assert.Equal(t, expected, s.String())
}

// TestFormatInput_unKnownType verifies an unknown type yaml is formatted correctly
func TestFormatInput_unKnownType(t *testing.T) {
	y := `
spec:
  template:
    spec:
      # these shouldn't be sorted because the type isn't whitelisted
      containers:
      - name: b
      - name: a
  replicas: 1
status:
  conditions:
  - 3
  - 1
  - 2
other:
  b: a1
  a: b1
apiVersion: example.com/v1beta1
kind: MyType
`

	expected := `apiVersion: example.com/v1beta1
kind: MyType
spec:
  replicas: 1
  template:
    spec:
      # these shouldn't be sorted because the type isn't whitelisted
      containers:
      - name: b
      - name: a
status:
  conditions:
  - 3
  - 1
  - 2
other:
  a: b1
  b: a1
`
	s, err := FormatInput(strings.NewReader(y))
	require.NoError(t, err)
	assert.Equal(t, expected, s.String())
}

// TestFormatInput_deployment verifies a Deployment yaml is formatted correctly
func TestFormatInput_resources(t *testing.T) {
	input := &bytes.Buffer{}
	_, err := io.Copy(input, bytes.NewReader(testyaml.UnformattedYaml1))
	require.NoError(t, err)
	_, err = io.Copy(input, strings.NewReader("---\n"))
	require.NoError(t, err)
	_, err = io.Copy(input, bytes.NewReader(testyaml.UnformattedYaml2))
	require.NoError(t, err)
	_, err = io.Copy(input, strings.NewReader("---\n"))
	require.NoError(t, err)
	_, err = io.Copy(input, bytes.NewReader(testyaml.UnformattedYaml3))
	require.NoError(t, err)

	expectedOutput := &bytes.Buffer{}
	_, err = io.Copy(expectedOutput, bytes.NewReader(testyaml.FormattedYaml1))
	require.NoError(t, err)
	_, err = io.Copy(expectedOutput, strings.NewReader("---\n"))
	require.NoError(t, err)
	_, err = io.Copy(expectedOutput, bytes.NewReader(testyaml.FormattedYaml2))
	require.NoError(t, err)
	_, err = io.Copy(expectedOutput, strings.NewReader("---\n"))
	require.NoError(t, err)
	_, err = io.Copy(expectedOutput, bytes.NewReader(testyaml.FormattedYaml3))
	require.NoError(t, err)

	s, err := FormatInput(input)
	require.NoError(t, err)
	assert.Equal(t, expectedOutput.String(), s.String())
}

func TestFormatInput_failMissingKind(t *testing.T) {
	y := `
spec:
  template:
    spec:
      containers:
      - b
      - a
  replicas: 1
status:
  conditions:
  - 3
  - 1
  - 2
other:
  b: a1
  a: b1
apiVersion: example.com/v1beta1
`

	b, err := FormatInput(strings.NewReader(y))
	require.NoError(t, err)
	assert.Equal(t, strings.TrimLeft(y, "\n"), b.String())
}

func TestFormatInput_failMissingApiVersion(t *testing.T) {
	y := `
spec:
  template:
    spec:
      containers:
      - a
      - b
  replicas: 1
status:
  conditions:
  - 3
  - 1
  - 2
other:
  b: a1
  a: b1
kind: MyKind
`

	b, err := FormatInput(strings.NewReader(y))
	require.NoError(t, err)
	assert.Equal(t, strings.TrimLeft(y, "\n"), b.String())
}

func TestFormatInput_failUnmarshal(t *testing.T) {
	y := `
spec:
  template:
    spec:
      containers:
      - a
      - b
  replicas: 1
status:
  conditions:
  - 3
  - 1
  - 2
other:
	b: a1
	a: b1
kind: MyKind
apiVersion: example.com/v1beta1
`

	_, err := FormatInput(strings.NewReader(y))
	assert.EqualError(t, err, "MalformedYAMLError: yaml: line 15: found character that cannot start any token")
}

// TestFormatFileOrDirectory_yamlExtFile verifies that FormatFileOrDirectory will format a file
// with a .yaml extension.
func TestFormatFileOrDirectory_yamlExtFile(t *testing.T) {
	// write the unformatted file
	f, err := os.CreateTemp("", "yamlfmt*.yaml")
	if !assert.NoError(t, err) {
		return
	}
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(), testyaml.UnformattedYaml1, 0600)
	if !assert.NoError(t, err) {
		return
	}

	// format the file
	err = FormatFileOrDirectory(f.Name())
	if !assert.NoError(t, err) {
		return
	}

	// check the result is formatted
	b, err := os.ReadFile(f.Name())
	if !assert.NoError(t, err) {
		return
	}
	assert.Equal(t, string(testyaml.FormattedYaml1), string(b))
}

func TestFormatFileOrDirectory_multipleYamlEntries(t *testing.T) {
	// write the unformatted file
	f, err := os.CreateTemp("", "yamlfmt*.yaml")
	require.NoError(t, err)
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(),
		[]byte(string(testyaml.UnformattedYaml1)+"---\n"+string(testyaml.UnformattedYaml2)), 0600)
	require.NoError(t, err)

	// format the file
	err = FormatFileOrDirectory(f.Name())
	require.NoError(t, err)

	// check the result is formatted
	b, err := os.ReadFile(f.Name())
	require.NoError(t, err)
	assert.Equal(t, string(testyaml.FormattedYaml1)+"---\n"+string(testyaml.FormattedYaml2), string(b))
}

// TestFormatFileOrDirectory_ymlExtFile verifies that FormatFileOrDirectory will format a file
// with a .yml extension.
func TestFormatFileOrDirectory_ymlExtFile(t *testing.T) {
	// write the unformatted file
	f, err := os.CreateTemp("", "yamlfmt*.yml")
	require.NoError(t, err)
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(), testyaml.UnformattedYaml1, 0600)
	require.NoError(t, err)

	// format the file
	err = FormatFileOrDirectory(f.Name())
	require.NoError(t, err)

	// check the result is formatted
	b, err := os.ReadFile(f.Name())
	require.NoError(t, err)
	assert.Equal(t, string(testyaml.FormattedYaml1), string(b))
}

// TestFormatFileOrDirectory_YamlExtFileWithJson verifies that the JSON compatible flow style
// YAML content is formatted as such.
func TestFormatFileOrDirectory_YamlExtFileWithJson(t *testing.T) {
	// write the unformatted JSON file contents
	f, err := os.CreateTemp("", "yamlfmt*.yaml")
	require.NoError(t, err)
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(), testyaml.UnformattedJSON1, 0600)
	require.NoError(t, err)

	// format the file
	err = FormatFileOrDirectory(f.Name())
	require.NoError(t, err)

	// check the result is formatted as yaml
	b, err := os.ReadFile(f.Name())
	require.NoError(t, err)
	assert.Equal(t, string(testyaml.FormattedFlowYAML1), string(b))
}

// TestFormatFileOrDirectory_JsonExtFileWithNotModified verifies that a file with .json extensions
// and JSON contents won't be modified.
func TestFormatFileOrDirectory_JsonExtFileWithNotModified(t *testing.T) {
	// write the unformatted JSON file contents
	f, err := os.CreateTemp("", "yamlfmt*.json")
	require.NoError(t, err)
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(), testyaml.UnformattedJSON1, 0600)
	require.NoError(t, err)

	// format the file
	err = FormatFileOrDirectory(f.Name())
	require.NoError(t, err)

	// check the result is formatted as yaml
	b, err := os.ReadFile(f.Name())
	require.NoError(t, err)
	assert.Equal(t, string(testyaml.UnformattedJSON1), string(b))
}

// TestFormatFileOrDirectory_partialKubernetesYamlFile verifies that if a yaml file contains both
// Kubernetes and non-Kubernetes documents, it will only format the Kubernetes documents
func TestFormatFileOrDirectory_partialKubernetesYamlFile(t *testing.T) {
	// write the unformatted file
	f, err := os.CreateTemp("", "yamlfmt*.yaml")
	require.NoError(t, err)
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(), []byte(string(testyaml.UnformattedYaml1)+`---
status:
  conditions:
  - 3
  - 1
  - 2
spec: a
---
`+string(testyaml.UnformattedYaml2)), 0600)
	require.NoError(t, err)

	// format the file
	err = FormatFileOrDirectory(f.Name())
	require.NoError(t, err)

	// check the result is  NOT formatted
	b, err := os.ReadFile(f.Name())
	require.NoError(t, err)
	assert.Equal(t, string(testyaml.FormattedYaml1)+`---
status:
  conditions:
  - 3
  - 1
  - 2
spec: a
---
`+string(testyaml.FormattedYaml2), string(b))
}

// TestFormatFileOrDirectory_nonKubernetesYamlFile verifies that if a yaml file does not contain
// kubernetes
func TestFormatFileOrDirectory_skipNonKubernetesYamlFile(t *testing.T) {
	// write the unformatted JSON file contents
	f, err := os.CreateTemp("", "yamlfmt*.yaml")
	require.NoError(t, err)
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(), []byte(`
status:
  conditions:
  - 3
  - 1
  - 2
spec: a
`), 0600)
	require.NoError(t, err)

	// format the file
	err = FormatFileOrDirectory(f.Name())
	require.NoError(t, err)

	// check the result is formatted as yaml
	b, err := os.ReadFile(f.Name())
	require.NoError(t, err)
	assert.Equal(t, `status:
  conditions:
  - 3
  - 1
  - 2
spec: a
`, string(b))
}

// TestFormatFileOrDirectory_jsonFile should not fmt the file even though it contains yaml.
func TestFormatFileOrDirectory_skipJsonExtFile(t *testing.T) {
	f, err := os.CreateTemp("", "yamlfmt*.json")
	require.NoError(t, err)
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(), testyaml.UnformattedYaml1, 0600)
	require.NoError(t, err)

	err = FormatFileOrDirectory(f.Name())
	require.NoError(t, err)

	b, err := os.ReadFile(f.Name())
	require.NoError(t, err)

	assert.Equal(t, string(testyaml.UnformattedYaml1), string(b))
}

// TestFormatFileOrDirectory_directory verifies that yaml files will be formatted,
// and other files will be ignored
func TestFormatFileOrDirectory_directory(t *testing.T) {
	d := t.TempDir()

	err := os.Mkdir(filepath.Join(d, "config"), 0700)
	require.NoError(t, err)

	err = os.WriteFile(filepath.Join(d, "c1.yaml"), testyaml.UnformattedYaml1, 0600)
	require.NoError(t, err)

	err = os.WriteFile(filepath.Join(d, "config", "c2.yaml"), testyaml.UnformattedYaml2, 0600)
	require.NoError(t, err)

	err = os.WriteFile(filepath.Join(d, "README.md"), []byte(`# Markdown`), 0600)
	require.NoError(t, err)

	err = FormatFileOrDirectory(d)
	require.NoError(t, err)

	b, err := os.ReadFile(filepath.Join(d, "c1.yaml"))
	require.NoError(t, err)
	assert.Equal(t, string(testyaml.FormattedYaml1), string(b))

	b, err = os.ReadFile(filepath.Join(d, "config", "c2.yaml"))
	require.NoError(t, err)
	assert.Equal(t, string(testyaml.FormattedYaml2), string(b))

	b, err = os.ReadFile(filepath.Join(d, "README.md"))
	require.NoError(t, err)
	assert.Equal(t, `# Markdown`, string(b))

	// verify no additional files were created
	files := []string{
		".", "c1.yaml", "README.md", "config", filepath.Join("config", "c2.yaml")}
	err = filepath.Walk(d, func(path string, info os.FileInfo, err error) error {
		require.NoError(t, err)
		path, err = filepath.Rel(d, path)
		require.NoError(t, err)
		assert.Contains(t, files, path)
		return nil
	})
	require.NoError(t, err)
}

// TestFormatFileOrDirectory_trimWhiteSpace verifies that trailling and leading whitespace is
// trimmed
func TestFormatFileOrDirectory_trimWhiteSpace(t *testing.T) {
	f, err := os.CreateTemp("", "yamlfmt*.yaml")
	require.NoError(t, err)
	defer os.Remove(f.Name())
	err = os.WriteFile(f.Name(), []byte("\n\n"+string(testyaml.UnformattedYaml1)+"\n\n"), 0600)
	require.NoError(t, err)

	err = FormatFileOrDirectory(f.Name())
	require.NoError(t, err)

	b, err := os.ReadFile(f.Name())
	require.NoError(t, err)

	assert.Equal(t, string(testyaml.FormattedYaml1), string(b))
}

func TestFormatFileOrDirectory_FmtAnnotation(t *testing.T) {
	testCases := []struct {
		name           string
		input          []byte
		expectedOutput []byte
		expectError    bool
	}{
		{
			name:           "no formatting annotation",
			input:          testyaml.UnformattedYaml1,
			expectedOutput: testyaml.FormattedYaml1,
		},
		{
			name: "formatting strategy none",
			input: []byte(`
spec: a
status:
  conditions:
  - 3
  - 1
  - 2
apiVersion: example.com/v1beta1
kind: MyType
metadata:
  annotations:
    config.kubernetes.io/formatting: none
`),
			expectedOutput: []byte(`
spec: a
status:
  conditions:
  - 3
  - 1
  - 2
apiVersion: example.com/v1beta1
kind: MyType
metadata:
  annotations:
    config.kubernetes.io/formatting: none
`),
		},
		{
			name: "formatting strategy standard",
			input: []byte(`
spec: a
status:
  conditions:
  - 3
  - 1
  - 2
apiVersion: example.com/v1beta1
kind: MyType
metadata:
  annotations:
    config.kubernetes.io/formatting: standard
`),
			expectedOutput: []byte(`
apiVersion: example.com/v1beta1
kind: MyType
metadata:
  annotations:
    config.kubernetes.io/formatting: standard
spec: a
status:
  conditions:
  - 3
  - 1
  - 2
`),
		},
		{
			name: "unknown formatting strategy",
			input: []byte(`
spec: a
status:
  conditions:
  - 3
  - 1
  - 2
apiVersion: example.com/v1beta1
kind: MyType
metadata:
  annotations:
    config.kubernetes.io/formatting: unknown
`),
			expectError: true,
		},
	}

	for i := range testCases {
		test := testCases[i]
		t.Run(test.name, func(t *testing.T) {
			f, err := os.CreateTemp("", "yamlfmt*.yaml")
			require.NoError(t, err)
			defer os.Remove(f.Name())

			err = os.WriteFile(f.Name(), test.input, 0600)
			require.NoError(t, err)

			err = FormatFileOrDirectory(f.Name())
			if test.expectError {
				require.Error(t, err)
				return
			}
			require.NoError(t, err)

			b, err := os.ReadFile(f.Name())
			require.NoError(t, err)

			assert.Equal(t, strings.TrimSpace(string(test.expectedOutput)),
				strings.TrimSpace(string(b)))
		})
	}
}

func TestFormatInput_NullCases(t *testing.T) {
	y := `
apiVersion: v1
kind: Service
metadata:
  name: nginx
  labels:
    app: null
spec:
  selector:
    app: nginx
  ports:
  - name: http
    port: 80
    targetPort: ~
    nodePort: null
  allocateLoadBalancerNodePorts: null
`

	// keep the style on values that parse as non-string types
	expected := `apiVersion: v1
kind: Service
metadata:
  name: nginx
  labels:
    app: null
spec:
  selector:
    app: nginx
  ports:
  - name: http
    port: 80
    targetPort: ~
    nodePort: null
  allocateLoadBalancerNodePorts: null
`

	buff := &bytes.Buffer{}
	err := kio.Pipeline{
		Inputs: []kio.Reader{&kio.ByteReader{Reader: strings.NewReader(y)}},
		Filters: []kio.Filter{FormatFilter{
			UseSchema: true,
		}},
		Outputs: []kio.Writer{kio.ByteWriter{Writer: buff}},
	}.Execute()
	require.NoError(t, err)
	assert.Equal(t, expected, buff.String())
}
