Skip to content

test(input): improve tests

Lasse Krabye requested to merge lakra/test-input-improve-tests into main

Har lagt til test-caser for å oppnå 100% dekning.

Fjernet ikon-klassene sds-input__icon--end og sds-input__icon--start som jeg ikke kunne finne andre referanser til enn testene. Som erstatning for klassene for å teste plasseringen av ikonet byttet jeg til sjekker via compareDocumentPosition.

Coverage forstår tydeligvis ikke følgende:

onChange?.(event.target.value, event);

Endret derfor dette til:

onChange && onChange(event.target.value, event);

Selv foretrekker jeg en litt mer lesbar men mer verbos if, men da eksisterende kode var kompakt antar jeg at dette er en preferanse.

Generelt spørsmål: Bumpes versjonsnummeret for pakka for hver endring som gjøres? Skal denne altså fra 1.0.1 til 1.0.2 med denne endringen?

Merge request reports