Skip to content

test(frontend): behavior tests for components migrated in #212 (waiver compensation) - #225

Merged
SabaTech-dev merged 1 commit into
mainfrom
test/migrated-components-coverage
Sep 4, 2026
Merged

test(frontend): behavior tests for components migrated in #212 (waiver compensation)#225
SabaTech-dev merged 1 commit into
mainfrom
test/migrated-components-coverage

Conversation

@SabaTech-dev

Copy link
Copy Markdown
Owner

Compensación del waiver de diff-coverage — PR #212 (card 17ca16b0)

Tests de comportamiento (Vitest + Testing Library) para los componentes de mayor lógica/riesgo migrados por el codemod MUI 9 / react-router 7 / zustand 5, según diff-coverage.txt del QA gate.

Gate ≥70% diff-cov (medido con diffcov.py, base 7dadb7d^)

Componente diff-cov
billing/InvoiceList 100% (41/41)
billing/PaymentMethodForm 100% (57/57)
billing/PlanCard 100% (37/37)
billing/SubscriptionStatus 100% (97/97)
common/KeyboardShortcutsDialog 100% (35/35)
OnboardingWizard 95% (296/313)
pages/DashboardEnhanced 85% (184/217)
filters/AdvancedFilter 73% (62/85)

Global diff-cov: 52.3% (1467/2805) vs 21% al momento del waiver.

Verificación local

  • npx vitest run83/83 pass (11 files), cero flakes
  • npx tsc --noEmit → clean
  • npm run lint → falla preexistente repo-wide: typescript-eslint no soporta TS 7.0 (upstream typescript-eslint#10940); no introducido por este PR

Notas

…17ca16b0)

Compensación del waiver de diff-coverage de PR #212: tests de
comportamiento (Vitest + Testing Library) para los componentes de mayor
lógica/riesgo migrados por el codemod MUI 9/RR7/zustand 5.

- billing/*: InvoiceList, PaymentMethodForm, PlanCard, SubscriptionStatus
- pages: DashboardEnhanced, OnboardingWizard
- components: AdvancedFilter, KeyboardShortcutsDialog

Suite: 83/83 (11 files). Diff-cov vs 7dadb7d^: billing 100% x4,
KeyboardShortcutsDialog 100%, OnboardingWizard 95%, DashboardEnhanced 85%,
AdvancedFilter 73% (gate >=70%). Global 52.3% (era 21% al momento del
waiver).
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

PR Check Results

Code Formatting: Passed
Linting: Passed
Type Checking: Failed
Tests: Failed

⚠️ Some checks failed. Please review the details above.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

⚠️ WARN Trivy Security Scan Results

Severity Count
🔴 CRITICAL 0
🟠 HIGH 46
Total 46

Scans performed:

  • 📁 Filesystem (vulnerabilities, secrets, misconfigurations)
  • 🐳 Backend Docker image
  • 🐳 Frontend Docker image
  • 🏗️ IaC misconfiguration detection

⚠️ HIGH vulnerabilities should be addressed. Review required.

📋 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

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

🎨 Frontend Preview Deployment

App Coolify: qa-framework-frontend-pr-225 (tdr25q1ciu7ki5dcqydk2sik)
Estado: ⚠️ running:unknown
Commit: 5433d53635f0a095c25afe6041046eaba6ab3383
Branch: test/migrated-components-coverage

Preview interna (sin FQDN público hasta decisión de proxy — card 52a85645).

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

🔧 Backend Preview Deployment

App Coolify: qa-framework-backend-pr-225 (uh48adkmosg6cgqgljospznu)
Estado: ⚠️ running:unknown
Commit: 5433d53635f0a095c25afe6041046eaba6ab3383
Branch: test/migrated-components-coverage

Preview interna (sin FQDN público hasta decisión de proxy — card 52a85645).

@SabaTech-dev
SabaTech-dev merged commit 4a015ae into main Sep 4, 2026
41 checks passed
@SabaTech-dev
SabaTech-dev deleted the test/migrated-components-coverage branch September 4, 2026 20:36
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

🧹 Preview Cleanup

PR #225 - Merged

✅ Preview apps eliminadas de Coolify (verificado via API): backend + frontend.


Resources freed at 2026-09-04T20:37:28.760Z

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant