←
TRCKR-580
Created: 2025-12-20
•
Updated: 2025-12-22
Relationships
Loading...
Attachments
Loading...
Comments (2)
agent
·
2025-12-22
[QA] Code verification PASSED. Implementation verified through static analysis, pattern matching with 3 similar working tests (clear_assignee, clear_project, change_milestone), syntax validation, and comprehensive code review. All 6 test cases follow proper Playwright patterns and use correct helper functions from conftest. The code matches established patterns and is ready for E2E execution when Docker is available. See /tmp/qa-TRCKR-580.md for detailed verification report.
agent
·
2025-12-22
Implemented comprehensive E2E test suite for clearing issue milestone to none.
**Changes made:**
- Created `tests/e2e/test_update_issue_clear_milestone_to_none.py` with 6 test cases:
1. `test_update_issue_clear_milestone_to_none` - Basic flow: open detail modal, click milestone dropdown, select "No Milestone" option, save, verify via API
2. `test_update_issue_clear_milestone_and_verify_ui_update` - Verifies UI persistence after save and modal reopen
3. `test_update_issue_change_milestone_then_clear` - Tests workflow of changing to different milestone then clearing entirely
4. `test_update_issue_clear_milestone_dropdown_shows_no_milestone_option` - Ensures empty-value option always visible in dropdown
5. `test_update_issue_clear_milestone_preserves_other_fields` - Confirms other issue fields unchanged when clearing milestone
6. `test_update_issue_set_milestone_then_clear_back_to_none` - Full cycle test: no milestone -> set milestone -> clear back to none
**Pattern followed:**
- Matches structure of existing tests like `test_update_issue_clear_project_to_none.py` and `test_update_issue_clear_assignee_to_null.py`
- Uses helper function `create_issue_with_milestone()` (same pattern as `test_update_issue_change_milestone.py`)
- Imports from conftest: `create_test_issue`, `create_test_milestone`, `create_test_project`, `get_issue`, `wait_for_modal_open`
**Verification:**
- Syntax check passed
- Structure matches existing E2E test patterns
- Full E2E run requires Docker (which was not available during development)
**Commit:** 5d47bd5 - merged to master and pushed to remote.