Watir::Container#frames(*args).to_a存在的bug
在做自动化脚本测试,通过@browser.frames(:xpath=>"//*[id='queryFrame']").to_a[0]查找一个iframe元素的时候,居然总是查找出一个不是我期待的iframe元素.一通追踪,终于发现上面的语句最终会等价于@browser.iframes.to_a[0]刚好我查找的iframe标签前面还有个iframe,所以结果就悲剧了!!
还好,现在找到了解决方案了.
相关的代码[源码与修改后的代码贴一起]:
# ../watir-webdriver-0.6.2/lib/watir-webdriver/elements/frame.rb
module Watir
##.....
module Container
##.....
def frames(*args)
FrameCollection.new(self, extract_selector(args).merge(:tag_name => /^(iframe|frame)$/)) # hack
end
##.....
end
##.....
class FrameCollection < ElementCollection
def to_a
#(0...elements.size).map { |idx| element_class.new @parent, :index => idx } #source code
(0...elements.size).map { |idx| element_class.new @parent, @selector.merge(:index => idx) } #fixed code
end
end
##.....
end
# ../watir-webdriver-0.6.2/lib/watir-webdriver/element_collection.rb
module Watir
#
# Base class for element collections.
#
class ElementCollection
include Enumerable
def initialize(parent, selector)
@parent = parent
@selector = selector
end
##.....
private
def elements
@elements ||= locator_class.new(
@parent.wd,
@selector,
element_class.attribute_list
).locate_all
end
##.....
end
end
从上面的代码可以分析@browser#iframes(*args).to_a的执行步骤:
1.@selector = *args ={:how=>what,...}
2.在to_a方法里,调用了elements方法,这里@selector首次被用于元素的一次查找,最终返回满足要求的元素集合.
3.在to_a方法里,根据查找到的元素集合的长度elements.size来二次查找,然后返回结果集,而抛弃了@selector过滤
就是说,上面的语句最终的返回集合是二次查找的结果,而二次查找没有根据给出的参数*args来查找.虽然结果集的个数是根据参数*args得出的.
为更加便于理解,这里例了个小例子(修改前):
#index.html
...
<iframe id="frame1" src="myframe1.html" class="c1">
</iframe>
<iframe id="frame2" src="myframe2.html" class="c2">
</iframe>
<iframe id="frame3" src="myframe3.html" class="c1">
</iframe>
...
#myframe1.html
<html>
<head>
</head>
<body>
<h1>Frame 1</h1>
</body>
</html>
#myframe2.html
<html>
<head>
</head>
<body>
<h1>Frame 2</h1>
</body>
</html>
#myframe3.html
<html>
<head>
</head>
<body>
<h1>Frame 3</h1>
</body>
</html>
备注:@browser#frames.each 同 @browser#frames.to_a.each
例1:执行
@browser.frames(:id=>"frame2").each{ |e| puts e.html } , 输出:
例2: 执行 @browser.frames(:class=>"c1").each{ |e| puts e.html } , 输出:
<html>
<head>
</head>
<body>
<h1>Frame 1</h1>
</body>
</html>
例2: 执行 @browser.frames(:class=>"c1").each{ |e| puts e.html } , 输出:
<html>
<head>
</head>
<body>
<h1>Frame 1</h1>
</body>
</html>
<html>
<head>
</head>
<body>
<h1>Frame 2</h1>
</body>
</html>
这里我只是贴出了一种解决方案,还没有深入的去寻找其他的解法.如果有哪位同仁有其他的方案,可以贴出来大家分享探讨.当然有哪儿写得不好或者出错的,欢迎批评指正!