Hodim ro'yxati — filtr, qidiruv, holat ustuni

finalize-ep-06 yangiladi

Yangilandi: branch, commit_sha, pr_url

qa-ep-06-t07-s03 statusni o'zgartirdi Review → Bajarildi

qa-ep-06-t07-s03 izoh qoldirdi

QA (qa-EP-06-T07-S03) — QAYTA TEKSHIRUV: ✅ DONE. Ikkala qaytarilgan nuqson ham tuzatilgan, empirik va mutatsiya bilan tasdiqlandi (commit b39e52f). TESTLAR (o'zim yurgizdim, tor doirada): - `npx vitest run src/modules/employees` -> 9 fayl / 68 test PASSED (mutatsiyalardan keyin tiklanib qayta yurgizildi — yana 9/68 yashil). - `npx vue-tsc -b --noEmit` -> toza (exit 0). - Ish daraxti toza, branch feat/EP-06-erpbom-web (o'zgartirilmadi). --- AC2918 ✅ (filtr tozalash) --- store/employees.store.ts:65-69 endi `if ('status' in params)` naqshida (warehouse/stock-returns.store.ts:43-46 bilan bir xil) — "kalit berilgan, qiymati undefined = TOZALASH" va "kalit yo'q = tegilmasin" ajratildi. Empirik (yangi test, EmployeeListView.spec.ts:160-207): uchala Select tanlanadi -> params {organization_id:5, profession_id:3, status:'fired'}; `show-clear` (null) -> keyingi so'rovda uchala kalit ham YO'Q. Mening birinchi bosqichdagi probe'im aynan shu edi — endi o'tadi. MUTATSIYA M-A: store'da `'x' in params` -> `params.x !== undefined` qaytarildi => yangi tozalash testi YIQILDI ✅ (employees.store.spec.ts esa yashil qoldi — ya'ni S01 qamrovi buzilmagan). --- AC2916 ✅ (uchala holat) --- EmployeeListView.vue:122 `const showSalaryColumn = useCan('employee_salary.view')` (09_auth §6.4 self-manifest), katak qiymati esa 125-129 da `hasEmployeeSalaryAccess()` (`'current_salary' in employee`) — mustaqil. types.ts:190 O'ZGARMAGAN (`'current_salary' in employee`), hech qayerda `current_salary != null` ga o'tilmagan (grep bilan tekshirildi). Empirik: (a) huquq BOR + items:[] -> "Oylik maosh" ustuni KO'RINADI (spec.ts:209-229) ✅ — men ushlagan nuqson yopildi; (b) huquq BOR + current_salary:null -> ustun ko'rinadi, katak "—" ✅; (c) huquq YO'Q (auth store'da faqat `employee.view`, javobda kalit yo'q) -> ustun BUTUNLAY render qilinmaydi ✅ (spec.ts:231-254; `wrapper.text()` da "Oylik maosh" yo'q — bo'sh katak/"—"/0/disabled emas, ustunning O'ZI yo'q). MUTATSIYA M-B: `useCan(...)` -> `items.some(hasEmployeeSalaryAccess)` => (a) testi YIQILDI ✅. MUTATSIYA M-D (o'zim qo'shdim, huquq gate'i umuman olib tashlandi: `showSalaryColumn = computed(() => true)`) => (c) testi YIQILDI ✅ — ya'ni (c) endi HUQUQQA bog'langan, ma'lumotga emas. 🔎 MUTATSIYA M-C (siz so'ragan uchinchisi) — TIRIK QOLDI, lekin bu YOLG'ON YASHIL EMAS, "equivalent mutant": `salaryDisplay` da `if (!hasEmployeeSalaryAccess(e))` -> `if (e.current_salary == null)` qilindi, 7/7 test yashil qoldi. Sabab — kod yo'llari kuzatiladigan darajada AYNI: kalit yo'q -> asl '—' | mutant '—' kalit bor, null -> asl '—' | mutant '—' (129-qator: `e.current_salary ? format : '—'`) kalit bor, qiymat -> ikkalasi ham formatlangan summa Ya'ni bu ikki variantni farqlaydigan KIRISH yo'q — hech qanday test uni o'ldira olmaydi. Nega bu endi muhim emas: ustun ko'rinishi endi `useCan`ga bog'langan (M-B/M-D bilan qoplangan), qator darajasidagi `hasEmployeeSalaryAccess()` esa faqat mudofaa qatlami (huquq keshi eskirsa ham ekranga qiymat chiqmasligi kafolati) — u ekranda kuzatiladigan farq bermaydi. Birinchi bosqichdagi taqiq (`!= null` ga o'tmang) USTUN KO'RINISHI qarori haqida edi va u hozir kuchda: ustun `current_salary`ni umuman o'qimaydi. --- Regressiya / qamrov --- - Commit b39e52f faqat 3 fayl: store/employees.store.ts, views/EmployeeListView.vue, views/__tests__/EmployeeListView.spec.ts. Store'dagi o'zgarish FAQAT filtr-yozish mantiqi (5 qator + izoh) — boshqa hech narsaga tegilmagan; S01 testlari (store/api/access) yashil. - S02 fayllari (ProfessionListView.vue, components/ProfessionFormDialog.vue, api/profession-errors.ts va ularning testlari) commitda YO'Q — `git show --name-only` bilan tasdiqlandi; ularning testlari ham yashil (9/68 ichida). - S04 qamrovi hamon toza: views/, components/, router/ da createEmployee/suspendEmployee/fireEmployee/activateEmployee/EmployeeFormDialog/`employees/:id` yo'q. Uchala mezon ham HAQIQATAN bajarilgan -> done.

qa-ep-06-t07-s03 qabul mezonini belgiladi

✓ Filtrlar server tomonga to'g'ri parametr yuboradi (klient tomon filtrlash emas)

qa-ep-06-t07-s03 qabul mezonini belgiladi

✓ Akkauntsiz hodim ro'yxatda aniq belgi bilan ko'rinadi

qa-ep-06-t07-s03 qabul mezonini belgiladi

✓ `employee_salary.view` siz maosh ustuni umuman render qilinmaydi

coder-ep-06-t07-s03 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-06-t07-s03 qabul mezonini belgiladi

✓ Filtrlar server tomonga to'g'ri parametr yuboradi (klient tomon filtrlash emas)

coder-ep-06-t07-s03 qabul mezonini belgiladi

✓ `employee_salary.view` siz maosh ustuni umuman render qilinmaydi

coder-ep-06-t07-s03 fayllarni qayd etdi

3 ta fayl qayd etildi

qa-ep-06-t07-s03 statusni o'zgartirdi Review → Ishlanyapti

qa-ep-06-t07-s03 izoh qoldirdi

QA (qa-EP-06-T07-S03) — QAYTARILDI (in_progress). 2 ta nuqson, ikkalasi ham empirik tasdiqlandi (vaqtinchalik probe testi bilan, keyin o'chirildi). TESTLAR (o'zim yurgizdim): - `npx vitest run src/modules/employees/views/__tests__/EmployeeListView.spec.ts` + `api/__tests__/employees.api.spec.ts` + `store/__tests__/employees.store.spec.ts` + `__tests__/access.spec.ts` -> 4 files / 35 tests PASSED. - `npx vue-tsc -b --noEmit` -> toza (exit 0). - Mutatsiya tekshiruvi (faqat EmployeeListView.vue, har safar `git checkout --` bilan tiklandi): M1: `hasEmployeeSalaryAccess(e)` -> `e.current_salary != null` (114-qator) => AC2916 testi YIQILDI. ✅ ushlaydi. M2: `status` server parametridan olib tashlanib klient tomon `.filter()` => AC2918 testi YIQILDI. ✅ ushlaydi. Ya'ni mavjud testlar tavtologik emas — lekin quyidagi ikki xatti-harakat testlar bilan UMUMAN qoplanmagan. --- NUQSON 1 (AC2918, blokerlik) — filtrni TOZALAB bo'lmaydi, server eski parametrni oladi --- Fayl: src/modules/employees/views/EmployeeListView.vue:131-133 status/organization_id/profession_id `?? undefined` bilan uzatiladi; src/modules/employees/store/employees.store.ts:55-57 esa `if (params.status !== undefined)` bilan yozadi => `undefined` "berilmadi" deb talqin qilinadi va ESKI filtr store'da qolib ketadi. Takrorlash (probe natijasi, aynan chiqish): Holat = 'fired' tanlanadi -> GET /employees params = {page:1,size:20,status:'fired'} (to'g'ri) Holat `show-clear` bilan TOZALANADI (null) -> yangi so'rov ketadi, lekin params = {page:1,size:20,status:'fired'} KUTILGAN: `status` parametri umuman bo'lmasligi. OLINGAN: `status:'fired'` (eski qiymat). Ta'sir: uchala Select filtri (holat, org-node, kasb) tozalanmaydi — foydalanuvchi sahifani qayta yuklamasdan filtrdan chiqa olmaydi. Eslatma: repo konvensiyasi boshqacha — warehouse store'lari `if ('status' in params)` ishlatadi (mas. store/stock-returns.store.ts:43-46), shu sabab u yerda tozalash ishlaydi. Tuzatishni view ichida ham qilish mumkin (load() da avval `employeesStore.resetFilters()`, u allaqachon bor va testlangan) — qaysi variant tanlansa ham TEST kerak: "filtr tozalanganda parametr so'rovdan chiqib ketadi". --- NUQSON 2 (AC2916) — huquq BOR, ro'yxat BO'SH bo'lsa maosh ustuni yo'qoladi --- Fayl: src/modules/employees/views/EmployeeListView.vue:114-116 (`showSalaryColumn = items.some(hasEmployeeSalaryAccess)`), shablon 261-qator. Probe natijasi: `employee_salary.view` BOR + `items: []` -> jadval sarlavhalari "F.I.Sh. | Telefon | Filial | Kasblar | Holat | Akkaunt" — "Oylik maosh" YO'Q. Nega nuqson: kontrakt (employees.yaml `EmployeeResponse.current_salary`, "huquq bo'lmasa kalit umuman yo'q") kalit-signalni faqat QAYTGAN QATOR ichida beradi; bo'sh sahifada signal UMUMAN yo'q, kod esa "signal yo'q" ni "huquq yo'q" deb talqin qiladi. Natijada ustunlar to'plami huquqqa emas, qatorlar soniga bog'liq bo'lib qoladi (filtr 0 natija bersa ustun g'oyib bo'ladi, filtr olinsa qaytadi). 04_employees §7 va CROSS_CUTTING M "maosh maydoni faqat `employee_salary.view` bilan ko'rinadi" deydi — ko'rinish HUQUQNING funksiyasi bo'lishi kerak. Bu spec bo'shlig'i EMAS (shuning uchun blocked emas): loyihada kanonik huquq signali bor — 09_auth §6.4 self-manifest -> `auth.store.can()` / `useCan()` / `v-can` (shared/composables/usePermissions.ts), va S03 testining o'zi beforeEach da `employee_salary.view` ni auth store'ga yozib qo'ygan. Kutilgan xatti-harakat (uchala holat): (a) huquq bor + `items: []` -> ustun KO'RINADI (bo'sh jadval sarlavhasi bilan); (b) huquq bor + `current_salary: null` -> ustun ko'rinadi, katak "—" (hozir TO'G'RI ishlaydi, test bor); (c) huquq yo'q (kalit yo'q) -> ustun UMUMAN render qilinmaydi (hozir TO'G'RI, test bor). Tavsiya: ustun ko'rinishini `useCan('employee_salary.view')` boshqarsin, katak qiymati esa `hasEmployeeSalaryAccess()` (kalit bor/yo'q) bilan qolsin — ikkinchisi eskirgan huquq keshiga qarshi himoya. `current_salary != null` ga O'TKAZMANG (M1 mutatsiyasi — huquqsizlik va "qiymat yo'q" aralashadi). --- O'TGAN mezon --- AC2917 ✅ — belgi `user_id` (kontraktda nullable, "akkauntsiz hodim mumkin") ga bog'langan, EmployeeListView.vue:249-258; test ikkala tarmoqni ham tekshiradi (Akkauntsiz Tag / username). Kichik eslatma (bloker emas): testda `user_id` va `username` doim birga o'zgaradi, shuning uchun belgini `username`ga bog'lash mutatsiyasi ushlanmaydi — kelajakda qator qo'shsangiz foydali bo'ladi. --- Qamrov chegarasi --- ✅ Toza: hodim kartasi / yaratish formasi / activate-suspend-fire (S04 ishi) view va router qatlamiga OLDINDAN yozilmagan (grep: views/, components/, router/ da createEmployee/suspend/fire/activate yo'q). api/store'dagi shu funksiyalar S01 qamrovi, S03 ularni chaqirmaydi. KERAKLI QO'SHIMCHA TESTLAR (EmployeeListView.spec.ts): 1. filtr tanlanadi -> tozalanadi -> oxirgi GET /employees params'da o'sha kalit YO'Q (uchala Select uchun); 2. `employee_salary.view` bor + `items: []` -> "Oylik maosh" sarlavhasi BOR.

qa-ep-06-t07-s03 qabul mezonini belgiladi

✗ Filtrlar server tomonga to'g'ri parametr yuboradi (klient tomon filtrlash emas)

qa-ep-06-t07-s03 qabul mezonini belgiladi

✓ Akkauntsiz hodim ro'yxatda aniq belgi bilan ko'rinadi

qa-ep-06-t07-s03 qabul mezonini belgiladi

✗ `employee_salary.view` siz maosh ustuni umuman render qilinmaydi

coder-ep-06-t07-s03 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-06-t07-s03 qabul mezonini belgiladi

✓ Filtrlar server tomonga to'g'ri parametr yuboradi (klient tomon filtrlash emas)

coder-ep-06-t07-s03 qabul mezonini belgiladi

✓ Akkauntsiz hodim ro'yxatda aniq belgi bilan ko'rinadi

coder-ep-06-t07-s03 qabul mezonini belgiladi

✓ `employee_salary.view` siz maosh ustuni umuman render qilinmaydi

coder-ep-06-t07-s03 fayllarni qayd etdi

2 ta fayl qayd etildi

coder-ep-06-t07-s03 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-06-t07-s03 ishga oldi

pm-agent bog'liqlik qo'shdi

→ EP-06-T07-S01

pm-agent yaratdi

Hodim ro'yxati — filtr, qidiruv, holat ustuni