KaotoIO / kaoto

The UI of the Kaoto project
https://kaoto.io
Apache License 2.0
37 stars 29 forks source link

Unexpected logs for `PasswordField.test.tsx` #1627

Closed lordrip closed 1 week ago

lordrip commented 1 week ago

Please describe the task that needs to be done

When running yarn workspace @kaoto/kaoto run test, there are unexpected logs for

src/components/Form/customField/PasswordField.test.tsx

Logs


 PASS  src/components/Form/customField/PasswordField.test.tsx
  ● Console

    console.error
      Warning: A component is changing an uncontrolled input to be controlled. This is likely caused by the value changing from undefined to a defined value, which should not happen. Decide between using a controlled or uncontrolled input element for the lifetime of the component. More info: https://reactjs.org/link/controlled-components
          at input
          at span
          at TextInputBase (/home/rmartinez/repos/kaoto/node_modules/@patternfly/react-core/src/components/TextInput/TextInput.tsx:118:5)
          at TextInput
          at div
          at InputGroupItem (/home/rmartinez/repos/kaoto/node_modules/@patternfly/react-core/src/components/InputGroup/InputGroupItem.tsx:21:3)
          at div
          at InputGroupBase (/home/rmartinez/repos/kaoto/node_modules/@patternfly/react-core/src/components/InputGroup/InputGroup.tsx:15:3)
          at InputGroup
          at div
          at div
          at GenerateId (/home/rmartinez/repos/kaoto/node_modules/@patternfly/react-core/dist/js/helpers/GenerateId/GenerateId.js:31:9)
          at FormGroup (/home/rmartinez/repos/kaoto/node_modules/@patternfly/react-core/src/components/Form/FormGroup.tsx:37:3)
          at onChange (/home/rmartinez/repos/kaoto/packages/ui/src/components/Form/customField/PasswordField.tsx:30:35)
          at name (/home/rmartinez/repos/kaoto/node_modules/uniforms/cjs/connectField.js:39:70)
          at form
          at AutoForm (/home/rmartinez/repos/kaoto/node_modules/uniforms/cjs/AutoForm.js:14:13)

      41 |     expect(input.getAttribute('type')).toEqual('password');
      42 |     await act(async () => {
    > 43 |       fireEvent.input(input, { target: { value: 'passwd' } });
         |                 ^
      44 |     });
      45 |     const toggle = screen.getByTestId('password-show-hide-button');
      46 |     await act(async () => {