SeleniumHQ / selenium-google-code-issue-archive

Archive, please see main selenium repo
https://github.com/seleniumhq/selenium
346 stars 194 forks source link

verifyAttribute regression in 1.0.9 (text case & disabled) #1088

Closed lukeis closed 8 years ago

lukeis commented 8 years ago

Originally reported on Google Code with ID 1088

Windows 7, Firefox 3.6.12, Selenium IDE http://xserve.openqa.org:8085/browse/IDE-IDE-25

Looks like where is one more regression in SelIDE 1.0.9.

How to reproduce bug:
1. Open verifyAttribute.htm in FF.
2. Run verifyAttribute.test in SelIDE 1.0.9 #25.

The result for me is:
    * [error] Actual value 'background-image: url("/resize/?idnews=1&v=poligonv&adminmode=1&width=180&height=105&ratio=0");'
did not match 'background-image: url("/resize/?idnews=*&v=poligonv*&adminMode=1&width=180&height=105&ratio=0");'
    * [error] Actual value 'true' did not match 'disabled' 

Both commands are broken. There are two problems: text case and boolean attribute.

In SelIDE 1.0.7 the test works fine.

I think this bug because it breaks backward compatibility. Can you check it, please?

Reported by dmiFedorenko on 2010-12-06 03:49:21


lukeis commented 8 years ago

Reported by Samit.Badle on 2010-12-06 10:19:25

lukeis commented 8 years ago
Hi,

Thanks! I will look into it. Sigh!

Cheers,
Samit

Reported by Samit.Badle on 2010-12-06 10:21:05

lukeis commented 8 years ago
Hi,

This is indeed the new behaviour of Selenium which should be consistent across all
of Selenium components. Here is the explaination:-

The style attribute is now forced into lowercase. Since this is behaviour across Selenium,
I will open another issue for clarification. As a workaround, if you convert your value
to lowercase, it will pass.

The following attribute values are now treated as boolean and will now return true
or false.

"async", "autofocus", "autoplay", "checked", "compact", "complete", "controls", "declare",
"defaultchecked", "defaultselected", "defer", "disabled", "draggable", "ended", "formnovalidate",
"hidden", "indeterminate", "iscontenteditable", "ismap", "itemscope", "loop", "multiple",
"muted", "nohref", "noresize", "noshade", "novalidate", "nowrap", "open", "paused",
"pubdate", "readonly", "required", "reversed", "scoped", "seamless", "seeking", "selected",
"spellcheck", "truespeed", "willvalidate"

This includes the disabled attribute. Since this is the way forward, I do not see any
way to still be backward compatible.

Cheers,
Samit

Reported by Samit.Badle on 2010-12-06 14:08:41

lukeis commented 8 years ago
I see. Ok, I'll will fix my tests according the new behaviour of Selenium.

Thank you.

Reported by dmiFedorenko on 2010-12-06 22:54:31

lukeis commented 8 years ago

Reported by luke.semerau on 2015-09-17 18:11:39