Skip to content

zavod #56: Не выдавать ответ после неверной попытки - #48

Merged
gon7187 merged 5 commits into
mainfrom
zavod/56-ne-vydavat-otvet-posle-nevernoy
Oct 22, 2026
Merged

gon7187 merged 5 commits into
mainfrom
zavod/56-ne-vydavat-otvet-posle-nevernoy

Conversation

@gon7187

@gon7187 gon7187 commented Oct 22, 2026

Copy link
Copy Markdown
Owner

Не выдавать ответ после неверной попытки

Круг: 2

Критерии

  • ✓ c1 — app/main.py:168–182,263–268; tests/test_question_check.py:17,44; залоченный tests/test_answer_reveal.py:122 проверяет choice/input/code и верный ответ; отчёт круга: 10 passed.
  • ✓ c2 — app/main.py:304–322 применяет public_answer_result; залоченный tests/test_answer_reveal.py:146 проверяет отсутствие explanation во всех полях неверных ответов и наличие при верных; отчёт круга: 10 passed.
  • ✓ c3 — app/main.py:186–197,224,357: итоговые проверки используют grade_answers без фильтрации; app/evaluator.py:645–651 сохраняет объяснения; залоченный tests/test_answer_reveal.py:184 проверяет результаты урока и экзамена; отчёт круга: 10 passed.
  • ✓ c4 — app/static/app.js:213–230: заголовки вердикта, условный абзац, explanation только при верном ответе, error_hint и checks сохранены; вызовы:165,247,563. Залоченный tests/test_answer_reveal.py:224; отчёт круга: 10 passed.
  • ✓ c5 — app/static/styles.css:92 задаёт margin-bottom:12px для .question-card .code-example; общий стиль теории на строке 71 сохраняет margin:14px 0 0. Залоченные tests/test_answer_reveal.py:269,280; отчёт круга: 10 passed.

Изменённые файлы

  • app/main.py
  • app/static/app.js
  • app/static/styles.css
  • tests/test_answer_reveal.py
  • tests/test_lesson_check.py
  • tests/test_question_check.py

Предложения новых задач

  • Обновить tests/test_lesson_check.py::test_check_choice_returns_verdict_and_explanation и tests/test_question_check.py::test_c1_check_returns_verdict_explanation_and_error_hint: они ожидают truthy explanation при неверном ответе, что противоречит новому требованию c1; файлы вне footprint задачи.
  • Обновить tests/test_lesson_check.py::test_check_choice_returns_verdict_and_explanation (строка 47) под новое поведение: при неверном ответе explanation пустой/отсутствует, иначе полный pytest остаётся красным.
  • Перестать отслеживать python_path.db-journal: файл транзиентный, после прогонов тестов рабочее дерево становится грязным — убрать из индекса и добавить в .gitignore.
  • Добавить *.db-journal в .gitignore (python_path.db там уже есть), чтобы транзитный журнал SQLite не попадал в коммиты: правка вне footprint задачи.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-22T23:54:32.616098Z 24ba4b9 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@gon7187
gon7187 merged commit 77f779d into main Oct 22, 2026
3 checks passed
@gon7187
gon7187 deleted the zavod/56-ne-vydavat-otvet-posle-nevernoy branch October 22, 2026 23:53
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.

1 participant