Skip to content

fix stale vote submission error after reset - #38

Merged
laceyp99 merged 1 commit into
mainfrom
fix/reset-vote-submission-state
Aug 6, 2026
Merged

laceyp99 merged 1 commit into
mainfrom
fix/reset-vote-submission-state

Conversation

@laceyp99

@laceyp99 laceyp99 commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Clear failed submission status, message, and log warnings when resetting an active vote.
  • Add regression coverage that verifies the stale error banner is hidden after reset.

Testing

  • py -3.13 -m ruff format --check . - passed
  • py -3.13 -m ruff check . - passed
  • py -3.13 -m pytest - passed (112 tests)

@laceyp99 laceyp99 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good simple fix. Just cleaning it up properly and a regression test. Love it!

@laceyp99
laceyp99 marked this pull request as ready for review August 6, 2026 04:03
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

Agent PR review

Validation

Validation passed: python -m ruff format --check ., python -m ruff check ., and python -m pytest all completed successfully.

Summary

PR #38 clears stale vote submission state during reset_vote, specifically log_warning, submission_status, and submission_message, and adds a regression test for hiding the stale error banner after reset.

Issues and risks

No blocking issues found. The change in arena/state/voting.py matches the stated intent and preserves the existing guards for missing, submitted, or not-ready rounds.

Test coverage

The new unit test covers the failed-submission reset path and asserts both state clearing and hidden status UI. Existing integration tests already cover failed vote submission and retry behavior.

Local scoped checks passed:

  • python -m ruff format --check arena tests
  • python -m ruff check --no-cache arena tests
  • git diff --check ...

I could not run local pytest fully in this sandbox because there is no writable temp directory. The provided review context reports the full project validation passed: ruff format --check ., ruff check ., and pytest.

Recommendation

Looks safe to continue review.


This is an automated review. Please treat it as advisory; human review is still required.

@laceyp99
laceyp99 merged commit 9dc020a into main Aug 6, 2026
5 checks passed
@laceyp99
laceyp99 deleted the fix/reset-vote-submission-state branch August 6, 2026 04:06
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