Phase 6.5: Tag-Forms on RHF + Zod
- TagPicker create-tag form: RHF+Zod (name required, color optional with default) - Error display under name field - Color picker uses setTagValue from RHF - Preserved all existing functionality: search, assign, unassign - Added 2 validation tests (empty name, valid submit)
This commit is contained in:
@@ -0,0 +1,66 @@
|
|||||||
|
import React from 'react';
|
||||||
|
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
||||||
|
import { render, screen, fireEvent, waitFor } from '@testing-library/react';
|
||||||
|
|
||||||
|
vi.mock('@/api/tags', () => ({
|
||||||
|
fetchTags: vi.fn().mockResolvedValue([]),
|
||||||
|
assignTag: vi.fn().mockResolvedValue({}),
|
||||||
|
unassignTag: vi.fn().mockResolvedValue({}),
|
||||||
|
createTag: vi.fn().mockResolvedValue({ id: 't1', name: 'NewTag', color: '#3B82F6' }),
|
||||||
|
}));
|
||||||
|
|
||||||
|
vi.mock('@/components/ui/Toast', () => ({
|
||||||
|
useToast: () => ({ success: vi.fn(), error: vi.fn(), info: vi.fn(), warning: vi.fn() }),
|
||||||
|
}));
|
||||||
|
|
||||||
|
import { TagPicker } from '@/components/tags/TagPicker';
|
||||||
|
import { createTag } from '@/api/tags';
|
||||||
|
|
||||||
|
beforeEach(() => {
|
||||||
|
vi.clearAllMocks();
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('TagPicker validation (RHF + Zod)', () => {
|
||||||
|
it('shows validation error when tag name is empty', async () => {
|
||||||
|
render(<TagPicker entityType="contact" entityId="e1" />);
|
||||||
|
// Wait for loading to finish and find create button
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(screen.getByTestId('tag-picker')).toBeInTheDocument();
|
||||||
|
}, { timeout: 3000 });
|
||||||
|
// Click create button to show form (German: 'Neues Tag')
|
||||||
|
const createBtn = screen.getByText(/neues tag|create/i);
|
||||||
|
fireEvent.click(createBtn);
|
||||||
|
// Submit empty form
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(screen.getByTestId('create-tag-form')).toBeInTheDocument();
|
||||||
|
});
|
||||||
|
const form = screen.getByTestId('create-tag-form').querySelector('form');
|
||||||
|
expect(form).toBeTruthy();
|
||||||
|
fireEvent.submit(form!);
|
||||||
|
await waitFor(() => {
|
||||||
|
const errors = screen.getAllByText(/erforderlich|required/i);
|
||||||
|
expect(errors.length).toBeGreaterThan(0);
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
it('calls createTag when form is valid', async () => {
|
||||||
|
render(<TagPicker entityType="contact" entityId="e1" />);
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(screen.getByTestId('tag-picker')).toBeInTheDocument();
|
||||||
|
}, { timeout: 3000 });
|
||||||
|
const createBtn = screen.getByText(/neues tag|create/i);
|
||||||
|
fireEvent.click(createBtn);
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(screen.getByTestId('create-tag-form')).toBeInTheDocument();
|
||||||
|
});
|
||||||
|
// Fill in tag name
|
||||||
|
const form = screen.getByTestId('create-tag-form').querySelector('form');
|
||||||
|
const input = form!.querySelector('input');
|
||||||
|
expect(input).toBeTruthy();
|
||||||
|
fireEvent.change(input!, { target: { value: 'Important' } });
|
||||||
|
fireEvent.submit(form!);
|
||||||
|
await waitFor(() => {
|
||||||
|
expect(createTag).toHaveBeenCalledWith(expect.objectContaining({ name: 'Important' }));
|
||||||
|
}, { timeout: 3000 });
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -6,6 +6,9 @@ import clsx from 'clsx';
|
|||||||
|
|
||||||
import React, { useState, useEffect, useCallback } from 'react';
|
import React, { useState, useEffect, useCallback } from 'react';
|
||||||
import { useTranslation } from 'react-i18next';
|
import { useTranslation } from 'react-i18next';
|
||||||
|
import { useForm } from 'react-hook-form';
|
||||||
|
import { zodResolver } from '@hookform/resolvers/zod';
|
||||||
|
import { z } from 'zod';
|
||||||
import { Badge } from '@/components/ui/Badge';
|
import { Badge } from '@/components/ui/Badge';
|
||||||
import { Button } from '@/components/ui/Button';
|
import { Button } from '@/components/ui/Button';
|
||||||
import { Input } from '@/components/ui/Input';
|
import { Input } from '@/components/ui/Input';
|
||||||
@@ -35,10 +38,22 @@ export function TagPicker({ entityType, entityId, assignedTags: initialAssigned
|
|||||||
const [loading, setLoading] = useState(true);
|
const [loading, setLoading] = useState(true);
|
||||||
const [showCreateForm, setShowCreateForm] = useState(false);
|
const [showCreateForm, setShowCreateForm] = useState(false);
|
||||||
const [searchQuery, setSearchQuery] = useState('');
|
const [searchQuery, setSearchQuery] = useState('');
|
||||||
const [newTagName, setNewTagName] = useState('');
|
|
||||||
const [newTagColor, setNewTagColor] = useState('#3B82F6');
|
|
||||||
const [submitting, setSubmitting] = useState(false);
|
const [submitting, setSubmitting] = useState(false);
|
||||||
|
|
||||||
|
// ── Tag create form (RHF + Zod) ──
|
||||||
|
const tagSchema = z.object({
|
||||||
|
name: z.string().min(1, 'required'),
|
||||||
|
color: z.string().optional().default('#3B82F6'),
|
||||||
|
});
|
||||||
|
type TagFormData = z.infer<typeof tagSchema>;
|
||||||
|
|
||||||
|
const { register: registerTag, handleSubmit: handleSubmitTag, reset: resetTag, watch: watchTag, setValue: setTagValue, formState: { errors: tagErrors } } = useForm<TagFormData>({
|
||||||
|
resolver: zodResolver(tagSchema),
|
||||||
|
defaultValues: { name: '', color: '#3B82F6' },
|
||||||
|
});
|
||||||
|
|
||||||
|
const newTagColor = watchTag('color');
|
||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
let cancelled = false;
|
let cancelled = false;
|
||||||
setLoading(true);
|
setLoading(true);
|
||||||
@@ -95,14 +110,13 @@ export function TagPicker({ entityType, entityId, assignedTags: initialAssigned
|
|||||||
setSubmitting(false);
|
setSubmitting(false);
|
||||||
}, [entityType, entityId, toast, t]);
|
}, [entityType, entityId, toast, t]);
|
||||||
|
|
||||||
const handleCreateTag = useCallback(async () => {
|
const handleCreateTag = useCallback(async (data: TagFormData) => {
|
||||||
if (!newTagName.trim()) return;
|
|
||||||
setSubmitting(true);
|
setSubmitting(true);
|
||||||
try {
|
try {
|
||||||
const tag = await createTag({ name: newTagName.trim(), color: newTagColor });
|
const tag = await createTag({ name: data.name.trim(), color: data.color });
|
||||||
setAllTags((prev) => [...prev, tag]);
|
setAllTags((prev) => [...prev, tag]);
|
||||||
await handleAssign(tag.id);
|
await handleAssign(tag.id);
|
||||||
setNewTagName('');
|
resetTag({ name: '', color: '#3B82F6' });
|
||||||
setShowCreateForm(false);
|
setShowCreateForm(false);
|
||||||
toast.success(t('tags.createSuccess'));
|
toast.success(t('tags.createSuccess'));
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
@@ -110,7 +124,7 @@ export function TagPicker({ entityType, entityId, assignedTags: initialAssigned
|
|||||||
toast.error(msg);
|
toast.error(msg);
|
||||||
}
|
}
|
||||||
setSubmitting(false);
|
setSubmitting(false);
|
||||||
}, [newTagName, newTagColor, handleAssign, toast, t]);
|
}, [handleAssign, toast, t, resetTag]);
|
||||||
|
|
||||||
const colorOptions = ['#3B82F6', '#EF4444', '#10B981', '#F59E0B', '#8B5CF6', '#F97316', '#EC4899', '#6B7280'];
|
const colorOptions = ['#3B82F6', '#EF4444', '#10B981', '#F59E0B', '#8B5CF6', '#F97316', '#EC4899', '#6B7280'];
|
||||||
|
|
||||||
@@ -195,10 +209,11 @@ export function TagPicker({ entityType, entityId, assignedTags: initialAssigned
|
|||||||
</Button>
|
</Button>
|
||||||
) : (
|
) : (
|
||||||
<div className="space-y-3 p-4 border border-secondary-200 rounded-lg" data-testid="create-tag-form">
|
<div className="space-y-3 p-4 border border-secondary-200 rounded-lg" data-testid="create-tag-form">
|
||||||
|
<form onSubmit={handleSubmitTag(handleCreateTag)}>
|
||||||
<Input
|
<Input
|
||||||
label={t('tags.tagName')}
|
label={t('tags.tagName')}
|
||||||
value={newTagName}
|
{...registerTag('name')}
|
||||||
onChange={(e) => setNewTagName(e.target.value)}
|
error={tagErrors.name?.message === 'required' ? t('validation.required') : undefined}
|
||||||
placeholder={t('tags.tagName')}
|
placeholder={t('tags.tagName')}
|
||||||
/>
|
/>
|
||||||
<div className="space-y-1">
|
<div className="space-y-1">
|
||||||
@@ -207,7 +222,8 @@ export function TagPicker({ entityType, entityId, assignedTags: initialAssigned
|
|||||||
{colorOptions.map((color) => (
|
{colorOptions.map((color) => (
|
||||||
<button
|
<button
|
||||||
key={color}
|
key={color}
|
||||||
onClick={() => setNewTagColor(color)}
|
type="button"
|
||||||
|
onClick={() => setTagValue('color', color)}
|
||||||
className={clsx(
|
className={clsx(
|
||||||
'w-6 h-6 rounded-full transition-transform',
|
'w-6 h-6 rounded-full transition-transform',
|
||||||
newTagColor === color ? 'ring-2 ring-offset-2 ring-secondary-400 scale-110' : ''
|
newTagColor === color ? 'ring-2 ring-offset-2 ring-secondary-400 scale-110' : ''
|
||||||
@@ -220,13 +236,10 @@ export function TagPicker({ entityType, entityId, assignedTags: initialAssigned
|
|||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
<div className="flex gap-2">
|
<div className="flex gap-2">
|
||||||
<Button size="sm" onClick={handleCreateTag} isLoading={submitting} disabled={!newTagName.trim()}>
|
<Button size="sm" type="submit" isLoading={submitting}>{t('tags.save')}</Button>
|
||||||
{t('tags.save')}
|
<Button variant="secondary" size="sm" type="button" onClick={() => setShowCreateForm(false)}>{t('tags.cancel')}</Button>
|
||||||
</Button>
|
|
||||||
<Button variant="secondary" size="sm" onClick={() => setShowCreateForm(false)}>
|
|
||||||
{t('tags.cancel')}
|
|
||||||
</Button>
|
|
||||||
</div>
|
</div>
|
||||||
|
</form>
|
||||||
</div>
|
</div>
|
||||||
)}
|
)}
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
Reference in New Issue
Block a user