From 9a39a98b57de4183e054ab170592c85c97fac183 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bartek=20Iwa=C5=84czuk?= <biwanczuk@gmail.com>
Date: Mon, 4 Nov 2024 17:57:29 +0000
Subject: fix(fmt): ignore file directive for YAML files (#26717)

Closes https://github.com/denoland/deno/issues/26712

Support `# deno-fmt-ignore-file` directive for YAML files.

Also added tests for single line ignores.
---
 tests/specs/fmt/yaml/__test__.jsonc         | 24 ++++++++++++++++++++++++
 tests/specs/fmt/yaml/ignore_file.yaml       |  2 ++
 tests/specs/fmt/yaml/ignore_file2.yaml      |  2 ++
 tests/specs/fmt/yaml/ignore_file3.yaml      |  5 +++++
 tests/specs/fmt/yaml/ignore_file4.yaml      |  2 ++
 tests/specs/fmt/yaml/ignore_line.yml        |  2 ++
 tests/specs/fmt/yaml/wrong_file_ignore.out  |  7 +++++++
 tests/specs/fmt/yaml/wrong_file_ignore.yaml |  5 +++++
 8 files changed, 49 insertions(+)
 create mode 100644 tests/specs/fmt/yaml/ignore_file.yaml
 create mode 100644 tests/specs/fmt/yaml/ignore_file2.yaml
 create mode 100644 tests/specs/fmt/yaml/ignore_file3.yaml
 create mode 100644 tests/specs/fmt/yaml/ignore_file4.yaml
 create mode 100644 tests/specs/fmt/yaml/ignore_line.yml
 create mode 100644 tests/specs/fmt/yaml/wrong_file_ignore.out
 create mode 100644 tests/specs/fmt/yaml/wrong_file_ignore.yaml

(limited to 'tests')

diff --git a/tests/specs/fmt/yaml/__test__.jsonc b/tests/specs/fmt/yaml/__test__.jsonc
index 499b4144b..71a2e0682 100644
--- a/tests/specs/fmt/yaml/__test__.jsonc
+++ b/tests/specs/fmt/yaml/__test__.jsonc
@@ -8,6 +8,30 @@
     "well_formatted": {
       "args": "fmt --check well_formatted.yml",
       "output": "Checked 1 file\n"
+    },
+    "ignore_line": {
+      "args": "fmt --check ignore_line.yml",
+      "output": "Checked 1 file\n"
+    },
+    "ignore_file": {
+      "args": "fmt ignore_file.yaml",
+      "output": "Checked 1 file\n"
+    },
+    "ignore_file2": {
+      "args": "fmt ignore_file2.yaml",
+      "output": "Checked 1 file\n"
+    },
+    "ignore_file3": {
+      "args": "fmt ignore_file3.yaml",
+      "output": "Checked 1 file\n"
+    },
+    "ignore_file4": {
+      "args": "fmt ignore_file4.yaml",
+      "output": "Checked 1 file\n"
+    },
+    "wrong_file_ignore": {
+      "args": "fmt wrong_file_ignore.yaml",
+      "output": "wrong_file_ignore.out"
     }
   }
 }
diff --git a/tests/specs/fmt/yaml/ignore_file.yaml b/tests/specs/fmt/yaml/ignore_file.yaml
new file mode 100644
index 000000000..c724c7bde
--- /dev/null
+++ b/tests/specs/fmt/yaml/ignore_file.yaml
@@ -0,0 +1,2 @@
+# deno-fmt-ignore-file
+{{something crazy
\ No newline at end of file
diff --git a/tests/specs/fmt/yaml/ignore_file2.yaml b/tests/specs/fmt/yaml/ignore_file2.yaml
new file mode 100644
index 000000000..a2053533e
--- /dev/null
+++ b/tests/specs/fmt/yaml/ignore_file2.yaml
@@ -0,0 +1,2 @@
+#deno-fmt-ignore-file
+{{something crazy
\ No newline at end of file
diff --git a/tests/specs/fmt/yaml/ignore_file3.yaml b/tests/specs/fmt/yaml/ignore_file3.yaml
new file mode 100644
index 000000000..a3927a7b1
--- /dev/null
+++ b/tests/specs/fmt/yaml/ignore_file3.yaml
@@ -0,0 +1,5 @@
+# Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor
+# incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, 
+# quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. 
+# deno-fmt-ignore-file
+{{something crazy
\ No newline at end of file
diff --git a/tests/specs/fmt/yaml/ignore_file4.yaml b/tests/specs/fmt/yaml/ignore_file4.yaml
new file mode 100644
index 000000000..c32fe9f2b
--- /dev/null
+++ b/tests/specs/fmt/yaml/ignore_file4.yaml
@@ -0,0 +1,2 @@
+# deno-fmt-ignore-file Because this is templated yaml file
+{{something crazy
\ No newline at end of file
diff --git a/tests/specs/fmt/yaml/ignore_line.yml b/tests/specs/fmt/yaml/ignore_line.yml
new file mode 100644
index 000000000..490adcb5f
--- /dev/null
+++ b/tests/specs/fmt/yaml/ignore_line.yml
@@ -0,0 +1,2 @@
+# deno-fmt-ignore
+-     Test
diff --git a/tests/specs/fmt/yaml/wrong_file_ignore.out b/tests/specs/fmt/yaml/wrong_file_ignore.out
new file mode 100644
index 000000000..065dc98db
--- /dev/null
+++ b/tests/specs/fmt/yaml/wrong_file_ignore.out
@@ -0,0 +1,7 @@
+Error formatting: [WILDCARD]wrong_file_ignore.yaml
+   parse error at line 5, column 1
+  |
+5 | {{something crazy
+  | ^
+
+Checked 1 file
diff --git a/tests/specs/fmt/yaml/wrong_file_ignore.yaml b/tests/specs/fmt/yaml/wrong_file_ignore.yaml
new file mode 100644
index 000000000..8019d989d
--- /dev/null
+++ b/tests/specs/fmt/yaml/wrong_file_ignore.yaml
@@ -0,0 +1,5 @@
+# File ignore directive only works if it's in the first cluster
+# of comment, ie. there are no empty lines after the first n-leading lines.
+
+# deno-fmt-ignore-file
+{{something crazy
\ No newline at end of file
-- 
cgit v1.2.3