should type be constrained by maxLength?

Is maxLength an attribute user-event should take into account?

If so, the following test should pass:

// __tests__/vue/type.js

test("maxlength", () => {
  const input = jest.fn();

  const maxlength = 10;

  const { getByTestId } = renderComponent('input',
    { input },
    { maxlength }
  );

  const text = "Hello, world!";
  userEvent.type(getByTestId("input"), text);

  expect(input).toHaveBeenCalledTimes(maxlength);
  expect(getByTestId("input")).toHaveProperty("value", text.slice(0, maxlength));
});

I’ll be happy to implement it if you think it makes sense.

[Gpx]

It looks like the browser doesn’t fire the events if the max length is reached: link. Since user-event is the one responsible for firing events I think the check should happen on our side rather than with JSDOM.

What do you think?

[weyert]

Sounds good to me. I can work on something tomorrow at work