| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260 |
- /**
- * Tests for the ConfigureAmsSlotModal component.
- */
- import { describe, it, expect, vi, beforeEach } from 'vitest';
- import { screen, fireEvent, waitFor } from '@testing-library/react';
- import { render } from '../utils';
- import { ConfigureAmsSlotModal } from '../../components/ConfigureAmsSlotModal';
- import { api } from '../../api/client';
- // Mock the API client
- vi.mock('../../api/client', () => ({
- api: {
- getCloudSettings: vi.fn(),
- getKProfiles: vi.fn(),
- configureAmsSlot: vi.fn(),
- getCloudSettingDetail: vi.fn(),
- saveSlotPreset: vi.fn(),
- getSettings: vi.fn().mockResolvedValue({}),
- // Queried by the modal for the @BBL short-code matcher. Omitted, it is
- // undefined here, so the query runs with no queryFn and rejects -- a
- // stray render at an unpredictable moment in every one of these tests.
- getSlicerPrinterModels: vi.fn().mockResolvedValue({}),
- updateSettings: vi.fn().mockResolvedValue({}),
- getLocalPresets: vi.fn(),
- getBuiltinFilaments: vi.fn(),
- orcaCloudListProfiles: vi.fn(),
- orcaCloudGetProfile: vi.fn(),
- searchColors: vi.fn(),
- getColorCatalog: vi.fn(),
- resetAmsSlot: vi.fn(),
- // The fitted nozzle's flow; none reported unless a test sets one.
- getPrinterStatus: vi.fn().mockResolvedValue({}),
- },
- }));
- const mockCloudSettings = {
- filament: [
- {
- setting_id: 'GFSL05_09',
- name: 'Bambu PLA Basic @BBL X1C',
- filament_id: 'GFL05',
- },
- {
- setting_id: 'PFUScd84f663d2c2ef',
- name: '# Overture Matte PLA @BBL H2D',
- filament_id: null,
- },
- ],
- };
- const mockKProfiles = {
- profiles: [
- {
- id: 1,
- name: 'PLA Basic',
- k_value: '0.020',
- filament_id: 'GFL05',
- setting_id: '',
- extruder_id: 1,
- cali_idx: 1,
- },
- ],
- };
- const defaultProps = {
- isOpen: true,
- onClose: vi.fn(),
- printerId: 1,
- slotInfo: {
- amsId: 0,
- trayId: 0,
- trayType: 'PLA',
- trayColor: 'FFFFFF',
- traySubBrands: 'PLA Basic',
- },
- nozzleDiameter: '0.4',
- onSuccess: vi.fn(),
- };
- describe('ConfigureAmsSlotModal', () => {
- beforeEach(() => {
- vi.clearAllMocks();
- (api.getPrinterStatus as ReturnType<typeof vi.fn>).mockResolvedValue({});
- // Mock scrollIntoView which is not available in jsdom
- Element.prototype.scrollIntoView = vi.fn();
- (api.getCloudSettings as ReturnType<typeof vi.fn>).mockResolvedValue(mockCloudSettings);
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue(mockKProfiles);
- (api.configureAmsSlot as ReturnType<typeof vi.fn>).mockResolvedValue({ success: true });
- (api.saveSlotPreset as ReturnType<typeof vi.fn>).mockResolvedValue({ success: true });
- (api.getLocalPresets as ReturnType<typeof vi.fn>).mockResolvedValue({ filament: [] });
- (api.orcaCloudListProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament: [], printer: [], process: [],
- });
- (api.getBuiltinFilaments as ReturnType<typeof vi.fn>).mockResolvedValue([]);
- (api.searchColors as ReturnType<typeof vi.fn>).mockResolvedValue([]);
- (api.getColorCatalog as ReturnType<typeof vi.fn>).mockResolvedValue([]);
- (api.resetAmsSlot as ReturnType<typeof vi.fn>).mockResolvedValue({ success: true, message: 'ok' });
- });
- it('renders nothing visible when closed', () => {
- render(<ConfigureAmsSlotModal {...defaultProps} isOpen={false} />);
- expect(screen.queryByText('Configure AMS Slot')).not.toBeInTheDocument();
- });
- it('renders a clear tray as the transparency checkerboard rather than solid black (#2912)', () => {
- render(
- <ConfigureAmsSlotModal
- {...defaultProps}
- slotInfo={{ ...defaultProps.slotInfo, trayType: 'PETG', trayColor: '00000000' }}
- />,
- );
- const styled = Array.from(document.querySelectorAll<HTMLElement>('[style]'));
- expect(
- styled.some((el) => el.style.backgroundImage.includes('repeating-conic-gradient')),
- ).toBe(true);
- // The regression this pins: the alpha byte was sliced off before painting,
- // so a clear tray rendered as opaque black — the reported symptom, in the UI.
- expect(styled.some((el) => el.style.backgroundColor === 'rgb(0, 0, 0)')).toBe(false);
- });
- it('renders modal when open', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- expect(screen.getByText(/Configure AMS/)).toBeInTheDocument();
- });
- });
- it('displays basic color buttons', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- // Check for basic color buttons by their title attribute
- expect(screen.getByTitle('White')).toBeInTheDocument();
- expect(screen.getByTitle('Black')).toBeInTheDocument();
- expect(screen.getByTitle('Red')).toBeInTheDocument();
- expect(screen.getByTitle('Blue')).toBeInTheDocument();
- expect(screen.getByTitle('Green')).toBeInTheDocument();
- expect(screen.getByTitle('Yellow')).toBeInTheDocument();
- expect(screen.getByTitle('Orange')).toBeInTheDocument();
- expect(screen.getByTitle('Gray')).toBeInTheDocument();
- });
- });
- it('does not show extended colors by default', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- expect(screen.getByTitle('White')).toBeInTheDocument();
- });
- // Extended colors should not be visible initially
- expect(screen.queryByTitle('Cyan')).not.toBeInTheDocument();
- expect(screen.queryByTitle('Purple')).not.toBeInTheDocument();
- expect(screen.queryByTitle('Coral')).not.toBeInTheDocument();
- });
- it('shows extended colors when expand button is clicked', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- expect(screen.getByTitle('White')).toBeInTheDocument();
- });
- // Click the expand button (+ button)
- const expandButton = screen.getByTitle('Show more colors');
- fireEvent.click(expandButton);
- // Extended colors should now be visible
- await waitFor(() => {
- expect(screen.getByTitle('Cyan')).toBeInTheDocument();
- expect(screen.getByTitle('Purple')).toBeInTheDocument();
- expect(screen.getByTitle('Pink')).toBeInTheDocument();
- expect(screen.getByTitle('Brown')).toBeInTheDocument();
- expect(screen.getByTitle('Coral')).toBeInTheDocument();
- });
- });
- it('hides extended colors when collapse button is clicked', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- expect(screen.getByTitle('White')).toBeInTheDocument();
- });
- // Click the expand button
- const expandButton = screen.getByTitle('Show more colors');
- fireEvent.click(expandButton);
- // Wait for extended colors to appear
- await waitFor(() => {
- expect(screen.getByTitle('Cyan')).toBeInTheDocument();
- });
- // Click the collapse button
- const collapseButton = screen.getByTitle('Show less colors');
- fireEvent.click(collapseButton);
- // Extended colors should be hidden again
- await waitFor(() => {
- expect(screen.queryByTitle('Cyan')).not.toBeInTheDocument();
- });
- });
- it('selects a color when color button is clicked', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- expect(screen.getByTitle('Red')).toBeInTheDocument();
- });
- // Click the red color button
- const redButton = screen.getByTitle('Red');
- fireEvent.click(redButton);
- // The color input should now show "Red"
- const colorInput = screen.getByPlaceholderText(/Color name or hex/);
- expect(colorInput).toHaveValue('Red');
- });
- it('sends no tray_info_idx when cloud detail has filament_id: null (#3003)', async () => {
- // Cloud returns a user preset that inherits from a generic Bambu base and
- // has no distinct filament_id of its own — this is how Bambu Cloud responds
- // for custom presets built on top of "Generic ABS @BBL H2D" etc.
- (api.getCloudSettingDetail as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament_id: null,
- base_id: 'GFSB99_07',
- name: '# Overture Matte PLA @BBL H2D',
- });
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'PFUScd84f663d2c2ef',
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText('# Overture Matte PLA @BBL H2D')).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- const payload = (api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3];
- // #1053 sent the PFUS here instead, to stop the slot collapsing to
- // "Generic ABS" in the slicer. The A1 capture in #3003 showed that does not
- // work: tray_info_idx is 8 characters on the printer, so an 18-character
- // PFUS is stored truncated (PFUScd84 here) and acknowledged as a success.
- // The slot then resolves to nothing — "Generic" anyway, plus a calibration
- // table keyed by an id that does not exist. Sending nothing lets the
- // backend pick a filament id the printer can actually hold; setting_id
- // still carries the preset.
- expect(payload.tray_info_idx).toBe('');
- expect(payload.setting_id).toBe('PFUScd84f663d2c2ef');
- });
- it('has the backend resolve an Orca Cloud profile\'s own filament_id (#3003, #3216)', async () => {
- // OrcaSlicer's "Sync filaments" matches a slot by filament_id alone, so a
- // generic here turns the profile into "Generic PLA" in the slicer. The
- // lookup used to run here and fall back silently; it now runs in configure,
- // which logs the outcome and reports a fallback.
- const ORCA_ID = '3f2a9c1e-4b7d-4a02-9f61-8c5e2d1a7b30';
- (api.orcaCloudListProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament: [{ setting_id: ORCA_ID, name: 'Overture Matte PLA @Orca', type: 'filament', is_custom: true }],
- printer: [],
- process: [],
- });
- (api.configureAmsSlot as ReturnType<typeof vi.fn>).mockResolvedValue({
- success: true, tray_info_idx: 'P56e1be0', orca_fallback_reason: '',
- });
- const slotInfo = { ...defaultProps.slotInfo, savedPresetId: ORCA_ID };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText('Overture Matte PLA @Orca')).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.saveSlotPreset).toHaveBeenCalled();
- });
- const payload = (api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3];
- expect(payload.orca_profile_id).toBe(ORCA_ID);
- expect(payload.tray_info_idx).toBe('');
- // The UUID is what the slicer cannot resolve; it goes in neither field.
- expect(payload.setting_id).toBe('');
- expect(api.orcaCloudGetProfile).not.toHaveBeenCalled();
- // The slot preset row records the id the slot was actually given, so the
- // slot card notices when the Device tab re-configures the slot.
- const saveArgs = (api.saveSlotPreset as ReturnType<typeof vi.fn>).mock.calls[0];
- expect(saveArgs[5]).toBe('orca_cloud');
- expect(saveArgs[6]).toBe('P56e1be0');
- expect(screen.queryByText(/OrcaSlicer will see this slot as Generic/)).not.toBeInTheDocument();
- });
- it('warns when an Orca profile went into the slot as a generic (#3216)', async () => {
- const ORCA_ID = '3f2a9c1e-4b7d-4a02-9f61-8c5e2d1a7b30';
- (api.orcaCloudListProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament: [{ setting_id: ORCA_ID, name: 'Homebrew PLA @Orca', type: 'filament', is_custom: true }],
- printer: [],
- process: [],
- });
- (api.configureAmsSlot as ReturnType<typeof vi.fn>).mockResolvedValue({
- success: true, tray_info_idx: 'GFL99', orca_fallback_reason: 'no_filament_id',
- });
- const slotInfo = { ...defaultProps.slotInfo, savedPresetId: ORCA_ID };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText('Homebrew PLA @Orca')).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- expect(
- await screen.findByText(
- 'This Orca profile has no filament ID of its own, so OrcaSlicer will see this slot as Generic PLA.',
- ),
- ).toBeInTheDocument();
- });
- it('warns when a manual K value could not be sent (FW-C5)', async () => {
- // The route used to report the slot configured either way.
- const ORCA_ID = '3f2a9c1e-4b7d-4a02-9f61-8c5e2d1a7b30';
- (api.orcaCloudListProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament: [{ setting_id: ORCA_ID, name: 'Homebrew PLA @Orca', type: 'filament', is_custom: true }],
- printer: [],
- process: [],
- });
- (api.configureAmsSlot as ReturnType<typeof vi.fn>).mockResolvedValue({
- success: true, tray_info_idx: 'P56e1be0', orca_fallback_reason: '', k_value_unsent: true,
- });
- const slotInfo = { ...defaultProps.slotInfo, savedPresetId: ORCA_ID };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText('Homebrew PLA @Orca')).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- expect(
- await screen.findByText('The filament was set, but the K value could not be sent to the printer.'),
- ).toBeInTheDocument();
- });
- it('uses the filament_id nested under setting when the envelope has none (#3003)', async () => {
- // Bambu Cloud returns a custom preset's own filament_id on the envelope for
- // some presets and inside the preset JSON for others. Only the envelope was
- // read, so presets of the second shape reached the slot as their inherited
- // base — the "Bambu profile instead of my profile" half of #3003.
- (api.getCloudSettingDetail as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament_id: null,
- base_id: 'GFSB99_07',
- name: '# Overture Matte PLA @BBL H2D',
- setting: { filament_id: 'P4d64437' },
- });
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'PFUScd84f663d2c2ef',
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText('# Overture Matte PLA @BBL H2D')).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- const payload = (api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3];
- // 8 characters, so the printer stores it whole and the slicer matches it.
- expect(payload.tray_info_idx).toBe('P4d64437');
- expect(payload.setting_id).toBe('PFUScd84f663d2c2ef');
- });
- it('uses cloud detail filament_id when present', async () => {
- (api.getCloudSettingDetail as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament_id: 'P285e239',
- base_id: 'GFSB99_07',
- name: '# Overture Matte PLA @BBL H2D',
- });
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'PFUScd84f663d2c2ef',
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText('# Overture Matte PLA @BBL H2D')).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- const payload = (api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3];
- expect(payload.tray_info_idx).toBe('P285e239');
- expect(payload.setting_id).toBe('PFUScd84f663d2c2ef');
- });
- it('sends short GF filament_id for Bambu GFS* presets (cloud detail not consulted)', async () => {
- // Bambu-provided presets (GFS*) convert the setting_id → filament_id locally.
- // The cloud detail endpoint must NOT be consulted for them; the rewrite that
- // fixed #1053 preserves this pre-existing shortcut.
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'GFSL05_09',
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText('Bambu PLA Basic @BBL X1C')).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- const payload = (api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3];
- expect(payload.tray_info_idx).toBe('GFL05');
- expect(payload.setting_id).toBe('GFSL05_09');
- expect(api.getCloudSettingDetail).not.toHaveBeenCalled();
- });
- it('sends no tray_info_idx when the cloud detail fetch fails (#3003)', async () => {
- // Network/5xx from /cloud/settings/{id} must not abort the configure flow.
- // It leaves no filament_id to send, and the PFUS default is not a usable
- // stand-in (see the filament_id: null case above), so the field goes out
- // empty and the backend resolves it.
- (api.getCloudSettingDetail as ReturnType<typeof vi.fn>).mockRejectedValue(
- new Error('cloud unreachable')
- );
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'PFUScd84f663d2c2ef',
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText('# Overture Matte PLA @BBL H2D')).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- const payload = (api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3];
- expect(payload.tray_info_idx).toBe('');
- expect(payload.setting_id).toBe('PFUScd84f663d2c2ef');
- });
- it('renders configure slot button', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- expect(screen.getByText(/Configure AMS/)).toBeInTheDocument();
- });
- // Find the Configure Slot button
- const configureButton = screen.getByRole('button', { name: /Configure Slot/i });
- expect(configureButton).toBeInTheDocument();
- });
- it('filters presets by printer model', async () => {
- // Render with printerModel="H2D"
- render(<ConfigureAmsSlotModal {...defaultProps} printerModel="H2D" />);
- // Wait for presets to load - the H2D preset should be visible
- await waitFor(() => {
- expect(screen.getByText(/Overture Matte PLA/)).toBeInTheDocument();
- });
- // The X1C preset should NOT be visible (filtered out by model)
- expect(screen.queryByText(/Bambu PLA Basic @BBL X1C/)).not.toBeInTheDocument();
- });
- it('treats Bambu cloud rename @BBL A1M as a match for A1 Mini (#1649)', async () => {
- // Bambu cloud shifted A1 Mini filament profiles from
- // "Bambu PLA Basic @BBL A1 Mini ..." to the terse "@BBL A1M" mid-2026.
- // Without an alias-aware compare, the model filter strips every cloud
- // profile from the picker when the user selects an A1 Mini printer.
- (api.getCloudSettings as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament: [
- { setting_id: 'GFA00_A1M', name: 'Bambu PLA Basic @BBL A1M', filament_id: 'GFA00' },
- { setting_id: 'GFA00_A1', name: 'Bambu PLA Basic @BBL A1', filament_id: 'GFA00' },
- ],
- });
- render(<ConfigureAmsSlotModal {...defaultProps} printerModel="A1 Mini" />);
- await waitFor(() => {
- expect(screen.getByText('Bambu PLA Basic @BBL A1M')).toBeInTheDocument();
- });
- // The A1 (non-mini) preset must still be filtered out — the alias
- // table must not collapse two physically distinct printers.
- expect(screen.queryByText('Bambu PLA Basic @BBL A1')).not.toBeInTheDocument();
- });
- it('still filters cross-model cloud profiles when the printer is A1 Mini', async () => {
- // Sanity check that the alias addition didn't accidentally widen the
- // matcher: an X1C cloud preset stays hidden when the picker is for an
- // A1 Mini printer.
- render(<ConfigureAmsSlotModal {...defaultProps} printerModel="A1 Mini" />);
- await waitFor(() => {
- expect(screen.getByText(/Configure AMS/)).toBeInTheDocument();
- });
- expect(screen.queryByText('Bambu PLA Basic @BBL X1C')).not.toBeInTheDocument();
- });
- it('shows current preset even when it does not match model filter', async () => {
- // Render with printerModel="H2D" but savedPresetId pointing to the X1C preset
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'GFSL05_09', // X1C preset
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} printerModel="H2D" />);
- await waitFor(() => {
- // Both should be visible - H2D matches model, X1C is saved preset
- // Use the full preset name to match the list item (not the "Filtering for" label)
- expect(screen.getByText('Bambu PLA Basic @BBL X1C')).toBeInTheDocument();
- expect(screen.getByText(/Overture Matte PLA/)).toBeInTheDocument();
- });
- });
- it('preset row expands inline on hover so the full name is readable (#1237)', async () => {
- // Long preset names (e.g. "SUNLU PETG GLOW IN THE DARK GEN2 @Bambu Lab H2C 0.4 nozzle")
- // get visually truncated; the row un-truncates on hover via group-hover so the
- // nozzle suffix is readable without waiting on the browser's title-tooltip delay,
- // and the title attribute remains as a fallback for assistive tech / touch.
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- const fullName = 'Bambu PLA Basic @BBL X1C';
- const span = screen.getByText(fullName);
- expect(span).toHaveAttribute('title', fullName);
- expect(span).toHaveClass('truncate');
- expect(span).toHaveClass('group-hover:whitespace-normal');
- expect(span).toHaveClass('group-hover:break-all');
- expect(span.closest('button')).toHaveClass('group');
- });
- });
- it('pre-selects saved preset when opening configured slot', async () => {
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'GFSL05_09',
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- // The saved preset should have the selected style (green border)
- // Use the full preset name to avoid matching the "Filtering for" label
- const presetButton = screen.getByText('Bambu PLA Basic @BBL X1C').closest('button');
- expect(presetButton).toHaveClass('bg-bambu-green/20');
- });
- });
- it('pre-populates color from trayColor', async () => {
- const slotInfo = {
- ...defaultProps.slotInfo,
- trayColor: 'FF0000FF', // Red with alpha
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByTitle('White')).toBeInTheDocument();
- });
- // The hex display should show the pre-populated color
- expect(screen.getByText('Hex: #FF0000', { exact: false })).toBeInTheDocument();
- });
- it('uses translated text for modal elements', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} />);
- await waitFor(() => {
- expect(screen.getByText('Configure AMS Slot')).toBeInTheDocument();
- expect(screen.getByText('Filament Profile')).toBeInTheDocument();
- });
- // Check footer buttons
- expect(screen.getByRole('button', { name: /Configure Slot/i })).toBeInTheDocument();
- expect(screen.getByRole('button', { name: /Cancel/i })).toBeInTheDocument();
- expect(screen.getByRole('button', { name: /Reset Slot/i })).toBeInTheDocument();
- });
- it('surfaces a K-profile whose name does not match the preset when filament_id agrees (#1688)', async () => {
- // Spool was edited with slicer_filament = "GFSL05_09" (the setting_id form
- // for Bambu PLA Basic). The printer has a *custom* K-profile saved on the
- // same filament_id, but the user named it something that doesn't include
- // "PLA". Pre-fix, the name-only filter dropped it; the id-match path now
- // surfaces it because both sides normalise to "GFL05".
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- {
- slot_id: 3,
- extruder_id: 0,
- nozzle_id: 'HH00-0.4',
- nozzle_diameter: '0.4',
- filament_id: 'GFL05',
- name: 'my-custom-tune',
- k_value: '0.025',
- n_coef: '0',
- ams_id: 0,
- tray_id: 0,
- setting_id: '',
- },
- ],
- });
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'GFSL05_09', // setting_id form for Bambu PLA Basic
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- // Renders as an <option> on the K-profile select even though
- // "my-custom-tune" doesn't contain "PLA" anywhere.
- expect(screen.getByRole('option', { name: /my-custom-tune/ })).toBeInTheDocument();
- });
- });
- it("always includes the slot's currently-active K-profile when name and id don't match (#1689)", async () => {
- // Reporter scenario: spool assigned under "Generic PLA" but the slot has
- // a custom K-profile (filament_id "GFG98" = PETG-something) actively
- // selected via cali_idx. Pre-fix the modal showed "default 0.020"; the
- // safety net now surfaces the active profile so Configure Slot reflects
- // what the printer is actually using.
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- {
- slot_id: 7, // matches caliIdx below
- extruder_id: 0,
- nozzle_id: 'HH00-0.4',
- nozzle_diameter: '0.4',
- filament_id: 'GFG98', // unrelated to "Generic PLA"
- name: 'unrelated-petg-tune',
- k_value: '0.030',
- n_coef: '0',
- ams_id: 0,
- tray_id: 0,
- setting_id: '',
- },
- ],
- });
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'GFSL05_09', // Generic PLA preset
- caliIdx: 7,
- extruderId: 0,
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /unrelated-petg-tune/ })).toBeInTheDocument();
- });
- });
- it("does not carry the slot's K-profile over to a different filament (#3216)", async () => {
- // X1C slot on Devil Design PLA with its K-profile active, re-configured for
- // Azurefilm PLA Wood. The active profile used to join Azurefilm's list and
- // win the auto-select; the backend then realigned the slot to that
- // profile's filament, so the printer -- and OrcaSlicer -- stayed on Devil
- // Design. Only a slot reset first made it work.
- const DEVIL = '0ed901cd-6700-5772-80b2-25ed659df0d1';
- const AZURE = '2161df52-3d0e-579a-b4b6-dfbc62b93b76';
- (api.orcaCloudListProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- filament: [
- { setting_id: DEVIL, name: 'Devil Design PLA @Orca', type: 'filament', is_custom: true },
- { setting_id: AZURE, name: 'Azurefilm PLA Wood @Orca', type: 'filament', is_custom: true },
- ],
- printer: [],
- process: [],
- });
- const kp = (slot_id: number, name: string, filament_id: string, setting_id: string) => ({
- slot_id, extruder_id: 0, nozzle_id: 'HS00-0.4', nozzle_diameter: '0.4', filament_id, name,
- k_value: '0.020', n_coef: '0', ams_id: 0, tray_id: 0, setting_id,
- });
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- kp(7, 'Devil Design PLA', 'P4d64437', 'PFUSedbf16b803ff3e'),
- kp(9, 'Azurefilm PLA Wood', 'P285e239', 'PFUS220b72bdc39f90'),
- ],
- });
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: `orca_${DEVIL}`,
- trayInfoIdx: 'P4d64437',
- caliIdx: 7,
- extruderId: 0,
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- // Reopening on the slot's own filament still shows its active profile.
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /Devil Design PLA/ })).toBeInTheDocument();
- });
- fireEvent.click(screen.getByText('Azurefilm PLA Wood @Orca'));
- // Azurefilm's own profile is offered and picked; Devil Design's is not.
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /Azurefilm PLA Wood/ })).toBeInTheDocument();
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- const payload = (api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3];
- expect(payload.orca_profile_id).toBe(AZURE);
- expect(payload.kprofile_filament_id).toBe('P285e239');
- expect(payload.cali_idx).toBe(9);
- });
- it("surfaces the slot's active K-profile when no preset is resolvable (#1689 follow-up)", async () => {
- // Repro from Spionkiller01: slot is physically loaded but unconfigured —
- // tray_type='', tray_info_idx='', no slot_preset_mappings row — so
- // selectedPresetInfo resolves to null. Before the patch the main matcher's
- // early return on !selectedPresetInfo skipped past the cali_idx safety net
- // entirely; on reopen the dropdown went back to default 0.020 even though
- // the printer still holds the active profile at cali_idx=6.
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- {
- slot_id: 6,
- extruder_id: 0,
- nozzle_id: 'HH00-0.4',
- nozzle_diameter: '0.4',
- filament_id: 'GFG98',
- name: 'active-on-unconfigured-slot',
- k_value: '0.030',
- n_coef: '0',
- ams_id: 0,
- tray_id: 0,
- setting_id: '',
- },
- ],
- });
- const slotInfo = {
- ...defaultProps.slotInfo,
- trayType: '',
- traySubBrands: '',
- caliIdx: 6,
- extruderId: 0,
- // savedPresetId intentionally omitted — no preset bound yet
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /active-on-unconfigured-slot/ })).toBeInTheDocument();
- });
- });
- describe('Generic presets and the K-profile picker (#2710)', () => {
- // Reporter's printer: nine Flow-Dynamics entries, every one of them
- // calibrated against Generic PLA (filament_id GFL99) and named after the
- // spool's colour rather than its material. Bambu Studio lists all nine for
- // a Generic PLA slot; Bambuddy showed only the one already bound to the
- // slot via cali_idx.
- const genericPlaProfiles = [
- 'Black PLA+', 'Dark Brown', 'Glow', 'Gray', 'Lt Brown',
- 'Marble', 'Orange PLA', 'Sunlu White PLA+', 'White PLA+ Duramic',
- ].map((name, i) => ({
- slot_id: i + 1,
- extruder_id: 0,
- nozzle_id: 'HH00-0.4',
- nozzle_diameter: '0.4',
- filament_id: 'GFL99',
- name,
- k_value: `0.0${30 + i}`,
- n_coef: '0',
- ams_id: 0,
- tray_id: 0,
- setting_id: '',
- }));
- const genericPlaSlot = {
- ...defaultProps.slotInfo,
- savedPresetId: 'builtin_GFL99',
- extruderId: 0,
- };
- beforeEach(() => {
- (api.getBuiltinFilaments as ReturnType<typeof vi.fn>).mockResolvedValue([
- { filament_id: 'GFL99', name: 'Generic PLA', filament_type: 'PLA' },
- ]);
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: genericPlaProfiles,
- });
- });
- it('offers every Generic PLA profile when the slot preset is Generic PLA', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={genericPlaSlot} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /Dark Brown/ })).toBeInTheDocument();
- });
- // All nine, not just the one bound via cali_idx — matching the printer's
- // own calibration table for GFL99.
- for (const profile of genericPlaProfiles) {
- expect(
- screen.getByRole('option', { name: `[HF] ${profile.name} (K=${profile.k_value})` }),
- ).toBeInTheDocument();
- }
- });
- it('offers them on a freshly reset slot with no active cali_idx', async () => {
- // "When I reset the AMS slot, the generic PLA shows no k-values at all."
- // With no cali_idx the #1689 safety net has nothing to surface, so the
- // list came back empty; the id match has to stand on its own.
- render(
- <ConfigureAmsSlotModal
- {...defaultProps}
- slotInfo={{ ...genericPlaSlot, caliIdx: 0 }}
- />
- );
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /Marble/ })).toBeInTheDocument();
- });
- expect(screen.getByRole('option', { name: /Glow/ })).toBeInTheDocument();
- });
- it('matches on name when the profile carries no filament_id at all', async () => {
- // Not every firmware fills filament_id in extrusion_cali_get. "Generic"
- // is not a brand, so the name path must fall back to the material
- // instead of demanding "GENERIC" in the profile name.
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- { ...genericPlaProfiles[0], filament_id: '', name: 'Orange PLA' },
- { ...genericPlaProfiles[1], filament_id: '', name: 'Dark Brown' },
- ],
- });
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={genericPlaSlot} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /Orange PLA/ })).toBeInTheDocument();
- });
- });
- it('does not sweep generic profiles into a brand preset', async () => {
- // Guard against the id path widening: "Bambu PLA Basic" is GFL05, so
- // GFL99 profiles must still be filtered out of the matching group and
- // only reachable through the explicit "other profiles" group.
- render(
- <ConfigureAmsSlotModal
- {...defaultProps}
- slotInfo={{ ...defaultProps.slotInfo, savedPresetId: 'GFSL05_09', extruderId: 0 }}
- />
- );
- await waitFor(() => {
- expect(screen.getByText('Bambu PLA Basic @BBL X1C')).toBeInTheDocument();
- });
- // Every GFL99 profile is demoted to the other group: the brand gate on
- // "Bambu" still applies, and none of these names carry it.
- for (const profile of genericPlaProfiles) {
- const option = screen.getByRole('option', { name: `[HF] ${profile.name} (K=${profile.k_value})` });
- expect(option.closest('optgroup')).toHaveAttribute(
- 'label',
- 'Other K profiles on this printer',
- );
- }
- });
- it("offers the printer's other profiles even when nothing matches the preset", async () => {
- // The escape hatch: a PETG preset matches none of the PLA profiles, but
- // the user can still reach every profile the printer holds instead of
- // being sent to the slicer.
- (api.getBuiltinFilaments as ReturnType<typeof vi.fn>).mockResolvedValue([
- { filament_id: 'GFG99', name: 'Generic PETG', filament_type: 'PETG' },
- ]);
- render(
- <ConfigureAmsSlotModal
- {...defaultProps}
- slotInfo={{ ...genericPlaSlot, savedPresetId: 'builtin_GFG99', caliIdx: 0 }}
- />
- );
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /Dark Brown/ })).toBeInTheDocument();
- });
- expect(screen.getByRole('option', { name: /Dark Brown/ }).closest('optgroup')).toHaveAttribute(
- 'label',
- 'Other K profiles on this printer',
- );
- });
- it('sends the cali_idx of a profile picked from the other-profiles group', async () => {
- (api.getBuiltinFilaments as ReturnType<typeof vi.fn>).mockResolvedValue([
- { filament_id: 'GFG99', name: 'Generic PETG', filament_type: 'PETG' },
- ]);
- render(
- <ConfigureAmsSlotModal
- {...defaultProps}
- slotInfo={{ ...genericPlaSlot, savedPresetId: 'builtin_GFG99', caliIdx: 0 }}
- />
- );
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /Marble/ })).toBeInTheDocument();
- });
- const marble = genericPlaProfiles.find(p => p.name === 'Marble')!;
- // Option identity carries the extruder, so a filament calibrated on both
- // hotends yields two distinguishable entries rather than one.
- fireEvent.change(screen.getByRole('combobox'), {
- target: { value: `${marble.extruder_id}|${marble.name}|${marble.k_value}` },
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- const payload = (api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3];
- expect(payload.cali_idx).toBe(marble.slot_id);
- expect(payload.kprofile_filament_id).toBe('GFL99');
- });
- it('distinguishes two profiles that share a name but differ in K', async () => {
- // The picker used to key options by name alone, so same-named profiles
- // were indistinguishable and the first always won. The key now also
- // carries the extruder, for the same reason one step further out: the
- // printer numbers its calibration table per hotend.
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- { ...genericPlaProfiles[0], slot_id: 1, name: 'PLA', k_value: '0.020' },
- { ...genericPlaProfiles[1], slot_id: 2, name: 'PLA', k_value: '0.045' },
- ],
- });
- render(
- <ConfigureAmsSlotModal
- {...defaultProps}
- slotInfo={{ ...genericPlaSlot, caliIdx: 0 }}
- />
- );
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /K=0.045/ })).toBeInTheDocument();
- });
- fireEvent.change(screen.getByRole('combobox'), { target: { value: '0|PLA|0.045' } });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- expect((api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3].cali_idx).toBe(2);
- });
- });
- describe('A slot whose extruder claims no profile (#3044)', () => {
- // Reporter's X2D: two AMS 2 Pro, one per hotend, the same filaments in
- // both. The printer files one profile per filament rather than one per
- // hotend, so the second AMS's slots point at entries tagged extruder 0.
- // Scoping the picker to the slot's own extruder left it with nothing:
- // the slot read as unconfigured and choosing a profile changed nothing
- // the user could see.
- const sharedProfiles = [
- { name: 'PLA', k_value: '0.021', slot_id: 1 },
- { name: 'PETG', k_value: '0.026', slot_id: 4 },
- ].map(p => ({
- ...p,
- extruder_id: 0,
- nozzle_id: 'HH00-0.4',
- nozzle_diameter: '0.4',
- filament_id: 'GFL99',
- n_coef: '0',
- ams_id: 0,
- tray_id: 0,
- setting_id: '',
- }));
- // AMS-B, right-hand hotend, already bound to the PLA profile the A-side
- // slots use.
- const rightHandSlot = {
- ...defaultProps.slotInfo,
- amsId: 1,
- savedPresetId: 'builtin_GFL99',
- extruderId: 1,
- caliIdx: 1,
- };
- beforeEach(() => {
- (api.getBuiltinFilaments as ReturnType<typeof vi.fn>).mockResolvedValue([
- { filament_id: 'GFL99', name: 'Generic PLA', filament_type: 'PLA' },
- ]);
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({ profiles: sharedProfiles });
- });
- it('offers the profiles as matches rather than an empty picker', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={rightHandSlot} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /PLA \(K=0.021\)/ })).toBeInTheDocument();
- });
- // Matches are the select's direct children; anything demoted to the
- // "Other K profiles" group sits inside an optgroup instead. Being merely
- // present is what the slot already had, and it read as unconfigured.
- const matches = Array.from(screen.getByRole('combobox').querySelectorAll(':scope > option'))
- .map(o => (o as HTMLOptionElement).value)
- .filter(Boolean);
- expect(matches).toEqual(['0|PLA|0.021', '0|PETG|0.026']);
- });
- it('shows the slot as already bound to its active profile', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={rightHandSlot} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /PLA \(K=0.021\)/ })).toBeInTheDocument();
- });
- // Not the "no K profile" placeholder, which is what the slot showed while
- // the active profile could not be found on this extruder.
- expect((screen.getByRole('combobox') as HTMLSelectElement).value).toBe('0|PLA|0.021');
- });
- it('still scopes to the slot own hotend when that hotend has its own profiles', async () => {
- // The H2C case behind the scoping: one filament calibrated on both
- // hotends, two profiles, one right answer per slot. The fallback is a
- // last resort and must not reopen this.
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- { ...sharedProfiles[0], extruder_id: 0, k_value: '0.020', slot_id: 1 },
- { ...sharedProfiles[0], extruder_id: 1, k_value: '0.018', slot_id: 2 },
- ],
- });
- render(
- <ConfigureAmsSlotModal
- {...defaultProps}
- slotInfo={{ ...rightHandSlot, caliIdx: 2 }}
- />
- );
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /K=0.018/ })).toBeInTheDocument();
- });
- expect((screen.getByRole('combobox') as HTMLSelectElement).value).toBe('1|PLA|0.018');
- // The left hotend's copy is still reachable, but under "Other".
- const other = screen.getByRole('option', { name: /K=0.020/ });
- expect(other.parentElement?.tagName).toBe('OPTGROUP');
- });
- it('offers the profiles of the fitted nozzle flow, the others under Other with their flow', async () => {
- // H2D with a standard nozzle fitted: the high-flow copy of the same
- // filament is not applied at assign time, so it must not be the match
- // the picker selects (FD-16).
- (api.getPrinterStatus as ReturnType<typeof vi.fn>).mockResolvedValue({
- nozzles: [{ nozzle_type: 'HS01', nozzle_diameter: '0.4' }],
- });
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- { ...sharedProfiles[0], extruder_id: 0, nozzle_id: 'HH00-0.4', k_value: '0.030', slot_id: 1 },
- { ...sharedProfiles[0], extruder_id: 0, nozzle_id: 'HS00-0.4', k_value: '0.020', slot_id: 2 },
- ],
- });
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={{ ...rightHandSlot, extruderId: 0, caliIdx: null }} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /\[S\] PLA \(K=0.020\)/ })).toBeInTheDocument();
- });
- const matches = Array.from(screen.getByRole('combobox').querySelectorAll(':scope > option'))
- .map(o => (o as HTMLOptionElement).value)
- .filter(Boolean);
- expect(matches).toEqual(['0|PLA|0.020']);
- const highFlow = screen.getByRole('option', { name: /\[HF\] PLA \(K=0.030\)/ });
- expect(highFlow.parentElement?.tagName).toBe('OPTGROUP');
- });
- it('names each hotend once, not two or three times', async () => {
- // The label rendered kProfileNozzleSuffix twice in the matching group and
- // three times under "Other", so every option on a dual-nozzle printer
- // read "... . Left . Left".
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- { ...sharedProfiles[0], extruder_id: 0, k_value: '0.020', slot_id: 1 },
- { ...sharedProfiles[0], extruder_id: 1, k_value: '0.018', slot_id: 2 },
- ],
- });
- render(
- <ConfigureAmsSlotModal
- {...defaultProps}
- slotInfo={{ ...rightHandSlot, caliIdx: 2 }}
- />
- );
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /K=0.018/ })).toBeInTheDocument();
- });
- for (const option of screen.getAllByRole('option')) {
- // The suffix is the only thing that puts a middot in an option label,
- // so counting separators counts how many times the hotend was named.
- const separators = (option.textContent ?? '').match(/\u00b7/g) ?? [];
- expect(separators.length).toBeLessThanOrEqual(1);
- }
- });
- });
- it('does not include the active K-profile when caliIdx is 0 or null (#1689 guard)', async () => {
- // cali_idx == 0 / null means no profile is active (printer default 0.020).
- // The safety net only triggers for activeIdx > 0 — otherwise unrelated
- // profiles whose slot_id happens to equal 0 would leak in.
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [
- {
- slot_id: 0,
- extruder_id: 0,
- nozzle_id: 'HH00-0.4',
- nozzle_diameter: '0.4',
- filament_id: 'GFG98',
- name: 'should-not-appear',
- k_value: '0.030',
- n_coef: '0',
- ams_id: 0,
- tray_id: 0,
- setting_id: '',
- },
- ],
- });
- const slotInfo = {
- ...defaultProps.slotInfo,
- savedPresetId: 'GFSL05_09',
- caliIdx: 0,
- extruderId: 0,
- };
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slotInfo} />);
- await waitFor(() => {
- expect(screen.getByText(/Configure AMS/)).toBeInTheDocument();
- });
- expect(screen.queryByRole('option', { name: /should-not-appear/ })).not.toBeInTheDocument();
- });
- });
- describe('ConfigureAmsSlotModal — per-nozzle K-profiles', () => {
- /**
- * K-profiles are per-nozzle and the printer numbers its calibration table per
- * nozzle too, so one filament calibrated on both hotends gives two profiles
- * with the same name and the same index space. Measured on the maintainer's
- * H2C: "HF Bambu PLA Matte Black" is K=0.018 at index 16 on the left hotend
- * and K=0.020 at index 15 on the right.
- *
- * A tray holds exactly one cali_idx, so the picker has to resolve it against
- * the slot's own nozzle. It used to match on the index alone and, failing
- * that, take the first profile in the list — which on a Filament Track Switch
- * machine (where no AMS reports an extruder at all) was arbitrary.
- */
- const matteBlack = (extruder: number, caliIdx: number, k: string) => ({
- slot_id: caliIdx,
- extruder_id: extruder,
- nozzle_id: 'HH00-0.4',
- nozzle_diameter: '0.4',
- filament_id: 'GFL99',
- name: 'HF Bambu PLA Matte Black',
- k_value: k,
- n_coef: '0',
- ams_id: 0,
- tray_id: 0,
- setting_id: '',
- });
- const bothNozzles = [matteBlack(1, 16, '0.018'), matteBlack(0, 15, '0.020')];
- beforeEach(() => {
- vi.clearAllMocks();
- (api.getPrinterStatus as ReturnType<typeof vi.fn>).mockResolvedValue({});
- Element.prototype.scrollIntoView = vi.fn();
- (api.getCloudSettings as ReturnType<typeof vi.fn>).mockResolvedValue(mockCloudSettings);
- (api.getBuiltinFilaments as ReturnType<typeof vi.fn>).mockResolvedValue([
- { filament_id: 'GFL99', name: 'Generic PLA', filament_type: 'PLA' },
- ]);
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({ profiles: bothNozzles });
- (api.configureAmsSlot as ReturnType<typeof vi.fn>).mockResolvedValue({ success: true });
- });
- const slot = (extruderId: number | undefined, caliIdx: number) => ({
- ...defaultProps.slotInfo,
- savedPresetId: 'builtin_GFL99',
- extruderId,
- caliIdx,
- });
- it('preselects the right hotend profile for a slot on the right hotend', async () => {
- // The reported bug, from the other side: the slot is bound to cali_idx 16,
- // which is the LEFT hotend's entry. On a right-hotend slot the picker must
- // land on 15, not follow the index into the wrong table.
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slot(0, 16)} />);
- await waitFor(() => {
- expect(screen.getByRole('combobox')).toBeInTheDocument();
- });
- expect((screen.getByRole('combobox') as HTMLSelectElement).value).toBe('0|HF Bambu PLA Matte Black|0.020');
- });
- it('preselects the left hotend profile for a slot on the left hotend', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slot(1, 16)} />);
- await waitFor(() => {
- expect(screen.getByRole('combobox')).toBeInTheDocument();
- });
- expect((screen.getByRole('combobox') as HTMLSelectElement).value).toBe('1|HF Bambu PLA Matte Black|0.018');
- });
- it('does not offer the other hotend profile as a match', async () => {
- // Both are still reachable — the other lands in the "Other K profiles"
- // group — but only one is a match for this slot.
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slot(0, 15)} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /K=0.020/ })).toBeInTheDocument();
- });
- const matchGroup = screen.getByRole('combobox').querySelectorAll(':scope > option');
- const matchValues = Array.from(matchGroup).map(o => (o as HTMLOptionElement).value).filter(Boolean);
- expect(matchValues).toEqual(['0|HF Bambu PLA Matte Black|0.020']);
- });
- it('names the hotend on each option when the printer has two', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slot(0, 15)} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /K=0.020/ })).toBeInTheDocument();
- });
- expect(screen.getByRole('option', { name: /K=0\.020.*Right/ })).toBeInTheDocument();
- expect(screen.getByRole('option', { name: /K=0\.018.*Left/ })).toBeInTheDocument();
- });
- it('leaves single-nozzle printers unlabelled', async () => {
- (api.getKProfiles as ReturnType<typeof vi.fn>).mockResolvedValue({
- profiles: [matteBlack(0, 15, '0.020')],
- });
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slot(0, 15)} />);
- await waitFor(() => {
- expect(screen.getByRole('option', { name: /K=0.020/ })).toBeInTheDocument();
- });
- expect(screen.queryByRole('option', { name: /Right/ })).not.toBeInTheDocument();
- });
- it('sends the cali_idx of the nozzle-correct profile', async () => {
- render(<ConfigureAmsSlotModal {...defaultProps} slotInfo={slot(0, 16)} />);
- // Wait for the preselect to settle, not merely for the control to mount —
- // the submit reads the selected profile, which lands an effect later.
- await waitFor(() => {
- expect((screen.getByRole('combobox') as HTMLSelectElement).value).toBe(
- '0|HF Bambu PLA Matte Black|0.020'
- );
- });
- fireEvent.click(screen.getByRole('button', { name: /Configure Slot/i }));
- await waitFor(() => {
- expect(api.configureAmsSlot).toHaveBeenCalled();
- });
- expect((api.configureAmsSlot as ReturnType<typeof vi.fn>).mock.calls[0][3].cali_idx).toBe(15);
- });
- });
|