Sxema va maxfiylik testlari — AC11/AC18/AC52

team-lead-ep-09 yangiladi

Yangilandi: pr_url

team-lead-ep-09 yangiladi

Yangilandi: estimate_hours

qa-ep-09-t10-s02 izoh qoldirdi

QA tekshiruvi — done. 1) `mvn -Dtest=SchemaAndPrivacyIntegrationTest test` (flock ostida) — 5/5 yashil (surefire-reports: Tests run: 5, Failures: 0, Errors: 0). 2) AC52(a) tenant_id yo'qligi — `information_schema.columns` bo'ylab qat'iy tekshiruv. Mutatsiya tekshiruvi: repo kodini o'zgartirmasdan, alohida vaqtinchalik Postgres konteynerda (qa-mutation-sim, keyin o'chirildi) `permission_groups`ga `tenant_id` va `users`ga `organization_id` qo'shib xuddi shu SQL so'rovni yurgizdim — ikkalasi ham jadval nomini qaytardi, ya'ni test aynan shu mutatsiyani ushlagan bo'lardi (isEmpty()/doesNotContain() yiqiladi). 3) AC52(b/c) — V3/V5 migratsiyalarida haqiqiy FK borligi (`fk_permission_group_items_permission`, `fk_user_permissions_permission` -> permissions.code) va V7'da `rule_code`/`object_id`da FK yo'qligi kod darajasida tasdiqlandi; test information_schema.table_constraints orqali xuddi shuni tekshiradi. 4) AC11 — 09_auth §2.3 jadvalidagi 9 ta tenant-umumiy entiti (users, user_sessions, permission_groups, permission_group_items, user_permissions, user_permission_groups, auth_logs, organization_settings, ip_login_locks) qattiq (hard) tekshiriladi — bu ro'yxat spec §2.3 jadvali bilan so'zma-so'z bir xil. Boshqa 5 modulning 15 jadvali (§2.1.6c) hozircha bu repoda YO'Q (V1-V7 migratsiyalarini tekshirdim — faqat 09_auth jadvallari bor), shuning uchun forward-compatible shart (`if existingTables.contains(table)`) qo'yilgan. Bu qaror OQILONA: (a) asosiy 9 jadval uchun tekshiruv qattiq va mutatsiyani ushlaydi (yuqoridagi simulyatsiyada tasdiqlandi — `users`ga organization_id qo'shilganda darhol aniqlandi); (b) boshqa modullar migratsiyaga qo'shilganda XUDDI SHU testni o'zgartirmasdan avtomatik ularni ham qamraydi (dinamik table-existence tekshiruvi); (c) hozir mavjud bo'lmagan jadval uchun qattiq test yozib bo'lmaydi. Bu "hech narsa tekshirmay yashil" emas — asosiy qismi haqiqiy tekshiruv. 5) AC18 — to'liq ssenariy (xato parol, mavjud bo'lmagan login, audience mismatch, partner web niqobi, muvaffaqiyatli login, refresh rotation, logout, reuse detection) yurgizilib, `auth_logs`ning HAR qatori (`row_to_json` — details jsonb bilan birga) haqiqiy qiymatlar (parol, ikkala password_hash, ikkala access token, ikkala refresh token, barcha refresh_token_hash) bilan solishtiriladi + generic BCrypt/JWT regex skaneri. Regex sifatini alohida (python) tekshirdim: haqiqiy shakldagi BCrypt xesh va JWT satrlarini to'g'ri ushlaydi, lekin oddiy UUID (masalan request_id)ni FALSE POSITIVE qilib belgilamaydi — demak muvozanatli. Qo'shimcha: `AuditServiceImpl.rejectSecretsIn` fail-loud himoyasi ham bor (yozish vaqtida taqiqlangan kalit so'zlarni rad etadi) — ikki qatlamli himoya. Hammasi spec (09_auth.md §12 AC11/AC18/AC52, §2.1.6c, §2.3) bilan mos. Kod repo'siga hech narsa o'zgartirilmadi (faqat vaqtinchalik alohida Docker konteynerda simulyatsiya, keyin tozalandi).

qa-ep-09-t10-s02 statusni o'zgartirdi Review → Bajarildi

qa-ep-09-t10-s02 qabul mezonini belgiladi

✓ To'liq auth ssenariysidan keyin `auth_logs` da hech qanday sir qiymat topilmaydi (regexp skaneri) — AC18

qa-ep-09-t10-s02 qabul mezonini belgiladi

✓ Tenant-umumiy jadvallarda `organization_id` ustuni 0 ta — AC11

qa-ep-09-t10-s02 qabul mezonini belgiladi

✓ Ikki `permission_code` ustunida FK bor, `auth_logs.rule_code`/`object_id` da yo'q — AC52(b,c)

qa-ep-09-t10-s02 qabul mezonini belgiladi

✓ `information_schema` bo'ylab `tenant_id` ustuni 0 ta — AC52(a)

coder-ep-09-t10-s02 statusni o'zgartirdi Ishlanyapti → Review

coder-ep-09-t10-s02 yangiladi

Yangilandi: branch, commit_sha

coder-ep-09-t10-s02 izoh qoldirdi

SchemaAndPrivacyIntegrationTest yozildi (5 test, hammasi yashil): tenant_id yo'qligi, permission_code FK bor, auth_logs.rule_code/object_id FK yo'q, tenant-umumiy jadvallarda organization_id yo'q, va to'liq auth ssenariysi (login/xato parol/audience mismatch/refresh rotation/reuse/logout) dan keyin auth_logs regexp skaneri (bcrypt+JWT naqshlari va aniq parol/hash/token qiymatlari) toza. AC11/2.1.6(c) uchun boshqa 5 modulning 15 jadvali hali kod bosqichida yo'q (faqat 09_auth migratsiyalari mavjud) - test ularni faqat mavjud bo'lsa tekshiradi (forward-compat), hozircha 09_auth ning o'ziga tegishli 9 tenant-umumiy jadval qattiq tekshiriladi.

coder-ep-09-t10-s02 qabul mezonini belgiladi

✓ To'liq auth ssenariysidan keyin `auth_logs` da hech qanday sir qiymat topilmaydi (regexp skaneri) — AC18

coder-ep-09-t10-s02 qabul mezonini belgiladi

✓ Tenant-umumiy jadvallarda `organization_id` ustuni 0 ta — AC11

coder-ep-09-t10-s02 qabul mezonini belgiladi

✓ Ikki `permission_code` ustunida FK bor, `auth_logs.rule_code`/`object_id` da yo'q — AC52(b,c)

coder-ep-09-t10-s02 qabul mezonini belgiladi

✓ `information_schema` bo'ylab `tenant_id` ustuni 0 ta — AC52(a)

coder-ep-09-t10-s02 fayllarni qayd etdi

1 ta fayl qayd etildi

coder-ep-09-t10-s02 statusni o'zgartirdi Backlog → Ishlanyapti

Agent ishni boshladi

coder-ep-09-t10-s02 ishga oldi

pm-agent yaratdi

Sxema va maxfiylik testlari — AC11/AC18/AC52