diff --git a/.gitea/workflows/test.yml b/.gitea/workflows/test.yml new file mode 100644 index 0000000..05891af --- /dev/null +++ b/.gitea/workflows/test.yml @@ -0,0 +1,40 @@ +name: Tests + +on: + push: + branches: + - main + pull_request: + workflow_dispatch: + +jobs: + frontend-test: + runs-on: ubuntu-latest + + steps: + - name: Checkout + uses: actions/checkout@v4 + + - name: Set up Node.js 22 + uses: actions/setup-node@v3 + with: + node-version: '22' + cache: 'npm' + cache-dependency-path: package-lock.json + + - name: Install dependencies + run: | + npm config set fetch-retries 5 + npm config set fetch-retry-mintimeout 10000 + npm config set fetch-retry-maxtimeout 120000 + for i in 1 2 3; do + npm install && break + echo "Attempt $i failed, retrying in 20s..." + sleep 20 + done + + # npm run ci = проверка типов + Vitest. Это НЕ то же самое, что npm run + # build: там дополнительно собирается production-бандл, а для проверки + # PR он не нужен. + - name: Typecheck & unit tests (Vitest) + run: npm run ci diff --git a/README.md b/README.md index 9f82eb8..13921e4 100644 --- a/README.md +++ b/README.md @@ -12,6 +12,7 @@ - Vite 6 - SCSS Modules - [`@tnsu/ui-kit`](http://git.tnsu.kz/timur.gorodetskiy/tnsu-ui-kit) — общие UI-компоненты +- Vitest — тесты бизнес-логики React именно 17-й: `@tnsu/ui-kit` объявляет его в `peerDependencies`. @@ -28,11 +29,25 @@ npm run dev Прочие команды: ```bash -npm run build # проверка типов + production-сборка в dist/ +npm run build # проверка типов + тесты + production-сборка в dist/ npm run typecheck # только проверка типов +npm run test # тесты один раз (то же, что гоняет CI) +npm run test:watch # тесты в режиме наблюдения при разработке npm run preview # просмотр собранной версии ``` +## Тесты и CI + +Покрыты чистые функции бизнес-логики — `validateFeedback` и `getAppealFieldsConfig` +(`*.test.ts` рядом с тестируемым модулем). React-компоненты не тестируются: сначала +нужно решить, куда форма отправляет данные (см. «Что не сделано»), тестировать UI до +этого — тесты, которые придётся сразу переписывать. + +`.gitea/workflows/test.yml` гоняет `npm run ci` (проверка типов + Vitest) на каждый +push в `main` и на каждый PR — тот же принцип, что в `request-portal`, но без Jest/Heft +(там их привязка к SPFx, нам он не нужен) и без шага деплоя: куда и как разворачивать +собранный `dist/`, ещё не решено. + ## Структура ``` @@ -42,8 +57,10 @@ src/ PageShell/ шапка страницы и центрированный контент features/feedback/ собственно форма обращения model.ts типы данных, справочники, пустое состояние + model.test.ts texts.ts все пользовательские строки (задел под локализацию) validateFeedback.ts правила заполнения — чистая функция + validateFeedback.test.ts useFeedbackForm.ts состояние формы, валидация, отправка FeedbackForm.tsx сборка формы из секций ThankYouScreen.tsx экран после отправки @@ -62,5 +79,5 @@ src/ пишет результат в консоль (см. `TODO` в файле). Реальную интеграцию нужно подключить отдельно — только там, без правок в компонентах. -**Список компаний.** В `model.ts` заполнено одно значение — «ТОО АСТЕЛ-К», единственное -видимое в исходной форме. Полный список нужно уточнить у владельца формы. +**Деплой.** Тестовый workflow есть, деплоя нет — не решено, где и как будет жить +собранный `dist/`. diff --git a/package-lock.json b/package-lock.json index eb1d82b..ca2f75f 100644 --- a/package-lock.json +++ b/package-lock.json @@ -1,11 +1,11 @@ { - "name": "feedback-form", + "name": "hr-appeals", "version": "0.1.0", "lockfileVersion": 3, "requires": true, "packages": { "": { - "name": "feedback-form", + "name": "hr-appeals", "version": "0.1.0", "dependencies": { "@fluentui/react": "^8.106.4", @@ -21,7 +21,8 @@ "@vitejs/plugin-react": "^4.3.4", "sass": "^1.83.0", "typescript": "~5.8.0", - "vite": "^6.0.7" + "vite": "^6.0.7", + "vitest": "^4.1.11" }, "engines": { "node": ">=20" @@ -1682,6 +1683,13 @@ "win32" ] }, + "node_modules/@standard-schema/spec": { + "version": "1.1.0", + "resolved": "https://registry.npmjs.org/@standard-schema/spec/-/spec-1.1.0.tgz", + "integrity": "sha512-l2aFy5jALhniG5HgqrD6jXLi/rUWrKvqN/qJx6yoJsgKhblVd+iqqU4RCXavm/jPityDo5TCvKMnpjKnOriy0w==", + "dev": true, + "license": "MIT" + }, "node_modules/@swc/helpers": { "version": "0.5.23", "resolved": "https://registry.npmjs.org/@swc/helpers/-/helpers-0.5.23.tgz", @@ -1748,6 +1756,24 @@ "@babel/types": "^7.28.2" } }, + "node_modules/@types/chai": { + "version": "5.2.3", + "resolved": "https://registry.npmjs.org/@types/chai/-/chai-5.2.3.tgz", + "integrity": "sha512-Mw558oeA9fFbv65/y4mHtXDs9bPnFMZAL/jxdPFUpOHHIXX91mcgEHbS5Lahr+pwZFR8A7GQleRWeI6cGFC2UA==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/deep-eql": "*", + "assertion-error": "^2.0.1" + } + }, + "node_modules/@types/deep-eql": { + "version": "4.0.2", + "resolved": "https://registry.npmjs.org/@types/deep-eql/-/deep-eql-4.0.2.tgz", + "integrity": "sha512-c9h9dVVMigMPc4bwTvC5dxqtqJZwQPePsWjPlpSOnojbor6pGqdk541lfA7AqFQr5pB1BRdq0juY9db81BwyFw==", + "dev": true, + "license": "MIT" + }, "node_modules/@types/estree": { "version": "1.0.9", "resolved": "https://registry.npmjs.org/@types/estree/-/estree-1.0.9.tgz", @@ -1818,6 +1844,129 @@ "vite": "^4.2.0 || ^5.0.0 || ^6.0.0 || ^7.0.0" } }, + "node_modules/@vitest/expect": { + "version": "4.1.11", + "resolved": "https://registry.npmjs.org/@vitest/expect/-/expect-4.1.11.tgz", + "integrity": "sha512-VX2x5vNJXET47KAFzwERI+KRMtTTCSWTfSMKsW7JsUsXV4psq++e3DvZpuTDOpHcxytiDs6p2nhVb2tVDiiUYw==", + "dev": true, + "license": "MIT", + "dependencies": { + "@standard-schema/spec": "^1.1.0", + "@types/chai": "^5.2.2", + "@vitest/spy": "4.1.11", + "@vitest/utils": "4.1.11", + "chai": "^6.2.2", + "tinyrainbow": "^3.1.0" + }, + "funding": { + "url": "https://opencollective.com/vitest" + } + }, + "node_modules/@vitest/mocker": { + "version": "4.1.11", + "resolved": "https://registry.npmjs.org/@vitest/mocker/-/mocker-4.1.11.tgz", + "integrity": "sha512-2XJVD55d1o5AZous5CCGKS74g/riOj9odEt2bQpCVZeblHyHdnMeFl4jl0XjU21stf4mbjUkew2eXQZt65g5CQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@vitest/spy": "4.1.11", + "estree-walker": "^3.0.3", + "magic-string": "^0.30.21" + }, + "funding": { + "url": "https://opencollective.com/vitest" + }, + "peerDependencies": { + "msw": "^2.4.9", + "vite": "^6.0.0 || ^7.0.0 || ^8.0.0" + }, + "peerDependenciesMeta": { + "msw": { + "optional": true + }, + "vite": { + "optional": true + } + } + }, + "node_modules/@vitest/pretty-format": { + "version": "4.1.11", + "resolved": "https://registry.npmjs.org/@vitest/pretty-format/-/pretty-format-4.1.11.tgz", + "integrity": "sha512-yiZzPbGTS9Sr/JpFl8zHrcIkAofNbFV6k21vIgQN/cY/oxZeXhJv5sc/MBJ5jFKWmWs+oJHw0UXLZjmf931+Vw==", + "dev": true, + "license": "MIT", + "dependencies": { + "tinyrainbow": "^3.1.0" + }, + "funding": { + "url": "https://opencollective.com/vitest" + } + }, + "node_modules/@vitest/runner": { + "version": "4.1.11", + "resolved": "https://registry.npmjs.org/@vitest/runner/-/runner-4.1.11.tgz", + "integrity": "sha512-LztvUgdwMNJMIkj3hQnnxiC2Xy1zNxq928W/xhjCLaNCzqTZOudjwbQf6v9IntZGPw132i2Lq2rgTRZHD3JHNw==", + "dev": true, + "license": "MIT", + "dependencies": { + "@vitest/utils": "4.1.11", + "pathe": "^2.0.3" + }, + "funding": { + "url": "https://opencollective.com/vitest" + } + }, + "node_modules/@vitest/snapshot": { + "version": "4.1.11", + "resolved": "https://registry.npmjs.org/@vitest/snapshot/-/snapshot-4.1.11.tgz", + "integrity": "sha512-pN7ikn1ON7h8ee4gIAp4AzyK+zBtJPzVbqOgu5LCEh4VaJVbPQcgYQYJIMGQPXVeJJq1fnfazis7a5pFNPahog==", + "dev": true, + "license": "MIT", + "dependencies": { + "@vitest/pretty-format": "4.1.11", + "@vitest/utils": "4.1.11", + "magic-string": "^0.30.21", + "pathe": "^2.0.3" + }, + "funding": { + "url": "https://opencollective.com/vitest" + } + }, + "node_modules/@vitest/spy": { + "version": "4.1.11", + "resolved": "https://registry.npmjs.org/@vitest/spy/-/spy-4.1.11.tgz", + "integrity": "sha512-apNa/prQy2qCeywhnixOHPRCgGNhvg7T4Dapfl1GahLp/R+uhBm5cPyFoNVyqsNd2h1nJxL6BqqdIjiABL60YA==", + "dev": true, + "license": "MIT", + "funding": { + "url": "https://opencollective.com/vitest" + } + }, + "node_modules/@vitest/utils": { + "version": "4.1.11", + "resolved": "https://registry.npmjs.org/@vitest/utils/-/utils-4.1.11.tgz", + "integrity": "sha512-zTCVGpyFsGWBhllOyKlTw/vnr6D9qxsfSDyfbyZmTyjHw5N/VuvzHpHoQjm2ZJzn4RJgx5w4r7V0er69CmLgPQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@vitest/pretty-format": "4.1.11", + "convert-source-map": "^2.0.0", + "tinyrainbow": "^3.1.0" + }, + "funding": { + "url": "https://opencollective.com/vitest" + } + }, + "node_modules/assertion-error": { + "version": "2.0.1", + "resolved": "https://registry.npmjs.org/assertion-error/-/assertion-error-2.0.1.tgz", + "integrity": "sha512-Izi8RQcffqCeNVgFigKli1ssklIbpHnCYc6AknXGYoB6grJqyeby7jv12JUQgmTAnIDnbck1uxksT4dzN3PWBA==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=12" + } + }, "node_modules/baseline-browser-mapping": { "version": "2.11.15", "resolved": "https://registry.npmjs.org/baseline-browser-mapping/-/baseline-browser-mapping-2.11.15.tgz", @@ -1886,6 +2035,16 @@ ], "license": "CC-BY-4.0" }, + "node_modules/chai": { + "version": "6.2.2", + "resolved": "https://registry.npmjs.org/chai/-/chai-6.2.2.tgz", + "integrity": "sha512-NUPRluOfOiTKBKvWPtSD4PhFvWCqOi0BGStNWs57X9js7XGTprSmFoz5F0tWhR4WPjNeR9jXqdC7/UpSJTnlRg==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=18" + } + }, "node_modules/chokidar": { "version": "5.0.0", "resolved": "https://registry.npmjs.org/chokidar/-/chokidar-5.0.0.tgz", @@ -1951,6 +2110,13 @@ "dev": true, "license": "ISC" }, + "node_modules/es-module-lexer": { + "version": "2.3.2", + "resolved": "https://registry.npmjs.org/es-module-lexer/-/es-module-lexer-2.3.2.tgz", + "integrity": "sha512-poHGpORABojJJucnV9KbOavETW8lBVnphkW77ER5/BQ5Fz7oXSoCNek7IH3vR5nRjdsEz926ibFYX8KtLQmdyw==", + "dev": true, + "license": "MIT" + }, "node_modules/esbuild": { "version": "0.25.12", "resolved": "https://registry.npmjs.org/esbuild/-/esbuild-0.25.12.tgz", @@ -2003,6 +2169,26 @@ "node": ">=6" } }, + "node_modules/estree-walker": { + "version": "3.0.3", + "resolved": "https://registry.npmjs.org/estree-walker/-/estree-walker-3.0.3.tgz", + "integrity": "sha512-7RUKfXgSMMkzt6ZuXmqapOurLGPPfgj6l9uRZ7lRGolvk0y2yocc35LdcxKC5PQZdn2DMqioAQ2NoWcrTKmm6g==", + "dev": true, + "license": "MIT", + "dependencies": { + "@types/estree": "^1.0.0" + } + }, + "node_modules/expect-type": { + "version": "1.4.0", + "resolved": "https://registry.npmjs.org/expect-type/-/expect-type-1.4.0.tgz", + "integrity": "sha512-KfYbmpRm0VbLjEvVa9yGwCi9GI34xvi7A/HXYWQO65CSD2u3MczUJSuwXKFIxlGsgBQizV9q5J9NHj4VG0n+pA==", + "dev": true, + "license": "Apache-2.0", + "engines": { + "node": ">=12.0.0" + } + }, "node_modules/fdir": { "version": "6.5.0", "resolved": "https://registry.npmjs.org/fdir/-/fdir-6.5.0.tgz", @@ -2132,6 +2318,16 @@ "yallist": "^3.0.2" } }, + "node_modules/magic-string": { + "version": "0.30.21", + "resolved": "https://registry.npmjs.org/magic-string/-/magic-string-0.30.21.tgz", + "integrity": "sha512-vd2F4YUyEXKGcLHoq+TEyCjxueSeHnFxyyjNp80yg0XV4vUhnDer/lvvlqM/arB5bXQN5K2/3oinyCRyx8T2CQ==", + "dev": true, + "license": "MIT", + "dependencies": { + "@jridgewell/sourcemap-codec": "^1.5.5" + } + }, "node_modules/ms": { "version": "2.1.3", "resolved": "https://registry.npmjs.org/ms/-/ms-2.1.3.tgz", @@ -2185,6 +2381,27 @@ "node": ">=0.10.0" } }, + "node_modules/obug": { + "version": "2.1.4", + "resolved": "https://registry.npmjs.org/obug/-/obug-2.1.4.tgz", + "integrity": "sha512-4a+OsYv9UktOJKE+l1A4OufDgdRF9PifWj+tJnHURo/P+WOxpG4GzUFL9qCalmWauao6ogiG+QvnCovwPoyAWA==", + "dev": true, + "funding": [ + "https://github.com/sponsors/sxzz", + "https://opencollective.com/debug" + ], + "license": "MIT", + "engines": { + "node": ">=12.20.0" + } + }, + "node_modules/pathe": { + "version": "2.0.3", + "resolved": "https://registry.npmjs.org/pathe/-/pathe-2.0.3.tgz", + "integrity": "sha512-WUjGcAqP1gQacoQe+OBJsFA7Ld4DyXuUIjZ5cc75cLHvJ7dtNsTugphxIADwspS+AraAUePCKrSVtPLFj/F88w==", + "dev": true, + "license": "MIT" + }, "node_modules/picocolors": { "version": "1.1.1", "resolved": "https://registry.npmjs.org/picocolors/-/picocolors-1.1.1.tgz", @@ -2372,6 +2589,13 @@ "semver": "bin/semver.js" } }, + "node_modules/siginfo": { + "version": "2.0.0", + "resolved": "https://registry.npmjs.org/siginfo/-/siginfo-2.0.0.tgz", + "integrity": "sha512-ybx0WO1/8bSBLEWXZvEd7gMW3Sn3JFlW3TvX1nREbDLRNQNaeNN8WK0meBwPdAaOI7TtRRRJn/Es1zhrrCHu7g==", + "dev": true, + "license": "ISC" + }, "node_modules/source-map-js": { "version": "1.2.1", "resolved": "https://registry.npmjs.org/source-map-js/-/source-map-js-1.2.1.tgz", @@ -2382,6 +2606,37 @@ "node": ">=0.10.0" } }, + "node_modules/stackback": { + "version": "0.0.2", + "resolved": "https://registry.npmjs.org/stackback/-/stackback-0.0.2.tgz", + "integrity": "sha512-1XMJE5fQo1jGH6Y/7ebnwPOBEkIEnT4QF32d5R1+VXdXveM0IBMJt8zfaxX1P3QhVwrYe+576+jkANtSS2mBbw==", + "dev": true, + "license": "MIT" + }, + "node_modules/std-env": { + "version": "4.2.0", + "resolved": "https://registry.npmjs.org/std-env/-/std-env-4.2.0.tgz", + "integrity": "sha512-oCUKSupKTHX53EyjDtuZQ64pjLJ6yYCtpmEw0goYxtjG9KpbRe8KAsl2tBUGU9DyMcJ0RwJ8GqJAFzMXcXW1Rw==", + "dev": true, + "license": "MIT" + }, + "node_modules/tinybench": { + "version": "2.9.0", + "resolved": "https://registry.npmjs.org/tinybench/-/tinybench-2.9.0.tgz", + "integrity": "sha512-0+DUvqWMValLmha6lr4kD8iAMK1HzV0/aKnCtWb9v9641TnP/MFb7Pc2bxoxQjTXAErryXVgUOfv2YqNllqGeg==", + "dev": true, + "license": "MIT" + }, + "node_modules/tinyexec": { + "version": "1.3.0", + "resolved": "https://registry.npmjs.org/tinyexec/-/tinyexec-1.3.0.tgz", + "integrity": "sha512-QKAl9m8gWWGHV8jZcPeym6j+XULi6tOf1mT83WYJ4Lk2ytW/uwAWkrP0uFsdoYMdueVJ0qs26wZ+23xeB4ibNQ==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=18" + } + }, "node_modules/tinyglobby": { "version": "0.2.17", "resolved": "https://registry.npmjs.org/tinyglobby/-/tinyglobby-0.2.17.tgz", @@ -2399,6 +2654,16 @@ "url": "https://github.com/sponsors/SuperchupuDev" } }, + "node_modules/tinyrainbow": { + "version": "3.1.1", + "resolved": "https://registry.npmjs.org/tinyrainbow/-/tinyrainbow-3.1.1.tgz", + "integrity": "sha512-yau8yJdTt989Mm0Bd/236QnzEiPf2xLLTqUZRUJOo/3CB078LSwzei343DgtJVmfJKJE3TMINY1u42SQsP6mXw==", + "dev": true, + "license": "MIT", + "engines": { + "node": ">=14.0.0" + } + }, "node_modules/tslib": { "version": "2.8.1", "resolved": "https://registry.npmjs.org/tslib/-/tslib-2.8.1.tgz", @@ -2532,6 +2797,113 @@ } } }, + "node_modules/vitest": { + "version": "4.1.11", + "resolved": "https://registry.npmjs.org/vitest/-/vitest-4.1.11.tgz", + "integrity": "sha512-fhACrNXUidIbGSBr5FlbuBkO7VWC1ZyLl0DO4CU2DrQoAPxX84Ysxs+HeGQpii5lZWV1Q4gBZTTu49mF+A6Edw==", + "dev": true, + "license": "MIT", + "dependencies": { + "@vitest/expect": "4.1.11", + "@vitest/mocker": "4.1.11", + "@vitest/pretty-format": "4.1.11", + "@vitest/runner": "4.1.11", + "@vitest/snapshot": "4.1.11", + "@vitest/spy": "4.1.11", + "@vitest/utils": "4.1.11", + "es-module-lexer": "^2.0.0", + "expect-type": "^1.3.0", + "magic-string": "^0.30.21", + "obug": "^2.1.1", + "pathe": "^2.0.3", + "picomatch": "^4.0.3", + "std-env": "^4.0.0-rc.1", + "tinybench": "^2.9.0", + "tinyexec": "^1.0.2", + "tinyglobby": "^0.2.15", + "tinyrainbow": "^3.1.0", + "vite": "^6.0.0 || ^7.0.0 || ^8.0.0", + "why-is-node-running": "^2.3.0" + }, + "bin": { + "vitest": "vitest.mjs" + }, + "engines": { + "node": "^20.0.0 || ^22.0.0 || >=24.0.0" + }, + "funding": { + "url": "https://opencollective.com/vitest" + }, + "peerDependencies": { + "@edge-runtime/vm": "*", + "@opentelemetry/api": "^1.9.0", + "@types/node": "^20.0.0 || ^22.0.0 || >=24.0.0", + "@vitest/browser-playwright": "4.1.11", + "@vitest/browser-preview": "4.1.11", + "@vitest/browser-webdriverio": "4.1.11", + "@vitest/coverage-istanbul": "4.1.11", + "@vitest/coverage-v8": "4.1.11", + "@vitest/ui": "4.1.11", + "happy-dom": "*", + "jsdom": "*", + "vite": "^6.0.0 || ^7.0.0 || ^8.0.0" + }, + "peerDependenciesMeta": { + "@edge-runtime/vm": { + "optional": true + }, + "@opentelemetry/api": { + "optional": true + }, + "@types/node": { + "optional": true + }, + "@vitest/browser-playwright": { + "optional": true + }, + "@vitest/browser-preview": { + "optional": true + }, + "@vitest/browser-webdriverio": { + "optional": true + }, + "@vitest/coverage-istanbul": { + "optional": true + }, + "@vitest/coverage-v8": { + "optional": true + }, + "@vitest/ui": { + "optional": true + }, + "happy-dom": { + "optional": true + }, + "jsdom": { + "optional": true + }, + "vite": { + "optional": false + } + } + }, + "node_modules/why-is-node-running": { + "version": "2.3.0", + "resolved": "https://registry.npmjs.org/why-is-node-running/-/why-is-node-running-2.3.0.tgz", + "integrity": "sha512-hUrmaWBdVDcxvYqnyh09zunKzROWjbZTiNy8dBEjkS7ehEDQibXJ7XvlmtbwuTclUiIyN+CyXQD4Vmko8fNm8w==", + "dev": true, + "license": "MIT", + "dependencies": { + "siginfo": "^2.0.0", + "stackback": "0.0.2" + }, + "bin": { + "why-is-node-running": "cli.js" + }, + "engines": { + "node": ">=8" + } + }, "node_modules/yallist": { "version": "3.1.1", "resolved": "https://registry.npmjs.org/yallist/-/yallist-3.1.1.tgz", diff --git a/package.json b/package.json index 71fe48e..077aa5d 100644 --- a/package.json +++ b/package.json @@ -8,9 +8,12 @@ }, "scripts": { "dev": "vite", - "build": "tsc --noEmit && vite build", + "build": "tsc --noEmit && vitest run && vite build", "preview": "vite preview", - "typecheck": "tsc --noEmit" + "typecheck": "tsc --noEmit", + "test": "vitest run", + "test:watch": "vitest", + "ci": "tsc --noEmit && vitest run" }, "dependencies": { "@fluentui/react": "^8.106.4", @@ -26,6 +29,7 @@ "@vitejs/plugin-react": "^4.3.4", "sass": "^1.83.0", "typescript": "~5.8.0", - "vite": "^6.0.7" + "vite": "^6.0.7", + "vitest": "^4.1.11" } } diff --git a/src/features/feedback/model.test.ts b/src/features/feedback/model.test.ts new file mode 100644 index 0000000..6642779 --- /dev/null +++ b/src/features/feedback/model.test.ts @@ -0,0 +1,49 @@ +import { COMPANY_OPTIONS, getAppealFieldsConfig } from './model'; + +// Форма (AppealSection) и валидация (validateFeedback) читают одно и то же +// правило из getAppealFieldsConfig — если оно разойдётся, форма и валидация +// начнут спорить друг с другом. Эти тесты фиксируют контракт правила. +describe('getAppealFieldsConfig', () => { + it('жалоба: описание обязательно, предложение показано, но необязательно', () => { + const fields = getAppealFieldsConfig('complaint'); + expect(fields.situation).toEqual({ visible: true, required: true }); + expect(fields.suggestion).toEqual({ visible: true, required: false }); + }); + + it('предложение: описания ситуации нет вовсе, само предложение обязательно', () => { + const fields = getAppealFieldsConfig('suggestion'); + expect(fields.situation).toEqual({ visible: false, required: false }); + expect(fields.suggestion).toEqual({ visible: true, required: true }); + }); + + it('другое: обязательно только описание, предложения не показываем', () => { + const fields = getAppealFieldsConfig('other'); + expect(fields.situation).toEqual({ visible: true, required: true }); + expect(fields.suggestion).toEqual({ visible: false, required: false }); + }); + + it('обязательное поле не может быть скрыто ни для одного вида обращения', () => { + // Форма не сможет попросить исправить поле, которого не видно на экране — + // required: true при visible: false — это несогласованная настройка. + (['complaint', 'suggestion', 'other'] as const).forEach((kind) => { + const fields = getAppealFieldsConfig(kind); + expect(fields.situation.required && !fields.situation.visible).toBe(false); + expect(fields.suggestion.required && !fields.suggestion.visible).toBe(false); + }); + }); +}); + +// Справочник компаний используется бэкендом по Id (см. комментарий в model.ts) — +// дубликат Id незаметно перепутал бы два разных юрлица. +describe('COMPANY_OPTIONS', () => { + it('не содержит повторяющихся Id', () => { + const ids = COMPANY_OPTIONS.map((option) => option.value); + expect(new Set(ids).size).toBe(ids.length); + }); + + it('у каждой компании есть непустое название', () => { + COMPANY_OPTIONS.forEach((option) => { + expect(option.label.trim().length).toBeGreaterThan(0); + }); + }); +}); diff --git a/src/features/feedback/validateFeedback.test.ts b/src/features/feedback/validateFeedback.test.ts new file mode 100644 index 0000000..70e6705 --- /dev/null +++ b/src/features/feedback/validateFeedback.test.ts @@ -0,0 +1,122 @@ +import { EMPTY_FEEDBACK, type IFeedbackData } from './model'; +import { TEXTS } from './texts'; +import { validateFeedback } from './validateFeedback'; + +const VALID_COMPANY = '9de064a4-44c8-88ef-11ec-8285925d83d2'; +const LONG_TEXT = 'Развёрнутое описание, которого достаточно для проверки правила'; +const SHORT_TEXT = 'коротко'; + +/** Полностью заполненное «жалоба»-обращение — стартовая точка для тестов. */ +function buildValidComplaint(overrides: Partial = {}): IFeedbackData { + return { + ...EMPTY_FEEDBACK, + company: VALID_COMPANY, + fullName: 'Иванов Иван Иванович', + kind: 'complaint', + situation: LONG_TEXT, + suggestion: LONG_TEXT, + ...overrides + }; +} + +describe('validateFeedback: базовые обязательные поля', () => { + it('полностью заполненное обращение проходит без ошибок', () => { + expect(validateFeedback(buildValidComplaint())).toEqual({}); + }); + + it('без компании — ошибка', () => { + const errors = validateFeedback(buildValidComplaint({ company: '' })); + expect(errors.company).toBeDefined(); + }); + + it('именное обращение без ФИО — ошибка', () => { + const errors = validateFeedback(buildValidComplaint({ fullName: '' })); + expect(errors.fullName).toBeDefined(); + }); + + it('анонимное обращение без ФИО — не ошибка (поле скрыто, требовать его нельзя)', () => { + const errors = validateFeedback( + buildValidComplaint({ isAnonymous: true, fullName: '' }) + ); + expect(errors.fullName).toBeUndefined(); + }); +}); + +describe('validateFeedback: вид обращения "Другое"', () => { + it('без уточнения вида — ошибка', () => { + const errors = validateFeedback( + buildValidComplaint({ kind: 'other', kindOther: '', situation: LONG_TEXT }) + ); + expect(errors.kindOther).toBeDefined(); + }); + + it('с уточнением — ошибки по kindOther нет', () => { + const errors = validateFeedback( + buildValidComplaint({ kind: 'other', kindOther: 'Свой вариант', situation: LONG_TEXT }) + ); + expect(errors.kindOther).toBeUndefined(); + }); +}); + +// Состав обязательных полей зависит от вида обращения (см. getAppealFieldsConfig +// в model.ts) — эти тесты фиксируют контракт, чтобы форма и валидация не разошлись. +describe('validateFeedback: обязательность полей по видам обращения', () => { + it('жалоба: описание ситуации обязательно', () => { + const errors = validateFeedback(buildValidComplaint({ situation: '' })); + expect(errors.situation).toBeDefined(); + }); + + it('жалоба: предложение по улучшению НЕ обязательно', () => { + const errors = validateFeedback(buildValidComplaint({ suggestion: '' })); + expect(errors.suggestion).toBeUndefined(); + }); + + it('жалоба: если предложение всё же указано, но слишком коротко — ошибка', () => { + const errors = validateFeedback(buildValidComplaint({ suggestion: SHORT_TEXT })); + expect(errors.suggestion).toBeDefined(); + }); + + it('предложение: описание ситуации не требуется, даже если пустое', () => { + const errors = validateFeedback( + buildValidComplaint({ kind: 'suggestion', situation: '', suggestion: LONG_TEXT }) + ); + expect(errors.situation).toBeUndefined(); + }); + + it('предложение: сама суть предложения обязательна', () => { + const errors = validateFeedback( + buildValidComplaint({ kind: 'suggestion', situation: '', suggestion: '' }) + ); + expect(errors.suggestion).toBeDefined(); + }); + + it('другое: описание обязательно, предложение не спрашивается вовсе', () => { + const errors = validateFeedback( + buildValidComplaint({ + kind: 'other', + kindOther: 'Свой вариант', + situation: '', + suggestion: '' + }) + ); + expect(errors.situation).toBeDefined(); + expect(errors.suggestion).toBeUndefined(); + }); +}); + +describe('validateFeedback: минимальная длина текста', () => { + it('пустое описание — ошибка «заполните поле»', () => { + const errors = validateFeedback(buildValidComplaint({ situation: '' })); + expect(errors.situation).toBe(TEXTS.errors.required); + }); + + it('слишком короткое описание — ошибка «коротко», а не «пусто»', () => { + const errors = validateFeedback(buildValidComplaint({ situation: SHORT_TEXT })); + expect(errors.situation).toBe(TEXTS.errors.tooShort); + }); + + it('достаточно длинное описание — без ошибки', () => { + const errors = validateFeedback(buildValidComplaint({ situation: LONG_TEXT })); + expect(errors.situation).toBeUndefined(); + }); +}); diff --git a/tsconfig.json b/tsconfig.json index cba9967..d7a36aa 100644 --- a/tsconfig.json +++ b/tsconfig.json @@ -13,6 +13,9 @@ "noEmit": true, "jsx": "react-jsx", + // vitest/globals — describe/it/expect в *.test.ts типизированы без импорта. + "types": ["vitest/globals"], + "strict": true, "noUnusedLocals": true, "noUnusedParameters": true, diff --git a/vite.config.ts b/vite.config.ts index 140cedd..6ef91fc 100644 --- a/vite.config.ts +++ b/vite.config.ts @@ -1,4 +1,5 @@ -import { defineConfig } from 'vite'; +/// +import { defineConfig } from 'vitest/config'; import react from '@vitejs/plugin-react'; import { fileURLToPath, URL } from 'node:url'; @@ -12,5 +13,13 @@ export default defineConfig({ server: { port: 3000, open: true + }, + test: { + // globals: true — describe/it/expect доступны без импорта, как привычно по Jest. + globals: true, + // Тестируем чистую логику (валидация, конфиг полей), а не React-компоненты, + // поэтому DOM не нужен — node-окружение быстрее jsdom. + environment: 'node', + include: ['src/**/*.test.ts'] } });