feat(capa): allow overdue CAPAs to be acted on, fix owner page stats
- CapaOwnerActions: add 'overdue' to open/reopened branch so DB-overdue CAPAs show 'Mark In Progress' button (cron transitions open→overdue) - capa-owner page: compute overdue at runtime (dueDate < today) so stats and red highlight are correct even before cron runs - openCount now excludes overdue items to avoid double-counting - Add CapaOwnerActions test suite covering all status transitions Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HEYxFQiCyxJvnBCoZeYzB9
This commit is contained in:
@@ -52,8 +52,14 @@ export default async function CapaOwnerPage() {
|
||||
.orderBy(asc(capaActions.dueDate))
|
||||
)
|
||||
|
||||
const openCount = rows.filter(c => ['open', 'in_progress', 'reopened'].includes(c.status)).length
|
||||
const overdueCount = rows.filter(c => c.status === 'overdue').length
|
||||
const today = new Date().toISOString().slice(0, 10)
|
||||
function isCapaOverdue(c: { status: string; dueDate: string | null }): boolean {
|
||||
if (c.status === 'overdue') return true
|
||||
return (c.status === 'open' || c.status === 'in_progress') && !!c.dueDate && c.dueDate < today
|
||||
}
|
||||
|
||||
const openCount = rows.filter(c => ['open', 'in_progress', 'reopened'].includes(c.status) && !isCapaOverdue(c)).length
|
||||
const overdueCount = rows.filter(c => isCapaOverdue(c)).length
|
||||
const doneCount = rows.filter(c => ['verified', 'closed'].includes(c.status)).length
|
||||
|
||||
const PRIORITY_COLORS: Record<string, string> = {
|
||||
@@ -82,7 +88,7 @@ export default async function CapaOwnerPage() {
|
||||
) : (
|
||||
<div className="bg-white rounded-xl shadow-sm divide-y divide-gray-50">
|
||||
{activeRows.map(capa => {
|
||||
const isOverdue = capa.status === 'overdue'
|
||||
const isOverdue = isCapaOverdue(capa)
|
||||
return (
|
||||
<div key={capa.id} className={`p-4 ${isOverdue ? 'bg-red-50' : ''}`}>
|
||||
<div className="flex items-start justify-between gap-4">
|
||||
|
||||
@@ -31,7 +31,7 @@ export function CapaOwnerActions({ capaId, currentStatus }: Props) {
|
||||
}
|
||||
}
|
||||
|
||||
if (currentStatus === 'open' || currentStatus === 'reopened') {
|
||||
if (currentStatus === 'open' || currentStatus === 'reopened' || currentStatus === 'overdue') {
|
||||
return (
|
||||
<button
|
||||
onClick={() => updateStatus('in_progress')}
|
||||
|
||||
@@ -0,0 +1,40 @@
|
||||
import { describe, it, expect, vi, beforeEach } from 'vitest'
|
||||
import { render, screen } from '@testing-library/react'
|
||||
import { CapaOwnerActions } from '@/components/capa/capa-owner-actions'
|
||||
|
||||
// Mock Next.js router
|
||||
vi.mock('next/navigation', () => ({
|
||||
useRouter: () => ({ refresh: vi.fn() }),
|
||||
}))
|
||||
|
||||
describe('CapaOwnerActions', () => {
|
||||
it('shows Mark In Progress for open status', () => {
|
||||
render(<CapaOwnerActions capaId="abc" currentStatus="open" />)
|
||||
expect(screen.getByText('Mark In Progress')).toBeTruthy()
|
||||
})
|
||||
|
||||
it('shows Mark In Progress for reopened status', () => {
|
||||
render(<CapaOwnerActions capaId="abc" currentStatus="reopened" />)
|
||||
expect(screen.getByText('Mark In Progress')).toBeTruthy()
|
||||
})
|
||||
|
||||
it('shows Mark In Progress for overdue status', () => {
|
||||
render(<CapaOwnerActions capaId="abc" currentStatus="overdue" />)
|
||||
expect(screen.getByText('Mark In Progress')).toBeTruthy()
|
||||
})
|
||||
|
||||
it('shows Submit for Verification for in_progress status', () => {
|
||||
render(<CapaOwnerActions capaId="abc" currentStatus="in_progress" />)
|
||||
expect(screen.getByText('Submit for Verification')).toBeTruthy()
|
||||
})
|
||||
|
||||
it('renders nothing for pending_verification status', () => {
|
||||
const { container } = render(<CapaOwnerActions capaId="abc" currentStatus="pending_verification" />)
|
||||
expect(container.firstChild).toBeNull()
|
||||
})
|
||||
|
||||
it('renders nothing for verified status', () => {
|
||||
const { container } = render(<CapaOwnerActions capaId="abc" currentStatus="verified" />)
|
||||
expect(container.firstChild).toBeNull()
|
||||
})
|
||||
})
|
||||
Reference in New Issue
Block a user