Merge pull request teampoltergeist#448 from JonRowe/access_node_parents

lib/capybara/poltergeist/browser.rb

@@ -50,6 +50,10 @@ def title
       command 'title'
     end
 
+    def parents(page_id, id)
+      command 'parents', page_id, id
+    end
+
     def find(method, selector)
       result = command('find', method, selector)
       result['ids'].map { |id| [result['page_id'], id] }

lib/capybara/poltergeist/client/agent.coffee

@@ -83,6 +83,14 @@ class PoltergeistAgent.Node
   parentId: ->
     @agent.register(@element.parentNode)
 
+  parentIds: ->
+    ids = []
+    parent = @element.parentNode
+    while parent != document
+      ids.push @agent.register(parent)
+      parent = parent.parentNode
+    ids
+
   find: (method, selector) ->
     @agent.find(method, selector, @element)
 

lib/capybara/poltergeist/client/browser.coffee

@@ -132,6 +132,9 @@ class Poltergeist.Browser
   attributes: (page_id, id, name) ->
     this.sendResponse this.node(page_id, id).getAttributes()
 
+  parents: (page_id, id) ->
+    this.sendResponse this.node(page_id, id).parentIds()
+
   value: (page_id, id) ->
     this.sendResponse this.node(page_id, id).value()
 

lib/capybara/poltergeist/client/compiled/agent.js

@@ -157,6 +157,17 @@ PoltergeistAgent.Node = (function() {
     return this.agent.register(this.element.parentNode);
   };
 
+  Node.prototype.parentIds = function() {
+    var ids, parent;
+    ids = [];
+    parent = this.element.parentNode;
+    while (parent !== document) {
+      ids.push(this.agent.register(parent));
+      parent = parent.parentNode;
+    }
+    return ids;
+  };
+
   Node.prototype.find = function(method, selector) {
     return this.agent.find(method, selector, this.element);
   };

lib/capybara/poltergeist/client/compiled/browser.js

@@ -176,6 +176,10 @@ Poltergeist.Browser = (function() {
     return this.sendResponse(this.node(page_id, id).getAttributes());
   };
 
+  Browser.prototype.parents = function(page_id, id) {
+    return this.sendResponse(this.node(page_id, id).parentIds());
+  };
+
   Browser.prototype.value = function(page_id, id) {
     return this.sendResponse(this.node(page_id, id).value());
   };

lib/capybara/poltergeist/client/compiled/node.js

@@ -3,7 +3,7 @@ var __slice = [].slice;
 Poltergeist.Node = (function() {
   var name, _fn, _i, _len, _ref;
 
-  Node.DELEGATES = ['allText', 'visibleText', 'getAttribute', 'value', 'set', 'setAttribute', 'isObsolete', 'removeAttribute', 'isMultiple', 'select', 'tagName', 'find', 'getAttributes', 'isVisible', 'position', 'trigger', 'parentId', 'mouseEventTest', 'scrollIntoView', 'isDOMEqual', 'isDisabled', 'deleteText'];
+  Node.DELEGATES = ['allText', 'visibleText', 'getAttribute', 'value', 'set', 'setAttribute', 'isObsolete', 'removeAttribute', 'isMultiple', 'select', 'tagName', 'find', 'getAttributes', 'isVisible', 'position', 'trigger', 'parentId', 'parentIds', 'mouseEventTest', 'scrollIntoView', 'isDOMEqual', 'isDisabled', 'deleteText'];
 
   function Node(page, id) {
     this.page = page;

lib/capybara/poltergeist/client/node.coffee

@@ -3,7 +3,7 @@
 class Poltergeist.Node
   @DELEGATES = ['allText', 'visibleText', 'getAttribute', 'value', 'set', 'setAttribute', 'isObsolete',
                 'removeAttribute', 'isMultiple', 'select', 'tagName', 'find', 'getAttributes',
-                'isVisible', 'position', 'trigger', 'parentId', 'mouseEventTest',
+                'isVisible', 'position', 'trigger', 'parentId', 'parentIds', 'mouseEventTest',
                 'scrollIntoView', 'isDOMEqual', 'isDisabled', 'deleteText']
 
   constructor: (@page, @id) ->

lib/capybara/poltergeist/node.rb

@@ -26,6 +26,10 @@ def command(name, *args)
       end
     end
 
+    def parents
+      command(:parents).map { |parent_id| self.class.new(driver, page_id, parent_id) }
+    end
+
     def find(method, selector)
       command(:find_within, method, selector).map { |id| self.class.new(driver, page_id, id) }
     end

spec/integration/session_spec.rb

@@ -583,5 +583,11 @@
         'href' => '#', 'id' => 'my_link', 'class' => 'some_class', 'data' => 'rah!'
       )
     end
+
+    it "knows about its parents" do
+      @session.visit '/poltergeist/simple'
+      parents = @session.find(:css,'#nav').native.parents
+      expect(parents.map(&:tag_name)).to eq ['li','ul','body','html']
+    end
   end
 end

spec/support/views/simple.erb

@@ -5,6 +5,9 @@
   </head>
 
   <body>
+    <ul>
+      <li><a id="nav" href="/">Home</a></li>
+    </ul>
     <a href="/">Link</a>
 
     <p id="break">Foo<br>Bar</p>