fix(document): do not track value on HTMLSelectElement (#989)

ph-fritsche · web-flow · commit 77a7fa840a16 · 2022-07-15T11:30:10.000+02:00
* fix(document): do not track `value` on `HTMLSelectElement`

* throw error when trying to track non-existent property
diff --git a/src/document/index.ts b/src/document/index.ts
@@ -1,5 +1,6 @@
 import {dispatchUIEvent} from '../event'
 import {Config} from '../setup'
+import {isElementType} from '../utils'
 import {prepareSelectionInterceptor} from './selection'
 import {prepareRangeTextInterceptor} from './setRangeText'
 import {
@@ -24,7 +25,7 @@ export function prepareDocument(document: Document) {
   document.addEventListener(
     'focus',
     e => {
-      const el = e.target as Node
+      const el = e.target as Element
 
       prepareElement(el)
     },
@@ -62,12 +63,12 @@ export function prepareDocument(document: Document) {
   document[isPrepared] = isPrepared
 }
 
-function prepareElement(el: Node | HTMLInputElement) {
+function prepareElement(el: Element) {
   if (el[isPrepared]) {
     return
   }
 
-  if ('value' in el) {
+  if (isElementType(el, ['input', 'textarea'])) {
     prepareValueInterceptor(el)
     prepareSelectionInterceptor(el)
     prepareRangeTextInterceptor(el)
diff --git a/src/document/interceptor.ts b/src/document/interceptor.ts
@@ -10,7 +10,7 @@ type Params<Prop> = Prop extends anyFunc ? Parameters<Prop> : [Prop]
 type ImplReturn<Prop> = Prop extends anyFunc ? Parameters<Prop> : Prop
 
 export function prepareInterceptor<
-  ElementType extends Node,
+  ElementType extends Element,
   PropName extends keyof ElementType,
 >(
   element: ElementType,
@@ -39,11 +39,14 @@ export function prepareInterceptor<
 
   const target = prototypeDescriptor?.set ? 'set' : 'value'
 
+  /* istanbul ignore if */
   if (
     typeof prototypeDescriptor?.[target] !== 'function' ||
     (prototypeDescriptor[target] as Interceptable)[Interceptor]
   ) {
-    return
+    throw new Error(
+      `Element ${element.tagName} does not implement "${String(propName)}".`,
+    )
   }
 
   function intercept(
diff --git a/src/document/value.ts b/src/document/value.ts
@@ -57,7 +57,9 @@ function sanitizeValue(
   return String(v)
 }
 
-export function prepareValueInterceptor(element: HTMLInputElement) {
+export function prepareValueInterceptor(
+  element: HTMLInputElement | HTMLTextAreaElement,
+) {
   prepareInterceptor(element, 'value', valueInterceptor)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,9 @@ function sanitizeValue(`
`57`	`57`	`return String(v)`
`58`	`58`	`}`
`59`	`59`
`60`		`-export function prepareValueInterceptor(element: HTMLInputElement) {`
	`60`	`+export function prepareValueInterceptor(`
	`61`	`+ element: HTMLInputElement \| HTMLTextAreaElement,`
	`62`	`+) {`
`61`	`63`	`prepareInterceptor(element, 'value', valueInterceptor)`
`62`	`64`	`}`
`63`	`65`