mirror of
https://github.com/maziggy/bambuddy.git
synced 2026-10-22 03:49:18 +02:00
Merge branch '0.1.8b' into feature/updated_plate_view
This commit is contained in:
@@ -0,0 +1,185 @@
|
||||
/**
|
||||
* Tests for the API client auth token handling.
|
||||
*/
|
||||
|
||||
import { describe, it, expect, afterEach, vi } from 'vitest';
|
||||
import { http, HttpResponse } from 'msw';
|
||||
import { setupServer } from 'msw/node';
|
||||
import { setAuthToken, getAuthToken, api } from '../../api/client';
|
||||
|
||||
// Mock localStorage
|
||||
const localStorageMock = {
|
||||
store: {} as Record<string, string>,
|
||||
getItem: vi.fn((key: string) => localStorageMock.store[key] || null),
|
||||
setItem: vi.fn((key: string, value: string) => {
|
||||
localStorageMock.store[key] = value;
|
||||
}),
|
||||
removeItem: vi.fn((key: string) => {
|
||||
delete localStorageMock.store[key];
|
||||
}),
|
||||
clear: vi.fn(() => {
|
||||
localStorageMock.store = {};
|
||||
}),
|
||||
};
|
||||
|
||||
Object.defineProperty(window, 'localStorage', {
|
||||
value: localStorageMock,
|
||||
});
|
||||
|
||||
// Create MSW server
|
||||
const server = setupServer();
|
||||
|
||||
beforeAll(() => server.listen({ onUnhandledRequest: 'bypass' }));
|
||||
afterEach(() => {
|
||||
server.resetHandlers();
|
||||
localStorageMock.clear();
|
||||
setAuthToken(null);
|
||||
});
|
||||
afterAll(() => server.close());
|
||||
|
||||
describe('Auth Token Management', () => {
|
||||
it('setAuthToken stores token in localStorage', () => {
|
||||
setAuthToken('test-token-123');
|
||||
expect(localStorageMock.setItem).toHaveBeenCalledWith('auth_token', 'test-token-123');
|
||||
expect(getAuthToken()).toBe('test-token-123');
|
||||
});
|
||||
|
||||
it('setAuthToken removes token from localStorage when null', () => {
|
||||
setAuthToken('test-token-123');
|
||||
setAuthToken(null);
|
||||
expect(localStorageMock.removeItem).toHaveBeenCalledWith('auth_token');
|
||||
expect(getAuthToken()).toBeNull();
|
||||
});
|
||||
});
|
||||
|
||||
describe('API Client Auth Header', () => {
|
||||
it('includes Authorization header when token is set', async () => {
|
||||
let capturedHeaders: Headers | null = null;
|
||||
|
||||
server.use(
|
||||
http.get('/api/v1/settings/spoolman', ({ request }) => {
|
||||
capturedHeaders = request.headers;
|
||||
return HttpResponse.json({
|
||||
spoolman_enabled: 'false',
|
||||
spoolman_url: '',
|
||||
spoolman_sync_mode: 'auto',
|
||||
});
|
||||
})
|
||||
);
|
||||
|
||||
setAuthToken('test-jwt-token');
|
||||
await api.getSpoolmanSettings();
|
||||
|
||||
expect(capturedHeaders).not.toBeNull();
|
||||
expect(capturedHeaders!.get('Authorization')).toBe('Bearer test-jwt-token');
|
||||
});
|
||||
|
||||
it('does not include Authorization header when token is not set', async () => {
|
||||
let capturedHeaders: Headers | null = null;
|
||||
|
||||
server.use(
|
||||
http.get('/api/v1/settings/spoolman', ({ request }) => {
|
||||
capturedHeaders = request.headers;
|
||||
return HttpResponse.json({
|
||||
spoolman_enabled: 'false',
|
||||
spoolman_url: '',
|
||||
spoolman_sync_mode: 'auto',
|
||||
});
|
||||
})
|
||||
);
|
||||
|
||||
setAuthToken(null);
|
||||
await api.getSpoolmanSettings();
|
||||
|
||||
expect(capturedHeaders).not.toBeNull();
|
||||
expect(capturedHeaders!.get('Authorization')).toBeNull();
|
||||
});
|
||||
|
||||
it('clears token on 401 Unauthorized response', async () => {
|
||||
server.use(
|
||||
http.get('/api/v1/settings/spoolman', () => {
|
||||
return HttpResponse.json(
|
||||
{ detail: 'Not authenticated' },
|
||||
{ status: 401 }
|
||||
);
|
||||
})
|
||||
);
|
||||
|
||||
setAuthToken('expired-token');
|
||||
expect(getAuthToken()).toBe('expired-token');
|
||||
|
||||
try {
|
||||
await api.getSpoolmanSettings();
|
||||
} catch {
|
||||
// Expected to throw
|
||||
}
|
||||
|
||||
expect(getAuthToken()).toBeNull();
|
||||
expect(localStorageMock.removeItem).toHaveBeenCalledWith('auth_token');
|
||||
});
|
||||
});
|
||||
|
||||
describe('FormData requests include auth header', () => {
|
||||
it('importProjectFile includes Authorization header', async () => {
|
||||
// Mock fetch directly for FormData requests (MSW can be flaky with multipart in some environments)
|
||||
const originalFetch = global.fetch;
|
||||
let capturedHeaders: Headers | null = null;
|
||||
|
||||
global.fetch = vi.fn().mockImplementation((url: string, init?: RequestInit) => {
|
||||
if (url.includes('/projects/import/file')) {
|
||||
capturedHeaders = new Headers(init?.headers);
|
||||
return Promise.resolve(new Response(JSON.stringify({
|
||||
id: 1,
|
||||
name: 'Test Project',
|
||||
description: '',
|
||||
total_cost: 0,
|
||||
total_print_time_seconds: 0,
|
||||
total_prints: 0,
|
||||
total_quantity: 0,
|
||||
status: 'active',
|
||||
due_date: null,
|
||||
created_at: '2026-01-01T00:00:00Z',
|
||||
updated_at: '2026-01-01T00:00:00Z',
|
||||
archives: [],
|
||||
bom_items: [],
|
||||
}), { status: 200 }));
|
||||
}
|
||||
return originalFetch(url, init);
|
||||
});
|
||||
|
||||
try {
|
||||
setAuthToken('test-token');
|
||||
const file = new File(['test content'], 'test.zip', { type: 'application/zip' });
|
||||
await api.importProjectFile(file);
|
||||
|
||||
expect(capturedHeaders).not.toBeNull();
|
||||
expect(capturedHeaders!.get('Authorization')).toBe('Bearer test-token');
|
||||
} finally {
|
||||
global.fetch = originalFetch;
|
||||
}
|
||||
});
|
||||
|
||||
it('exportProjectZip includes Authorization header', async () => {
|
||||
let capturedHeaders: Headers | null = null;
|
||||
|
||||
server.use(
|
||||
http.get('/api/v1/projects/:projectId/export', ({ request }) => {
|
||||
capturedHeaders = request.headers;
|
||||
const zipContent = new Uint8Array([0x50, 0x4b, 0x03, 0x04]); // ZIP magic bytes
|
||||
return new HttpResponse(zipContent, {
|
||||
status: 200,
|
||||
headers: {
|
||||
'Content-Type': 'application/zip',
|
||||
'Content-Disposition': 'attachment; filename="project.zip"',
|
||||
},
|
||||
});
|
||||
})
|
||||
);
|
||||
|
||||
setAuthToken('test-token');
|
||||
await api.exportProjectZip(1);
|
||||
|
||||
expect(capturedHeaders).not.toBeNull();
|
||||
expect(capturedHeaders!.get('Authorization')).toBe('Bearer test-token');
|
||||
});
|
||||
});
|
||||
@@ -131,7 +131,7 @@ describe('ConfirmModal', () => {
|
||||
|
||||
it('shows default loading text when loadingText not provided', () => {
|
||||
render(<ConfirmModal {...defaultProps} isLoading={true} />);
|
||||
expect(screen.getByText('Processing...')).toBeInTheDocument();
|
||||
expect(screen.getByText('Loading...')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
it('disables buttons when loading', () => {
|
||||
|
||||
@@ -0,0 +1,291 @@
|
||||
/**
|
||||
* Tests for the LinkSpoolModal component.
|
||||
*
|
||||
* Tests the Spoolman link spool modal including:
|
||||
* - Displaying unlinked spools
|
||||
* - Selecting a spool to link
|
||||
* - Link success with toast notification
|
||||
* - Link error with toast notification
|
||||
*/
|
||||
|
||||
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
||||
import { screen, waitFor, fireEvent } from '@testing-library/react';
|
||||
import { render } from '../utils';
|
||||
import { LinkSpoolModal } from '../../components/LinkSpoolModal';
|
||||
|
||||
// Mock the API client
|
||||
vi.mock('../../api/client', () => ({
|
||||
api: {
|
||||
getUnlinkedSpools: vi.fn(),
|
||||
linkSpool: vi.fn(),
|
||||
getSettings: vi.fn().mockResolvedValue({}),
|
||||
getAuthStatus: vi.fn().mockResolvedValue({ enabled: false, configured: false }),
|
||||
},
|
||||
}));
|
||||
|
||||
// Mock the toast context
|
||||
const mockShowToast = vi.fn();
|
||||
vi.mock('../../contexts/ToastContext', async (importOriginal) => {
|
||||
const actual = await importOriginal<typeof import('../../contexts/ToastContext')>();
|
||||
return {
|
||||
...actual,
|
||||
useToast: () => ({ showToast: mockShowToast }),
|
||||
};
|
||||
});
|
||||
|
||||
// Import mocked module
|
||||
import { api } from '../../api/client';
|
||||
|
||||
describe('LinkSpoolModal', () => {
|
||||
const defaultProps = {
|
||||
isOpen: true,
|
||||
onClose: vi.fn(),
|
||||
trayUuid: 'A1B2C3D4E5F6A1B2C3D4E5F6A1B2C3D4',
|
||||
trayInfo: {
|
||||
type: 'PLA Basic',
|
||||
color: 'FF0000',
|
||||
location: 'AMS A1',
|
||||
},
|
||||
};
|
||||
|
||||
const mockUnlinkedSpools = [
|
||||
{
|
||||
id: 1,
|
||||
filament_name: 'PLA Red',
|
||||
filament_material: 'PLA',
|
||||
filament_color_hex: 'FF0000',
|
||||
remaining_weight: 800,
|
||||
location: 'Shelf A',
|
||||
},
|
||||
{
|
||||
id: 2,
|
||||
filament_name: 'PETG Blue',
|
||||
filament_material: 'PETG',
|
||||
filament_color_hex: '0000FF',
|
||||
remaining_weight: 500,
|
||||
location: null,
|
||||
},
|
||||
];
|
||||
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks();
|
||||
vi.mocked(api.getUnlinkedSpools).mockResolvedValue(mockUnlinkedSpools);
|
||||
vi.mocked(api.linkSpool).mockResolvedValue({ success: true, message: 'Linked' });
|
||||
});
|
||||
|
||||
describe('rendering', () => {
|
||||
it('renders modal title', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
// Look for the title in h2 element
|
||||
expect(screen.getByRole('heading', { name: /link to spoolman/i })).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
it('displays tray info', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('PLA Basic')).toBeInTheDocument();
|
||||
expect(screen.getByText('(AMS A1)')).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
it('displays tray UUID', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText(defaultProps.trayUuid)).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
it('shows loading state while fetching spools', async () => {
|
||||
// Delay the response
|
||||
vi.mocked(api.getUnlinkedSpools).mockImplementation(
|
||||
() => new Promise(() => {})
|
||||
);
|
||||
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(document.querySelector('.animate-spin')).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
it('displays unlinked spools list', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('PLA Red')).toBeInTheDocument();
|
||||
expect(screen.getByText('PETG Blue')).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
it('shows message when no unlinked spools', async () => {
|
||||
vi.mocked(api.getUnlinkedSpools).mockResolvedValue([]);
|
||||
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('No unlinked spools available')).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
it('does not render when isOpen is false', () => {
|
||||
render(<LinkSpoolModal {...defaultProps} isOpen={false} />);
|
||||
expect(screen.queryByRole('heading', { name: /link to spoolman/i })).not.toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
describe('spool selection', () => {
|
||||
it('allows selecting a spool', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('PLA Red')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
// Click to select spool
|
||||
fireEvent.click(screen.getByText('PLA Red'));
|
||||
|
||||
// Should show check mark (via visual styling)
|
||||
const selectedButton = screen.getByText('PLA Red').closest('button');
|
||||
expect(selectedButton).toHaveClass('border-bambu-green');
|
||||
});
|
||||
|
||||
it('link button is disabled until spool is selected', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('PLA Red')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
const linkButton = screen.getByRole('button', { name: /link to spoolman/i });
|
||||
expect(linkButton).toBeDisabled();
|
||||
|
||||
// Select a spool
|
||||
fireEvent.click(screen.getByText('PLA Red'));
|
||||
|
||||
expect(linkButton).not.toBeDisabled();
|
||||
});
|
||||
});
|
||||
|
||||
describe('linking', () => {
|
||||
it('calls linkSpool API on submit', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('PLA Red')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
// Select a spool
|
||||
fireEvent.click(screen.getByText('PLA Red'));
|
||||
|
||||
// Click link button
|
||||
fireEvent.click(screen.getByRole('button', { name: /link to spoolman/i }));
|
||||
|
||||
await waitFor(() => {
|
||||
expect(api.linkSpool).toHaveBeenCalledWith(1, defaultProps.trayUuid);
|
||||
});
|
||||
});
|
||||
|
||||
it('shows success toast on successful link', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('PLA Red')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
fireEvent.click(screen.getByText('PLA Red'));
|
||||
fireEvent.click(screen.getByRole('button', { name: /link to spoolman/i }));
|
||||
|
||||
await waitFor(() => {
|
||||
expect(mockShowToast).toHaveBeenCalledWith(
|
||||
'Spool linked to Spoolman successfully',
|
||||
'success'
|
||||
);
|
||||
});
|
||||
});
|
||||
|
||||
it('calls onClose after successful link', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('PLA Red')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
fireEvent.click(screen.getByText('PLA Red'));
|
||||
fireEvent.click(screen.getByRole('button', { name: /link to spoolman/i }));
|
||||
|
||||
await waitFor(() => {
|
||||
expect(defaultProps.onClose).toHaveBeenCalled();
|
||||
});
|
||||
});
|
||||
|
||||
it('shows error toast on link failure', async () => {
|
||||
const errorMessage = 'Failed to update spool';
|
||||
vi.mocked(api.linkSpool).mockRejectedValue(new Error(errorMessage));
|
||||
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('PLA Red')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
fireEvent.click(screen.getByText('PLA Red'));
|
||||
fireEvent.click(screen.getByRole('button', { name: /link to spoolman/i }));
|
||||
|
||||
await waitFor(() => {
|
||||
expect(mockShowToast).toHaveBeenCalledWith(
|
||||
`Failed to link spool: ${errorMessage}`,
|
||||
'error'
|
||||
);
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
describe('modal actions', () => {
|
||||
it('calls onClose when cancel button is clicked', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Cancel')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
fireEvent.click(screen.getByText('Cancel'));
|
||||
expect(defaultProps.onClose).toHaveBeenCalled();
|
||||
});
|
||||
|
||||
it('calls onClose when backdrop is clicked', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByRole('heading', { name: /link to spoolman/i })).toBeInTheDocument();
|
||||
});
|
||||
|
||||
// Click the backdrop (the element with bg-black/60)
|
||||
const backdrop = document.querySelector('.bg-black\\/60');
|
||||
if (backdrop) {
|
||||
fireEvent.click(backdrop);
|
||||
expect(defaultProps.onClose).toHaveBeenCalled();
|
||||
}
|
||||
});
|
||||
|
||||
it('calls onClose when X button is clicked', async () => {
|
||||
render(<LinkSpoolModal {...defaultProps} />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByRole('heading', { name: /link to spoolman/i })).toBeInTheDocument();
|
||||
});
|
||||
|
||||
// Find and click the X button in the header
|
||||
const closeButtons = screen.getAllByRole('button');
|
||||
const xButton = closeButtons.find(btn => btn.querySelector('svg.lucide-x'));
|
||||
if (xButton) {
|
||||
fireEvent.click(xButton);
|
||||
expect(defaultProps.onClose).toHaveBeenCalled();
|
||||
}
|
||||
});
|
||||
});
|
||||
});
|
||||
@@ -18,6 +18,8 @@ vi.mock('../../api/client', () => ({
|
||||
api: {
|
||||
getSettings: vi.fn().mockResolvedValue({}),
|
||||
updateSettings: vi.fn().mockResolvedValue({}),
|
||||
getSpoolmanSettings: vi.fn(),
|
||||
updateSpoolmanSettings: vi.fn(),
|
||||
getSpoolmanStatus: vi.fn(),
|
||||
connectSpoolman: vi.fn(),
|
||||
disconnectSpoolman: vi.fn(),
|
||||
@@ -30,17 +32,21 @@ vi.mock('../../api/client', () => ({
|
||||
// Import mocked module
|
||||
import { api } from '../../api/client';
|
||||
|
||||
// Mock fetch for Spoolman settings endpoints
|
||||
const mockFetchResponse = (data: object) => ({
|
||||
ok: true,
|
||||
json: () => Promise.resolve(data),
|
||||
});
|
||||
|
||||
describe('SpoolmanSettings', () => {
|
||||
beforeEach(() => {
|
||||
vi.clearAllMocks();
|
||||
|
||||
// Default API mocks
|
||||
vi.mocked(api.getSpoolmanSettings).mockResolvedValue({
|
||||
spoolman_enabled: 'false',
|
||||
spoolman_url: '',
|
||||
spoolman_sync_mode: 'auto',
|
||||
});
|
||||
vi.mocked(api.updateSpoolmanSettings).mockResolvedValue({
|
||||
spoolman_enabled: 'false',
|
||||
spoolman_url: '',
|
||||
spoolman_sync_mode: 'auto',
|
||||
});
|
||||
vi.mocked(api.getSpoolmanStatus).mockResolvedValue({
|
||||
enabled: false,
|
||||
connected: false,
|
||||
@@ -56,26 +62,12 @@ describe('SpoolmanSettings', () => {
|
||||
skipped: [],
|
||||
errors: [],
|
||||
});
|
||||
|
||||
// Default fetch mock for settings - disabled state
|
||||
global.fetch = vi.fn().mockImplementation((url: string) => {
|
||||
if (url.includes('/api/v1/settings/spoolman')) {
|
||||
return Promise.resolve(
|
||||
mockFetchResponse({
|
||||
spoolman_enabled: 'false',
|
||||
spoolman_url: '',
|
||||
spoolman_sync_mode: 'auto',
|
||||
})
|
||||
);
|
||||
}
|
||||
return Promise.reject(new Error('Unknown URL'));
|
||||
}) as any;
|
||||
});
|
||||
|
||||
describe('rendering', () => {
|
||||
it('renders loading state initially', () => {
|
||||
// Delay the fetch response to catch loading state
|
||||
global.fetch = vi.fn().mockImplementation(() => new Promise(() => {})) as any;
|
||||
// Delay the API response to catch loading state
|
||||
vi.mocked(api.getSpoolmanSettings).mockImplementation(() => new Promise(() => {}));
|
||||
render(<SpoolmanSettings />);
|
||||
|
||||
// Should show loading spinner
|
||||
@@ -159,27 +151,16 @@ describe('SpoolmanSettings', () => {
|
||||
|
||||
describe('enabled state', () => {
|
||||
beforeEach(() => {
|
||||
global.fetch = vi.fn().mockImplementation((url: string) => {
|
||||
if (url.includes('/api/v1/settings/spoolman')) {
|
||||
if (url.includes('PUT') || (url as any).method === 'PUT') {
|
||||
return Promise.resolve(
|
||||
mockFetchResponse({
|
||||
spoolman_enabled: 'true',
|
||||
spoolman_url: 'http://localhost:7912',
|
||||
spoolman_sync_mode: 'auto',
|
||||
})
|
||||
);
|
||||
}
|
||||
return Promise.resolve(
|
||||
mockFetchResponse({
|
||||
spoolman_enabled: 'true',
|
||||
spoolman_url: 'http://localhost:7912',
|
||||
spoolman_sync_mode: 'auto',
|
||||
})
|
||||
);
|
||||
}
|
||||
return Promise.reject(new Error('Unknown URL'));
|
||||
}) as any;
|
||||
vi.mocked(api.getSpoolmanSettings).mockResolvedValue({
|
||||
spoolman_enabled: 'true',
|
||||
spoolman_url: 'http://localhost:7912',
|
||||
spoolman_sync_mode: 'auto',
|
||||
});
|
||||
vi.mocked(api.updateSpoolmanSettings).mockResolvedValue({
|
||||
spoolman_enabled: 'true',
|
||||
spoolman_url: 'http://localhost:7912',
|
||||
spoolman_sync_mode: 'auto',
|
||||
});
|
||||
});
|
||||
|
||||
it('URL input is enabled when Spoolman is enabled', async () => {
|
||||
|
||||
@@ -42,6 +42,7 @@ const createMockSettings = (overrides = {}) => ({
|
||||
access_code_set: false,
|
||||
mode: 'immediate' as const,
|
||||
model: '3DPrinter-X1-Carbon',
|
||||
target_printer_id: null as number | null,
|
||||
status: {
|
||||
enabled: false,
|
||||
running: false,
|
||||
@@ -274,7 +275,7 @@ describe('VirtualPrinterSettings', () => {
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Review')).toBeInTheDocument();
|
||||
expect(screen.getByText('Review and tag before archiving')).toBeInTheDocument();
|
||||
expect(screen.getByText('Review before archiving')).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
@@ -283,7 +284,7 @@ describe('VirtualPrinterSettings', () => {
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Queue')).toBeInTheDocument();
|
||||
expect(screen.getByText('Archive and add to print queue')).toBeInTheDocument();
|
||||
expect(screen.getByText('Archive and add to queue')).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
@@ -472,4 +473,100 @@ describe('VirtualPrinterSettings', () => {
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
describe('proxy mode', () => {
|
||||
it('renders Proxy mode option', async () => {
|
||||
render(<VirtualPrinterSettings />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Proxy')).toBeInTheDocument();
|
||||
expect(screen.getByText('Relay to real printer')).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
it('highlights proxy mode when selected', async () => {
|
||||
vi.mocked(virtualPrinterApi.getSettings).mockResolvedValue(
|
||||
createMockSettings({ mode: 'proxy', target_printer_id: 1 })
|
||||
);
|
||||
|
||||
render(<VirtualPrinterSettings />);
|
||||
|
||||
await waitFor(() => {
|
||||
const proxyButton = screen.getByText('Proxy').closest('button');
|
||||
expect(proxyButton?.className).toContain('border-blue-500');
|
||||
});
|
||||
});
|
||||
|
||||
it('shows proxy status details when running in proxy mode', async () => {
|
||||
vi.mocked(virtualPrinterApi.getSettings).mockResolvedValue(
|
||||
createMockSettings({
|
||||
enabled: true,
|
||||
mode: 'proxy',
|
||||
target_printer_id: 1,
|
||||
status: {
|
||||
enabled: true,
|
||||
running: true,
|
||||
mode: 'proxy',
|
||||
name: 'Bambuddy (Proxy)',
|
||||
serial: '00M00A391800001',
|
||||
model: '3DPrinter-X1-Carbon',
|
||||
model_name: 'X1C',
|
||||
pending_files: 0,
|
||||
proxy: {
|
||||
running: true,
|
||||
target_host: '192.168.1.100',
|
||||
ftp_port: 9990,
|
||||
mqtt_port: 8883,
|
||||
ftp_connections: 1,
|
||||
mqtt_connections: 2,
|
||||
},
|
||||
},
|
||||
})
|
||||
);
|
||||
|
||||
render(<VirtualPrinterSettings />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Running')).toBeInTheDocument();
|
||||
expect(screen.getByText('Status Details')).toBeInTheDocument();
|
||||
// IP address appears in multiple places (target printer and status details)
|
||||
expect(screen.getAllByText('192.168.1.100').length).toBeGreaterThan(0);
|
||||
});
|
||||
});
|
||||
|
||||
it('shows target printer dropdown in proxy mode', async () => {
|
||||
vi.mocked(virtualPrinterApi.getSettings).mockResolvedValue(
|
||||
createMockSettings({ mode: 'proxy' })
|
||||
);
|
||||
|
||||
render(<VirtualPrinterSettings />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Target Printer')).toBeInTheDocument();
|
||||
expect(screen.getByText('Select a printer...')).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
it('changes mode to proxy on click', async () => {
|
||||
const user = userEvent.setup();
|
||||
vi.mocked(virtualPrinterApi.updateSettings).mockResolvedValue(
|
||||
createMockSettings({ mode: 'proxy' })
|
||||
);
|
||||
|
||||
render(<VirtualPrinterSettings />);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Proxy')).toBeInTheDocument();
|
||||
});
|
||||
|
||||
const proxyButton = screen.getByText('Proxy').closest('button');
|
||||
if (proxyButton) {
|
||||
await user.click(proxyButton);
|
||||
}
|
||||
|
||||
await waitFor(() => {
|
||||
expect(virtualPrinterApi.updateSettings).toHaveBeenCalledWith({ mode: 'proxy' });
|
||||
});
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
@@ -166,4 +166,99 @@ describe('AuthContext', () => {
|
||||
expect(result.current.hasPermission('printers:read' as Permission)).toBe(false);
|
||||
});
|
||||
});
|
||||
|
||||
describe('CVE-2026-25505 fix: auth disabled grants all access', () => {
|
||||
beforeEach(() => {
|
||||
server.use(
|
||||
http.get('/api/v1/auth/status', () => {
|
||||
return HttpResponse.json({
|
||||
auth_enabled: false,
|
||||
requires_setup: false,
|
||||
});
|
||||
})
|
||||
);
|
||||
});
|
||||
|
||||
it('isAdmin is true when auth is disabled', async () => {
|
||||
const { result } = renderHook(() => useAuth(), {
|
||||
wrapper: createWrapper(),
|
||||
});
|
||||
|
||||
await waitFor(() => {
|
||||
expect(result.current.loading).toBe(false);
|
||||
});
|
||||
|
||||
// When auth disabled, user is treated as admin
|
||||
expect(result.current.isAdmin).toBe(true);
|
||||
});
|
||||
|
||||
it('canModify allows all modifications when auth disabled', async () => {
|
||||
const { result } = renderHook(() => useAuth(), {
|
||||
wrapper: createWrapper(),
|
||||
});
|
||||
|
||||
await waitFor(() => {
|
||||
expect(result.current.loading).toBe(false);
|
||||
});
|
||||
|
||||
// All canModify checks should pass when auth is disabled
|
||||
expect(result.current.canModify('queue', 'update', 1)).toBe(true);
|
||||
expect(result.current.canModify('queue', 'update', 999)).toBe(true);
|
||||
expect(result.current.canModify('queue', 'update', null)).toBe(true);
|
||||
expect(result.current.canModify('archives', 'delete', 1)).toBe(true);
|
||||
expect(result.current.canModify('library', 'update', null)).toBe(true);
|
||||
});
|
||||
|
||||
it('all permissions are granted when auth is disabled', async () => {
|
||||
const { result } = renderHook(() => useAuth(), {
|
||||
wrapper: createWrapper(),
|
||||
});
|
||||
|
||||
await waitFor(() => {
|
||||
expect(result.current.loading).toBe(false);
|
||||
});
|
||||
|
||||
// All permission checks should pass
|
||||
expect(result.current.hasPermission('archives:read' as Permission)).toBe(true);
|
||||
expect(result.current.hasPermission('archives:delete_all' as Permission)).toBe(true);
|
||||
expect(result.current.hasPermission('settings:update' as Permission)).toBe(true);
|
||||
expect(result.current.hasPermission('api_keys:create' as Permission)).toBe(true);
|
||||
expect(result.current.hasPermission('groups:delete' as Permission)).toBe(true);
|
||||
});
|
||||
|
||||
it('hasAnyPermission returns true for protected permissions', async () => {
|
||||
const { result } = renderHook(() => useAuth(), {
|
||||
wrapper: createWrapper(),
|
||||
});
|
||||
|
||||
await waitFor(() => {
|
||||
expect(result.current.loading).toBe(false);
|
||||
});
|
||||
|
||||
expect(
|
||||
result.current.hasAnyPermission(
|
||||
'api_keys:create' as Permission,
|
||||
'groups:delete' as Permission
|
||||
)
|
||||
).toBe(true);
|
||||
});
|
||||
|
||||
it('hasAllPermissions returns true for any combination', async () => {
|
||||
const { result } = renderHook(() => useAuth(), {
|
||||
wrapper: createWrapper(),
|
||||
});
|
||||
|
||||
await waitFor(() => {
|
||||
expect(result.current.loading).toBe(false);
|
||||
});
|
||||
|
||||
expect(
|
||||
result.current.hasAllPermissions(
|
||||
'settings:update' as Permission,
|
||||
'api_keys:create' as Permission,
|
||||
'groups:delete' as Permission
|
||||
)
|
||||
).toBe(true);
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
@@ -0,0 +1,35 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import en from '../../i18n/locales/en';
|
||||
import de from '../../i18n/locales/de';
|
||||
|
||||
/**
|
||||
* Recursively extracts all keys from a nested object as dot-notation paths.
|
||||
* Example: { foo: { bar: 'baz' } } => ['foo.bar']
|
||||
*/
|
||||
const getKeys = (obj: object, prefix = ''): string[] => {
|
||||
return Object.entries(obj).flatMap(([key, value]) => {
|
||||
const path = prefix ? `${prefix}.${key}` : key;
|
||||
return typeof value === 'object' && value !== null
|
||||
? getKeys(value, path)
|
||||
: [path];
|
||||
});
|
||||
};
|
||||
|
||||
describe('i18n locale parity', () => {
|
||||
const enKeys = new Set(getKeys(en));
|
||||
const deKeys = new Set(getKeys(de));
|
||||
|
||||
it('German locale has all English keys', () => {
|
||||
const missingInGerman = [...enKeys].filter((k) => !deKeys.has(k)).sort();
|
||||
expect(missingInGerman, `Missing ${missingInGerman.length} key(s) in German locale`).toEqual([]);
|
||||
});
|
||||
|
||||
it('English locale has all German keys', () => {
|
||||
const missingInEnglish = [...deKeys].filter((k) => !enKeys.has(k)).sort();
|
||||
expect(missingInEnglish, `Missing ${missingInEnglish.length} key(s) in English locale`).toEqual([]);
|
||||
});
|
||||
|
||||
it('both locales have the same number of keys', () => {
|
||||
expect(enKeys.size).toBe(deKeys.size);
|
||||
});
|
||||
});
|
||||
@@ -11,6 +11,8 @@ import { MemoryRouter, Route, Routes } from 'react-router-dom';
|
||||
import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
|
||||
import { ThemeProvider } from '../../contexts/ThemeContext';
|
||||
import { ToastProvider } from '../../contexts/ToastContext';
|
||||
import { I18nextProvider } from 'react-i18next';
|
||||
import i18n from '../../i18n';
|
||||
|
||||
// Mock navigator.sendBeacon which isn't available in jsdom
|
||||
vi.stubGlobal('navigator', {
|
||||
@@ -39,15 +41,17 @@ function renderCameraPage(printerId: number) {
|
||||
|
||||
return rtlRender(
|
||||
<QueryClientProvider client={queryClient}>
|
||||
<MemoryRouter initialEntries={[`/cameras/${printerId}`]}>
|
||||
<ThemeProvider>
|
||||
<ToastProvider>
|
||||
<Routes>
|
||||
<Route path="/cameras/:printerId" element={<CameraPage />} />
|
||||
</Routes>
|
||||
</ToastProvider>
|
||||
</ThemeProvider>
|
||||
</MemoryRouter>
|
||||
<I18nextProvider i18n={i18n}>
|
||||
<MemoryRouter initialEntries={[`/cameras/${printerId}`]}>
|
||||
<ThemeProvider>
|
||||
<ToastProvider>
|
||||
<Routes>
|
||||
<Route path="/cameras/:printerId" element={<CameraPage />} />
|
||||
</Routes>
|
||||
</ToastProvider>
|
||||
</ThemeProvider>
|
||||
</MemoryRouter>
|
||||
</I18nextProvider>
|
||||
</QueryClientProvider>
|
||||
);
|
||||
}
|
||||
@@ -94,8 +98,9 @@ describe('CameraPage', () => {
|
||||
renderCameraPage(1);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Live')).toBeInTheDocument();
|
||||
expect(screen.getByText('Snapshot')).toBeInTheDocument();
|
||||
// Check for translation key or translated text
|
||||
expect(screen.getByText(/Live|camera\.live/)).toBeInTheDocument();
|
||||
expect(screen.getByText(/Snapshot|camera\.snapshot/)).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
|
||||
@@ -124,7 +129,8 @@ describe('CameraPage', () => {
|
||||
renderCameraPage(0);
|
||||
|
||||
await waitFor(() => {
|
||||
expect(screen.getByText('Invalid printer ID')).toBeInTheDocument();
|
||||
// Check for translation key or translated text
|
||||
expect(screen.getByText(/Invalid printer ID|camera\.invalidPrinterId/)).toBeInTheDocument();
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user