search

LearnBoost / tobi

Tobi: Expressive server-side functional testing with jQuery
408 stars 33 forks source link

Fix for selecting multiple select options test. #45

Closed mhemesath closed 13 years ago

mhemesath commented 13 years ago

Fix for issue: https://github.com/LearnBoost/tobi/issues/39. From what I can see, given the markup, the failure is what is supposed to happen. Multiple options can't be selected on a select box unless it is given the multiple attribute.

This test should never have passed, unless something in JSDOM changed to only allow multiple selection if the "multiple" attribute is present?

tj commented 13 years ago

hmm weird that might have been the case, or maybe it was working at one time and I updated jQuery or something :s

mhemesath commented 13 years ago

I dug through the history of that test file and didn't notice anything done to the test that would trigger it to fail. So yeah, its probably jQuery or JSDOM.