[js] use property instead of atom to get index of option element

titusfortner · titusfortner · commit bca66568e161 · 2023-09-01T11:41:17.000-05:00
diff --git a/javascript/node/selenium-webdriver/lib/select.js b/javascript/node/selenium-webdriver/lib/select.js
@@ -191,7 +191,7 @@ class Select {
     }
 
     for (let option of options) {
-      if ((await option.getAttribute('index')) === index.toString()) {
+      if ((await option.getProperty('index')) === index.toString()) {
         await this.setSelected(option)
       }
     }
@@ -415,7 +415,7 @@ class Select {
     }
 
     for (let option of options) {
-      if ((await option.getAttribute('index')) === index.toString()) {
+      if ((await option.getProperty('index')) === index.toString()) {
         if (await option.isSelected()) {
           await option.click()
         }

Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,7 @@ class Select {`
`191`	`191`	`}`
`192`	`192`
`193`	`193`	`for (let option of options) {`
`194`		`- if ((await option.getAttribute('index')) === index.toString()) {`
	`194`	`+ if ((await option.getProperty('index')) === index.toString()) {`
`195`	`195`	`await this.setSelected(option)`
`196`	`196`	`}`
`197`	`197`	`}`
`@@ -415,7 +415,7 @@ class Select {`
`415`	`415`	`}`
`416`	`416`
`417`	`417`	`for (let option of options) {`
`418`		`- if ((await option.getAttribute('index')) === index.toString()) {`
	`418`	`+ if ((await option.getProperty('index')) === index.toString()) {`
`419`	`419`	`if (await option.isSelected()) {`
`420`	`420`	`await option.click()`
`421`	`421`	`}`