Skip to content

Commit 3c7e115

Browse files
committed
Fix Data-Manipulation 422 bug; update data processing route to correctly parse JSON options and enhance test coverage for validation
1 parent 2005e8c commit 3c7e115

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

docs/notes/ROLLING_TASK_PLAN.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -746,6 +746,16 @@ Not:
746746
- Toast testi fake timers + `act()` ile sarmalandi (timer-driven state update warning'leri giderildi).
747747
- [x] P2: `docs/technical/ROUTE_KPI_CHECKLIST.md` eklendi — `creator-studio`, `analysis-history`, `system-status` icin entry source, first action, revisit indicator, raporlama frekansi dokumante edildi.
748748

749+
### Data-Manipulation 422 Bug Fix (P0 Hotfix)
750+
751+
- [x] P0: `hf-crowncode-backend/app/routes/data_processing.py``options: Json[AudioAugmentationOptions] = Form(...)``options: str = Form(...)` + `AudioAugmentationOptions.model_validate_json(options)` ile manual parse. Frontend'in `JSON.stringify()` ile gonderdig JSON string artik dogru parse ediliyor. Hatali JSON icin kontrollü `422 invalid_options` donuyor.
752+
- [x] P0: `hf-crowncode-backend/tests/test_data_processing.py` tamamen yeniden yazildi:
753+
- Gevşek `assert status_code in (400, 422)` kaldırıldı — her test kesin status code bekliyor.
754+
- Valid multipart request testi eklendi (422 olmadigi dogrulaniyor).
755+
- Invalid JSON options testi eklendi (`invalid_options` error code assertion).
756+
- camelCase options kabul testi eklendi (frontend uyumu).
757+
- Helper fonksiyonlari (`_valid_options`, `_fake_audio`) ile test DRY hale getirildi.
758+
749759
### Faz Q Dogrulama Logu (Analist)
750760

751761
- [x] `cmd /c npm --prefix platform run lint` -> passed

0 commit comments

Comments
 (0)