Fix missing Run icon on test method ruler

[AKunitsynGC]

Summary

• Completes markerAnnotationSpecification in plugin.xml: RGB was incorrectly set as colorPreferenceKey, and required preference keys (colorPreferenceValue, textPreferenceKey, overviewRulerPreferenceKey, …) were missing, so Eclipse dropped the annotation type (MarkerAnnotationPreferences.isComplete).
• Changes TestMethodMarker from @Check(CheckType.EXPENSIVE) to NORMAL, so markers (and the green Run icon) appear during normal module validation, not only after Project → Validate/Clean.

Fixes #86

Test plan

• Install build on EDT 2026.2 / Eclipse 4.38
• Open a YAxUnit module with ИсполняемыеСценарии() Экспорт
• Confirm green Run icons on the vertical ruler; click runs the test
• Optional: Preferences → Annotations shows «Тестовый метод» (may need clean restart after version bump)
• Cmd+Shift+T still works

Made with Cursor

Summary by CodeRabbit

• Исправления
  • Скорректированы настройки отображения аннотаций тестовых методов в редакторе Eclipse.
  • Проверка тестовых модулей теперь выполняется в рамках обычной валидации, без дополнительной ресурсоёмкой проверки.
  • Улучшено отображение линейки и стилей текста для тестовых маркеров.

[coderabbitai]

Walkthrough

Конфигурация Eclipse-аннотации тестового метода дополнена отдельными настройками отображения. Проверка тестовых модулей переведена с CheckType.EXPENSIVE на CheckType.NORMAL.

Changes

Маркер тестового метода

Layer / File(s)	Summary
Полная конфигурация аннотации viewer/plugin.xml	Для testMethodAnnotation добавлены именованные ключи и значения цвета, текста, стиля, подсветки и линеек. Установлены presentationLayer="5" и contributesToHeader="false".
Обычная проверка тестовых модулей viewer/src/main/java/ru/biatech/edt/junit/ui/editor/ruler/TestMethodMarker.java	Метод validate использует CheckType.NORMAL вместо CheckType.EXPENSIVE. Добавлен комментарий о проверке при обычной валидации модуля.

Estimated code review effort: 2 (Simple) | ~10 минут

Poem

Я兔ик вижу зелёный след,
Маркер прыгает на свет.
Настройки выстроились в ряд,
Проверки больше не спешат.
Пик-пик — тесты отмечены!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	Заголовок точно описывает основное изменение: восстановление значка запуска рядом с методами тестов.
Linked Issues check	✅ Passed	Изменения завершают markerAnnotationSpecification и переводят проверку на CheckType.NORMAL, что соответствует требованиям задачи #86.
Out of Scope Changes check	✅ Passed	Все изменения относятся к отображению значка тестового метода и созданию маркеров при обычной валидации.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

🧹 Nitpick comments (1)

viewer/src/main/java/ru/biatech/edt/junit/ui/editor/ruler/TestMethodMarker.java (1)

49-54: 🚀 Performance & Scalability | 🔵 Trivial | 🏗️ Heavy lift

Ограничьте полную пересборку маркеров при CheckType.NORMAL.

NORMAL запускается при сохранении, сборке и явной проверке, но не после каждого ввода. Каждый вызов удаляет все маркеры, просматривает все методы модуля и создаёт маркер для каждого теста. Для больших модулей обновляйте только изменившиеся маркеры или добавьте кэширование результата поиска тестов.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@viewer/src/main/java/ru/biatech/edt/junit/ui/editor/ruler/TestMethodMarker.java`
around lines 49 - 54, Ограничьте обработку в TestMethodMarker при
`@Check`(CheckType.NORMAL): не удаляйте и не пересоздавайте маркеры для всех
методов модуля при каждом вызове. Обновляйте только маркеры изменившихся методов
либо кэшируйте результаты поиска тестов, сохранив создание маркеров для новых и
изменение или удаление маркеров для затронутых тестов.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In
`@viewer/src/main/java/ru/biatech/edt/junit/ui/editor/ruler/TestMethodMarker.java`:
- Around line 49-54: Ограничьте обработку в TestMethodMarker при
`@Check`(CheckType.NORMAL): не удаляйте и не пересоздавайте маркеры для всех
методов модуля при каждом вызове. Обновляйте только маркеры изменившихся методов
либо кэшируйте результаты поиска тестов, сохранив создание маркеров для новых и
изменение или удаление маркеров для затронутых тестов.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3010b5eb-3989-43e6-a940-766b53c3e734

📥 Commits

Reviewing files that changed from the base of the PR and between d5def28 and db33427.

📒 Files selected for processing (2)

• viewer/plugin.xml
• viewer/src/main/java/ru/biatech/edt/junit/ui/editor/ruler/TestMethodMarker.java