Skip to content

Live-тесты: убрать проверки MODX/testbench вместо MiniShop3 - #700

Open
Ibochkarev wants to merge 1 commit into
betafrom
test/issue-697-drop-hollow-live-tests
Open

Live-тесты: убрать проверки MODX/testbench вместо MiniShop3#700
Ibochkarev wants to merge 1 commit into
betafrom
test/issue-697-drop-hollow-live-tests

Conversation

@Ibochkarev

Copy link
Copy Markdown
Member

Описание

Часть live-тестов в tests/Modx/ не могла покраснеть из-за поломки MiniShop3: они проверяли подготовку testbench, ядро MODX или дублировали соседние кейсы.

Удалены hollow-тесты и дубли. Оставлен phinx.php vs live $modx dbname (PhinxConfigLiveTest). Предупреждение про строковый action без processors_path осталось в README.

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

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

Связанные Issues

Closes #697

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

cd core/components/minishop3
php -l tests/Modx/PhinxConfigLiveTest.php          # exit 0
php -l tests/Modx/StatusCreateProcessorTest.php    # exit 0
php -l tests/Modx/Support/ExtraTestCase.php        # exit 0

php vendor/bin/phpunit -c phpunit.modx.xml --filter \
  'PhinxConfigLiveTest|StatusCreateProcessorTest|ServiceRegistryLiveTest'
# exit 0 — Tests: 6, Assertions: 193, Skipped: 1 (ACL deny без session, как на beta)

Локально ModelPersistTest красный на загрязнённой БД (msProductField / schema drift от Phinx). К #697 не относится. Полный composer test:modx ожидается в CI.

  • Ручное тестирование
  • Автоматические тесты (composer ci:php / composer test, npm run lint:ci, composer stan / GitHub Actions CI)
  • Тестирование на разных версиях PHP/MODX

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

  • MiniShop3: ветка test/issue-697-drop-hollow-live-tests
  • MODX: testbench 3.2.3-pl
  • PHP: 8.4.23

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

До После
n/a n/a

Чеклист

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

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

Удалено:

  • SchemaTablesTest (тавтология SHOW TABLES vs PackageModels::tables(); схема из миграций — Live-тесты: схема из Phinx-миграций вместо карты xPDO #695)
  • MsOrderPersistTest (дубль ModelPersistTest)
  • setting round-trip / registry smoke из бывшего TokenNameSettingTest (MODX helpers / дубль ServiceRegistryLiveTest)
  • StatusCreateProcessorTest::testStringActionWithoutProcessorsPathIsNotFound (поведение ядра, warning в README)

Оставлено: PhinxConfigLiveTest ($modx = $this->modx обязателен перед require phinx.php).

Keep only MiniShop3-backed checks: phinx.php vs live dbname, processors,
models, DI, events. Leave the processors_path warning in the suite README.
@Ibochkarev
Ibochkarev requested a review from biz87 September 11, 2026 03:07
@Ibochkarev Ibochkarev self-assigned this Sep 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Live-тесты: убрать тесты, которые проверяют MODX и testbench, а не MiniShop3

1 participant