Wizard: Swap arch select
This replaces deprecated PF4 select with non-deprecated PF5 one.
This commit is contained in:
parent
019faf0ff9
commit
2127476f98
2 changed files with 29 additions and 12 deletions
|
|
@ -1,11 +1,13 @@
|
|||
import React, { ReactElement, useState } from 'react';
|
||||
|
||||
import { FormGroup } from '@patternfly/react-core';
|
||||
import {
|
||||
FormGroup,
|
||||
MenuToggle,
|
||||
MenuToggleElement,
|
||||
Select,
|
||||
SelectList,
|
||||
SelectOption,
|
||||
SelectVariant,
|
||||
} from '@patternfly/react-core/deprecated';
|
||||
} from '@patternfly/react-core';
|
||||
|
||||
import { ARCHES } from '../../../../constants';
|
||||
import { useAppDispatch, useAppSelector } from '../../../../store/hooks';
|
||||
|
|
@ -49,17 +51,34 @@ const ArchSelect = () => {
|
|||
return options;
|
||||
};
|
||||
|
||||
const onToggleClick = () => {
|
||||
setIsOpen(!isOpen);
|
||||
};
|
||||
|
||||
const toggle = (toggleRef: React.Ref<MenuToggleElement>) => (
|
||||
<MenuToggle
|
||||
ref={toggleRef}
|
||||
onClick={onToggleClick}
|
||||
isExpanded={isOpen}
|
||||
isFullWidth
|
||||
data-testid="arch_select"
|
||||
>
|
||||
{arch}
|
||||
</MenuToggle>
|
||||
);
|
||||
|
||||
return (
|
||||
<FormGroup isRequired={true} label="Architecture">
|
||||
<Select
|
||||
ouiaId="arch_select"
|
||||
variant={SelectVariant.single}
|
||||
onToggle={() => setIsOpen(!isOpen)}
|
||||
onSelect={setArch}
|
||||
selections={arch}
|
||||
isOpen={isOpen}
|
||||
selected={arch}
|
||||
onSelect={setArch}
|
||||
onOpenChange={(isOpen: boolean) => setIsOpen(isOpen)}
|
||||
toggle={toggle}
|
||||
shouldFocusToggleOnSelect
|
||||
>
|
||||
{setSelectOptions()}
|
||||
<SelectList>{setSelectOptions()}</SelectList>
|
||||
</Select>
|
||||
</FormGroup>
|
||||
);
|
||||
|
|
|
|||
|
|
@ -89,10 +89,8 @@ const selectCentos9 = async () => {
|
|||
|
||||
const openArchitectureMenu = async () => {
|
||||
const user = userEvent.setup();
|
||||
const releaseMenu = screen.getAllByRole('button', {
|
||||
name: /options menu/i,
|
||||
})[1];
|
||||
await waitFor(() => user.click(releaseMenu));
|
||||
const archMenu = screen.getByTestId('arch_select');
|
||||
await waitFor(() => user.click(archMenu));
|
||||
};
|
||||
|
||||
const selectX86_64 = async () => {
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue