Skip to content

Migrate vueManager node tests to vitest behavior checks - #693

Open
Ibochkarev wants to merge 1 commit into
betafrom
test/issue-692-vitest-behavior
Open

Migrate vueManager node tests to vitest behavior checks#693
Ibochkarev wants to merge 1 commit into
betafrom
test/issue-692-vitest-behavior

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Четыре теста в vueManager/tests/ гонялись через node --test и читали .vue как текст. Такой тест ловит откат строки и пропускает переименование или вынос confirm.require в хелпер. Набор переехал на vitest: гриды монтируются, чистые функции остались проверками логики.

vueManager/tests/ удалён. npm run test:smoke оставляет только scripts/check-ui-group-smoke.mjs (#539, без DOM). Шаг Smoke tests в CI сохранён.

Тип изменений

  • Исправление бага (non-breaking change)
  • Новая функциональность (non-breaking change)
  • Breaking change (изменение, ломающее обратную совместимость)
  • Рефакторинг (без изменения функциональности)
  • Документация
  • Другое (опишите):

Связанные Issues

Closes #692

Как это было протестировано?

cd vueManager
npm test          # exit 0, 71 tests
npm run lint:ci   # exit 0
npm run test:smoke  # exit 0, OK: check-ui-group-smoke
  • Ручное тестирование
  • Автоматические тесты (npm test, npm run lint:ci, npm run test:smoke)
  • Тестирование на разных версиях PHP/MODX

Конфигурация тестирования:

  • MiniShop3: ветка test/issue-692-vitest-behavior
  • MODX: n/a
  • PHP: n/a
  • Node: локальный vitest 3 / ESLint из vueManager

Скриншоты (если применимо)

До После

Чеклист

  • Код соответствует стилю проекта
  • Добавлены/обновлены комментарии в сложных местах
  • Изменения не ломают существующую функциональность
  • Лексиконы добавлены на двух языках (ru/en)
  • PHPStan проходит без новых ошибок (composer stan / CI job PHPStan)
  • ESLint проходит без ошибок (npm run lint:ci для Vue)
  • Обновлён CHANGELOG.md (для значимых изменений)

Дополнительные заметки

settingsConfirmDialog.test.js монтирует все 7 settings-гридов: уникальный ConfirmDialog.group, изоляция Options+Groups на одной странице (#548), одиночный request.delete после accept в пяти ActionsColumn-гридах (#630).

Source-grep tests missed equivalent refactors that restored double confirm and ungrouped ConfirmDialog. Mount settings grids and assert isolation plus a single delete request.
@Ibochkarev Ibochkarev added priority: low Низкий приоритет, когда будет время javascript Pull requests that update javascript code tech-debt Maintainability / refactor / architecture debt labels Sep 10, 2026
@Ibochkarev
Ibochkarev requested a review from biz87 September 10, 2026 01:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Pull requests that update javascript code priority: low Низкий приоритет, когда будет время tech-debt Maintainability / refactor / architecture debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Перенести node --test тесты vueManager на vitest с проверкой поведения

1 participant