test: add more test cases (#444)

Author: JounQin

packages/autocorrect/test/__snapshots__/fixtures.spec.ts.snap

@@ -1,6 +1,6 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
-exports[`parser and printer > should format all fixtures > test.css 1`] = `
+exports[`parser and printer > should format test.css fixture 1`] = `
 "/* 
   这是多行 CSS 第 1 行
   这是第 2 行
@@ -14,7 +14,7 @@ exports[`parser and printer > should format all fixtures > test.css 1`] = `
 "
 `;
 
-exports[`parser and printer > should format all fixtures > test.go 1`] = `
+exports[`parser and printer > should format test.go fixture 1`] = `
 "// WithContext 创建基于 ctx 的 db
 // 第 2 行注释
 func (d *Dao) WithContext(ctx context.Context) (db *gorm.DB) {
@@ -34,7 +34,7 @@ func (d *Dao) WithContext(ctx context.Context) (db *gorm.DB) {
 "
 `;
 
-exports[`parser and printer > should format all fixtures > test.html 1`] = `
+exports[`parser and printer > should format test.html fixture 1`] = `
 "<h1>编译 rust 为 WebAssembly</h1>
 <div></div>
 <p class="summary">如果你写了一些 RUST 代码，你可以把它编译成 webAssembly！这份教程将带你编译 Rust 项目为 wasm 并在一个现存的 web 应用中使用它。</p>
@@ -49,7 +49,7 @@ exports[`parser and printer > should format all fixtures > test.html 1`] = `
 "
 `;
 
-exports[`parser and printer > should format all fixtures > test.md 1`] = `
+exports[`parser and printer > should format test.md fixture 1`] = `
 "# 这是 Heading 1 大标题
 
 **加粗**
@@ -102,7 +102,7 @@ let a = "你好 hello";
 "
 `;
 
-exports[`parser and printer > should format all fixtures > test.py 1`] = `
+exports[`parser and printer > should format test.py fixture 1`] = `
 "'''
 这是多行 1 注释
 这是多行 2 注释
@@ -125,7 +125,7 @@ def hello(a):
   "
 `;
 
-exports[`parser and printer > should format all fixtures > test.rb 1`] = `
+exports[`parser and printer > should format test.rb fixture 1`] = `
 "# 第 1 行注释
 # 第 2 行注释
 def hello(a, b: "第 1 个参数")
@@ -139,7 +139,7 @@ end
 "
 `;
 
-exports[`parser and printer > should format all fixtures > test0.js 1`] = `
+exports[`parser and printer > should format test0.js fixture 1`] = `
 "/* eslint-disable camelcase, no-unused-vars, sonarjs/no-dead-store, sonarjs/no-unused-vars */
 /**
  * Hello 你好

packages/autocorrect/test/fixtures.spec.ts

@@ -12,23 +12,24 @@ const _dirname =
     : __dirname
 
 describe('parser and printer', () => {
-  it('should format all fixtures', async () => {
-    const fixtures = path.resolve(_dirname, 'fixtures')
-    for (const relativeFilepath of fs.readdirSync(fixtures)) {
-      const filepath = path.resolve(fixtures, relativeFilepath)
-      const input = fs.readFileSync(filepath, 'utf8')
+  const fixtures = path.resolve(_dirname, 'fixtures')
 
+  for (const relativeFilepath of fs.readdirSync(fixtures)) {
+    const filepath = path.resolve(fixtures, relativeFilepath)
+    const input = fs.readFileSync(filepath, 'utf8')
+
+    it(`should format ${relativeFilepath} fixture`, async () => {
       try {
         const output = await format(input, {
           filepath,
           parser: 'autocorrect',
           plugins: [AutocorrectPlugin],
         })
 
-        expect(output).toMatchSnapshot(relativeFilepath)
+        expect(output).toMatchSnapshot()
       } catch (err: unknown) {
-        expect((err as Error).message).toMatchSnapshot(relativeFilepath)
+        expect((err as Error).message).toMatchSnapshot()
       }
-    }
-  })
+    })
+  }
 })

packages/sh/test/__snapshots__/fixtures.spec.ts.snap

@@ -1,6 +1,6 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
-exports[`parser and printer > should format .dockerignore fixtures > .dockerignore 1`] = `
+exports[`parser and printer > should format .dockerignore fixture correctly 1`] = `
 "*.less.json
 *.log
 *.tsbuildinfo
@@ -11,12 +11,12 @@ node_modules
 "
 `;
 
-exports[`parser and printer > should format .nvmrc fixtures > .nvmrc 1`] = `
+exports[`parser and printer > should format .nvmrc fixture correctly 1`] = `
 "v1.2.3
 "
 `;
 
-exports[`parser and printer > should format .properties fixtures > .properties 1`] = `
+exports[`parser and printer > should format .properties fixture correctly 1`] = `
 "# THIS IS AN AUTOGENERATED FILE. DO NOT EDIT THIS FILE DIRECTLY.
 # yarn lockfile v1
 
@@ -26,7 +26,7 @@ username jounqin
 "
 `;
 
-exports[`parser and printer > should format 133.sh fixtures > 133.sh 1`] = `
+exports[`parser and printer > should format 133.sh fixture correctly 1`] = `
 "#!/usr/bin/env bash
 
 set -euo pipefail
@@ -218,7 +218,7 @@ main "$@"
 "
 `;
 
-exports[`parser and printer > should format 162.sh fixtures > 162.sh 1`] = `
+exports[`parser and printer > should format 162.sh fixture correctly 1`] = `
 "#!/usr/bin/env bash
 
 { # this ensures the entire script is downloaded #
@@ -700,7 +700,7 @@ exports[`parser and printer > should format 162.sh fixtures > 162.sh 1`] = `
 "
 `;
 
-exports[`parser and printer > should format 182.sh fixtures > 182.sh 1`] = `
+exports[`parser and printer > should format 182.sh fixture correctly 1`] = `
 "#!/usr/bin/env bash
 set -euo pipefail
 
@@ -756,7 +756,7 @@ fi
 "
 `;
 
-exports[`parser and printer > should format 191.sh fixtures > 191.sh 1`] = `
+exports[`parser and printer > should format 191.sh fixture correctly 1`] = `
 "#!/bin/sh
 
 set -eu
@@ -836,20 +836,20 @@ esac
 "
 `;
 
-exports[`parser and printer > should format 278.Dockerfile fixtures > 278.Dockerfile 1`] = `
+exports[`parser and printer > should format 278.Dockerfile fixture correctly 1`] = `
 "FROM ubuntu
 
 RUN (cd out && ls)
 "
 `;
 
-exports[`parser and printer > should format 292.Dockerfile fixtures > 292.Dockerfile 1`] = `
+exports[`parser and printer > should format 292.Dockerfile fixture correctly 1`] = `
 "FROM node:20.9.0-alpine
 RUN if [[ x$LATEST_NPM = xtrue ]]; then yarn global add npm@latest; fi
 "
 `;
 
-exports[`parser and printer > should format 376.Dockerfile fixtures > 376.Dockerfile 1`] = `
+exports[`parser and printer > should format 376.Dockerfile fixture correctly 1`] = `
 "FROM debian:12.6-slim
 
 RUN set -eux; for x in {1..3}; do echo 'foo'; echo 'bar'; echo "$x"; done
@@ -865,12 +865,12 @@ EOF
 "
 `;
 
-exports[`parser and printer > should format 384.Dockerfile fixtures > 384.Dockerfile 1`] = `
+exports[`parser and printer > should format 384.Dockerfile fixture correctly 1`] = `
 "LABEL org.opencontainers.image.authors="Jean Luc Picard <[email protected]>"
 "
 `;
 
-exports[`parser and printer > should format 398.Dockerfile fixtures > 398.Dockerfile 1`] = `
+exports[`parser and printer > should format 398.Dockerfile fixture correctly 1`] = `
 "ENV a=1 \\
  b=2 \\
  # comment
@@ -881,7 +881,7 @@ exports[`parser and printer > should format 398.Dockerfile fixtures > 398.Docker
 "
 `;
 
-exports[`parser and printer > should format 441.Dockerfile fixtures > 441.Dockerfile 1`] = `
+exports[`parser and printer > should format 441.Dockerfile fixture correctly with options: {"indent":4} 1`] = `
 "RUN \\
  # install dependencies
  NODE_ENV=production npm install-clean \\
@@ -890,7 +890,9 @@ exports[`parser and printer > should format 441.Dockerfile fixtures > 441.Docker
 "
 `;
 
-exports[`parser and printer > should format Dockerfile fixtures > Dockerfile 1`] = `
+exports[`parser and printer > should format 445.sh fixture correctly 1`] = `"reached EOF without closing quote '"`;
+
+exports[`parser and printer > should format Dockerfile fixture correctly 1`] = `
 "FROM node:lts-alpine as builder
 
 # 安装与编译代码
@@ -907,7 +909,7 @@ EXPOSE 2015
 "
 `;
 
-exports[`parser and printer > should format hosts fixtures > hosts 1`] = `
+exports[`parser and printer > should format hosts fixture correctly 1`] = `
 "##
 # Host Database
 #
@@ -924,21 +926,21 @@ exports[`parser and printer > should format hosts fixtures > hosts 1`] = `
 "
 `;
 
-exports[`parser and printer > should format jvm.options fixtures > jvm.options 1`] = `
+exports[`parser and printer > should format jvm.options fixture correctly 1`] = `
 "# this is a comment
 -mx2048m -XX:MaxPermSize=2048m -Drebel.spring_plugin=true -Drebel.hibernate_plugin=true
 
 idea.cycle.buffer.size = 1024
 "
 `;
 
-exports[`parser and printer > should format no-ext fixtures > no-ext 1`] = `
+exports[`parser and printer > should format no-ext fixture correctly 1`] = `
 "#!/usr/bin/env sh
 echo 'foo'
 "
 `;
 
-exports[`parser and printer > should format shell.sh fixtures > shell.sh 1`] = `
+exports[`parser and printer > should format shell.sh fixture correctly 1`] = `
 "#!/bin/bash
 NAME= "ufc-web-client"
 

packages/sh/test/fixtures.spec.ts

@@ -5,27 +5,41 @@ import { format } from 'prettier'
 
 import * as sh from 'prettier-plugin-sh'
 
-const _dirname = import.meta.dirname
+const PARSER_OPTIONS: Partial<
+  Record<string, Omit<sh.ShPrintOptions, 'originalText'>>
+> = {
+  441: {
+    indent: 4,
+  },
+}
 
 describe('parser and printer', () => {
-  const fixtures = path.resolve(_dirname, 'fixtures')
+  const fixtures = path.resolve(import.meta.dirname, 'fixtures')
 
   for (const relativeFilepath of fs.readdirSync(fixtures)) {
     const filepath = path.resolve(fixtures, relativeFilepath)
     const input = fs.readFileSync(filepath, 'utf8')
 
-    it(`should format ${relativeFilepath} fixtures`, async () => {
+    const caseName = relativeFilepath.slice(
+      0,
+      relativeFilepath.lastIndexOf('.'),
+    )
+
+    const overrideOptions = PARSER_OPTIONS[caseName]
+
+    it(`should format ${relativeFilepath} fixture correctly${
+      overrideOptions ? ' with options: ' + JSON.stringify(overrideOptions) : ''
+    }`, async () => {
       try {
         const output = await format(input, {
           filepath,
           plugins: [sh],
+          ...overrideOptions,
         })
 
-        expect(output).toMatchSnapshot(relativeFilepath)
+        expect(output).toMatchSnapshot()
       } catch (err: unknown) {
-        expect((err as Error).message.split('\n').at(0)).toMatchSnapshot(
-          relativeFilepath,
-        )
+        expect((err as Error).message.split('\n').at(0)).toMatchSnapshot()
       }
     })
   }

packages/sh/test/fixtures/445.sh

@@ -0,0 +1,2 @@
+curl --location --request PUT 'https://url.com/bla?q=\'hi\'' \
+--header 'Content-Type: application/json'

packages/sql/test/__snapshots__/fixtures-eol.spec.ts.snap

@@ -1,9 +1,29 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
-exports[`parser and printer > printer should handle eol correctly > 556.sql 1`] = `""CREATE TABLE \\"test\\" (\\"id\\" uuid NOT NULL)\\r\\nWITH\\r\\n  (oids = false);\\r\\n""`;
+exports[`parser and printer > printer should handle 556.sql eol correctly with options: {"language":"mysql","endOfLine":"crlf"} 1`] = `
+"CREATE TABLE "test" ("id" uuid NOT NULL)
+WITH
+  (oids = false);
+"
+`;
 
-exports[`parser and printer > printer should handle eol correctly > 557.sql 1`] = `""CREATE TABLE \\"test\\" (\\"id\\" uuid NOT NULL)\\rWITH\\r  (oids = false);\\r""`;
+exports[`parser and printer > printer should handle 557.sql eol correctly with options: {"language":"mysql","endOfLine":"cr"} 1`] = `
+"CREATE TABLE "test" ("id" uuid NOT NULL)
+WITH
+  (oids = false);
+"
+`;
 
-exports[`parser and printer > printer should handle eol correctly > 558.sql 1`] = `""CREATE TABLE \\"test\\" (\\"id\\" uuid NOT NULL)\\nWITH\\n  (oids = false);\\n""`;
+exports[`parser and printer > printer should handle 558.sql eol correctly with options: {"language":"mysql","endOfLine":"lf"} 1`] = `
+"CREATE TABLE "test" ("id" uuid NOT NULL)
+WITH
+  (oids = false);
+"
+`;
 
-exports[`parser and printer > printer should handle eol correctly > 559.sql 1`] = `""CREATE TABLE \\"test\\" (\\"id\\" uuid NOT NULL)\\r\\nWITH\\r\\n  (oids = false);\\r\\n""`;
+exports[`parser and printer > printer should handle 559.sql eol correctly with options: {"language":"mysql","endOfLine":"auto"} 1`] = `
+"CREATE TABLE "test" ("id" uuid NOT NULL)
+WITH
+  (oids = false);
+"
+`;

packages/sql/test/__snapshots__/fixtures.spec.ts.snap

@@ -1,12 +1,11 @@
 import fs from 'node:fs'
 import path from 'node:path'
-import { fileURLToPath } from 'node:url'
 
 import { format } from 'prettier'
 
 import SqlPlugin, { type SqlFormatOptions } from 'prettier-plugin-sql'
 
-const PARSER_OPTIONS: Record<string, SqlFormatOptions> = {
+const PARSER_OPTIONS: Partial<Record<string, SqlFormatOptions>> = {
   556: {
     language: 'mysql',
     endOfLine: 'crlf',
@@ -25,31 +24,35 @@ const PARSER_OPTIONS: Record<string, SqlFormatOptions> = {
   },
 }
 
-const _dirname =
-  typeof __dirname === 'undefined'
-    ? path.dirname(fileURLToPath(import.meta.url))
-    : __dirname
-
 describe('parser and printer', () => {
-  it('printer should handle eol correctly', async () => {
-    const fixtures = path.resolve(_dirname, 'fixtures-eol')
-    for (const filepath of fs.readdirSync(fixtures)) {
-      const input = fs.readFileSync(path.resolve(fixtures, filepath)).toString()
+  const fixtures = path.resolve(import.meta.dirname, 'fixtures-eol')
+
+  for (const relativeFilepath of fs.readdirSync(fixtures)) {
+    const filepath = path.resolve(fixtures, relativeFilepath)
+    const input = fs.readFileSync(filepath, 'utf8')
+
+    const caseName = relativeFilepath.slice(
+      0,
+      relativeFilepath.lastIndexOf('.'),
+    )
 
-      const caseName = filepath.slice(0, filepath.lastIndexOf('.'))
+    const overrideOptions = PARSER_OPTIONS[caseName]
 
+    it(`printer should handle ${relativeFilepath} eol correctly${
+      overrideOptions ? ' with options: ' + JSON.stringify(overrideOptions) : ''
+    }`, async () => {
       try {
         const output = await format(input, {
           filepath,
           parser: 'sql',
           plugins: [SqlPlugin],
-          ...PARSER_OPTIONS[caseName],
+          ...overrideOptions,
         })
 
-        expect(JSON.stringify(output)).toMatchSnapshot(filepath)
+        expect(output).toMatchSnapshot()
       } catch (error) {
-        expect(error).toMatchSnapshot(filepath)
+        expect(error).toMatchSnapshot()
       }
-    }
-  })
+    })
+  }
 })