fix(dashboard): Register error prop pasa '' a MUI TextField (card 9151895a) - #218
fix(dashboard): Register error prop pasa '' a MUI TextField (card 9151895a)#218SabaTech-dev wants to merge 1 commit into
Conversation
…1895a)
error={confirmPassword && ...} evalua a '' (string) cuando confirmPassword
esta vacio; MUI FormControl/TextField espera boolean y emite
'Invalid prop `error` of type `string`' en desarrollo. Envuelto en
Boolean(...). Anadidos tests vitest+RTL: captura del propTypes warning
via spy de console.error (printf-style, texto en args secundarios),
mismatch muestra helper y match no muestra error.
Card: 9151895a-5745-4a0d-af3b-f439a564db9e
PR Check Results✅ Code Formatting: Passed |
|
| Severity | Count |
|---|---|
| 🔴 CRITICAL | 0 |
| 🟠 HIGH | 47 |
| Total | 47 |
Scans performed:
- 📁 Filesystem (vulnerabilities, secrets, misconfigurations)
- 🐳 Backend Docker image
- 🐳 Frontend Docker image
- 🏗️ IaC misconfiguration detection
📋 View detailed SARIF reports
Download artifacts from this workflow run for full SARIF reports.
Results are also available in the GitHub Security tab.
Trivy v0.71.0 | aquasecurity/trivy
🎨 Frontend Preview DeploymentApp Coolify: qa-framework-frontend-pr-218 ( Preview interna (sin FQDN público hasta decisión de proxy — card 52a85645). |
🔧 Backend Preview DeploymentApp Coolify: qa-framework-backend-pr-218 ( Preview interna (sin FQDN público hasta decisión de proxy — card 52a85645). |
1 participant
Card
Workboard card
9151895a-5745-4a0d-af3b-f439a564db9e(flujo corto QA-FW). Bug preexistente en main, detectado por el QA gate del PR #212. Este PR NO toca la branch del PR #212 (feat/frontend-major-upgrades): sale deorigin/main.Problema
Register.tsx:178pasabaerror={formData.confirmPassword && formData.password !== formData.confirmPassword}a MUI TextField/FormControl. ConconfirmPasswordvacio la expresion&&evalua a''(string), no afalse— pitfall clasico de JSX. MUI esperabooleany emite en desarrollo:(El warning se reproduce tanto en MUI 5 de main como se agrava con la subida a MUI 9 del PR #212.)
Fix
Envuelto en
Boolean(...)— unica ocurrencia del patron en el archivo (verificado con grep sobre todo el frontend).Tests (TDD: RED -> GREEN)
Nuevo
Register.test.tsx(vitest + RTL, 3 tests):console.error— ojo: React 18 emite el warning printf-style ('Warning: Failed %s type: %s%s'), el texto util va en argumentos secundarios, y React deduplica estos warnings por componente, por lo que el test renderiza primero.Verificado RED (test 1 falla con el bug presente) antes de aplicar el fix; GREEN despues.
Verificacion
npx vitest run— 23/23 (20 previos + 3 nuevos; el "24/24" de la card corresponde a la branch del PR feat(frontend): major upgrades — zustand 5, react-router 7, MUI 9, TS 7 #212, que anade 1 test propio)npm run build— OKerror={... && ...}: solo 1 ocurrencia en todo el frontend