capybara 2.15.0 → 3.0.0

data/lib/capybara/spec/session/window/window_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec Capybara::Window, requires: [:windows] do
   before(:each) do
     @window = @session.current_window
@@ -90,7 +91,7 @@ Capybara::SpecHelper.spec Capybara::Window, requires: [:windows] do
       @session.evaluate_script("[window.outerWidth || window.innerWidth, window.outerHeight || window.innerHeight]")
     end
 
-    it 'should return size of whole window', requires: [:windows, :js] do
+    it 'should return size of whole window', requires: %i[windows js] do
       expect(@session.current_window.size).to eq win_size
     end
 
@@ -108,15 +109,14 @@ Capybara::SpecHelper.spec Capybara::Window, requires: [:windows] do
   end
 
   describe '#resize_to' do
-    it 'should be able to resize window', requires: [:windows, :js] do
+    it 'should be able to resize window', requires: %i[windows js] do
       width, height = @session.current_window.size
-      @session.current_window.resize_to(width-100, height-100)
+      @session.current_window.resize_to(width - 100, height - 100)
       sleep 1
-      expect(@session.current_window.size).to eq([width-100, height-100])
+      expect(@session.current_window.size).to eq([width - 100, height - 100])
     end
 
-    it 'should stay on current window if invoked not for current window', requires: [:windows, :js] do
-
+    it 'should stay on current window if invoked not for current window', requires: %i[windows js] do
       @other_window = @session.window_opened_by do
         @session.find(:css, '#openWindow').click
       end
@@ -125,35 +125,35 @@ Capybara::SpecHelper.spec Capybara::Window, requires: [:windows] do
 
       # #size returns values larger than availWidth, availHeight with Chromedriver
       @session.within_window(@other_window) do
-        expect(@session.current_window.size).to eq([400,300])
+        expect(@session.current_window.size).to eq([400, 300])
         # expect(@session.evaluate_script("[window.outerWidth, window.outerHeight]")).to eq([400,300])
       end
     end
   end
 
   describe '#maximize' do
-    it 'should be able to maximize window', requires: [:windows, :js] do
+    it 'should be able to maximize window', requires: %i[windows js] do
       start_width, start_height = 400, 300
       @session.current_window.resize_to(start_width, start_height)
       sleep 0.5
 
       @session.current_window.maximize
-      sleep 0.5  # The timing on maximize is finicky on Travis -- wait a bit for maximize to occur
+      sleep 0.5 # The timing on maximize is finicky on Travis -- wait a bit for maximize to occur
 
       max_width, max_height = @session.current_window.size
 
-      #maximize behavior is window manage dependant, so just make sure it increases in size
+      # maximize behavior is window manage dependant, so just make sure it increases in size
       expect(max_width).to be > start_width
       expect(max_height).to be > start_height
     end
 
-    it 'should stay on current window if invoked not for current window', requires: [:windows, :js] do
+    it 'should stay on current window if invoked not for current window', requires: %i[windows js] do
       cur_window_size = @session.current_window.size
       @other_window = @session.window_opened_by do
         @session.find(:css, '#openWindow').click
       end
 
-      @other_window.resize_to(400,300)
+      @other_window.resize_to(400, 300)
       sleep 0.5
       @other_window.maximize
       sleep 0.5 # The timing on maximize is finicky on Travis -- wait a bit for maximize to occur

data/lib/capybara/spec/session/window/windows_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#windows', requires: [:windows] do
   before(:each) do
     @window = @session.current_window
@@ -25,8 +26,6 @@ Capybara::SpecHelper.spec '#windows', requires: [:windows] do
     titles = @session.windows.map do |window|
       @session.within_window(window) { @session.title }
     end
-    expect(titles).to match_array([
-      'With Windows', 'Title of the first popup', 'Title of popup two'
-    ])
+    expect(titles).to match_array(['With Windows', 'Title of the first popup', 'Title of popup two'])
   end
 end

data/lib/capybara/spec/session/window/within_window_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
   before(:each) do
     @window = @session.current_window
@@ -19,16 +20,15 @@ Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
 
   context "with an instance of Capybara::Window" do
     it "should not invoke driver#switch_to_window when given current window" do
-      # switch_to_window is invoked in after hook
-      expect(@session.driver).to receive(:switch_to_window).exactly(3).times.and_call_original
+      allow(@session.driver).to receive(:switch_to_window).and_call_original
       @session.within_window @window do
         expect(@session.title).to eq('With Windows')
       end
+      expect(@session.driver).not_to have_received(:switch_to_window)
     end
 
     it "should be able to switch to another window" do
       window = (@session.windows - [@window]).first
-      expect(@session.driver).to receive(:switch_to_window).exactly(5).times.and_call_original
       @session.within_window window do
         expect(@session).to have_title(/Title of the first popup|Title of popup two/)
       end
@@ -38,8 +38,8 @@ Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
     it "returns value from the block" do
       window = (@session.windows - [@window]).first
       value = @session.within_window window do
-                43252003274489856000
-              end
+        43252003274489856000
+      end
       expect(value).to eq(43252003274489856000)
     end
 
@@ -57,7 +57,7 @@ Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
       expect(@session.send(:scopes)).to eq([nil])
     end
 
-    it "should leave correct scopes after execution in case of error", requires: [:windows, :frames] do
+    it "should leave correct scopes after execution in case of error", requires: %i[windows frames] do
       window = (@session.windows - [@window]).first
       expect do
         @session.within_frame 'frameOne' do
@@ -87,25 +87,25 @@ Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
 
   context "with lambda" do
     it "should find the div in another window" do
-      @session.within_window(->{ @session.title == 'Title of the first popup'}) do
+      @session.within_window(-> { @session.title == 'Title of the first popup' }) do
         expect(@session).to have_css('#divInPopupOne')
       end
     end
 
     it "should find divs in both windows" do
-      @session.within_window(->{ @session.title == 'Title of popup two'}) do
+      @session.within_window(-> { @session.title == 'Title of popup two' }) do
         expect(@session).to have_css('#divInPopupTwo')
       end
-      @session.within_window(->{ @session.title == 'Title of the first popup'}) do
+      @session.within_window(-> { @session.title == 'Title of the first popup' }) do
         expect(@session).to have_css('#divInPopupOne')
       end
       expect(@session.title).to eq('With Windows')
     end
 
     it "should be able to nest within_window" do
-      @session.within_window(->{ @session.title == 'Title of popup two'}) do
+      @session.within_window(-> { @session.title == 'Title of popup two' }) do
         expect(@session).to have_css('#divInPopupTwo')
-        @session.within_window(->{ @session.title == 'Title of the first popup'}) do
+        @session.within_window(-> { @session.title == 'Title of the first popup' }) do
           expect(@session).to have_css('#divInPopupOne')
         end
         expect(@session).to have_css('#divInPopupTwo')
@@ -119,7 +119,7 @@ Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
     it "should work inside a normal scope" do
       expect(@session).to have_css('#openWindow')
       @session.within(:css, '#scope') do
-        @session.within_window(->{ @session.title == 'Title of the first popup'}) do
+        @session.within_window(-> { @session.title == 'Title of the first popup' }) do
           expect(@session).to have_css('#divInPopupOne')
         end
         expect(@session).to have_content('My scoped content')
@@ -129,7 +129,7 @@ Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
 
     it "should raise error if window wasn't found" do
       expect do
-        @session.within_window(->{ @session.title == 'Invalid title'}) do
+        @session.within_window(-> { @session.title == 'Invalid title' }) do
           expect(@session).to have_css('#divInPopupOne')
         end
       end.to raise_error(Capybara::WindowError, "Could not find a window matching block/lambda")
@@ -139,15 +139,13 @@ Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
     end
 
     it "returns value from the block" do
-      value = @session.within_window(->{ @session.title == 'Title of popup two'}) do
-                42
-              end
+      value = @session.within_window(-> { @session.title == 'Title of popup two' }) { 42 }
       expect(value).to eq(42)
     end
 
     it "should switch back if exception was raised inside block" do
       expect do
-        @session.within_window(->{ @session.title == 'Title of popup two'}) do
+        @session.within_window(-> { @session.title == 'Title of popup two' }) do
           raise 'some error'
         end
       end.to raise_error(StandardError, 'some error')
@@ -155,58 +153,4 @@ Capybara::SpecHelper.spec '#within_window', requires: [:windows] do
       expect(@session.send(:scopes)).to eq([nil])
     end
   end
-
-  context "with string" do
-    it "should warn" do
-      expect(@session).to receive(:warn).with(/DEPRECATION WARNING/).and_call_original
-      @session.within_window('firstPopup') {}
-    end
-
-    it "should find window by handle" do
-      window = (@session.windows - [@window]).first
-      @session.within_window window.handle do
-        expect(@session).to have_title(/Title of the first popup|Title of popup two/)
-      end
-    end
-
-    it "should find the div in firstPopup" do
-      @session.within_window("firstPopup") do
-        expect(@session.find("//*[@id='divInPopupOne']").text).to eq 'This is the text of divInPopupOne'
-      end
-    end
-    it "should find the div in secondPopup" do
-      @session.within_window("secondPopup") do
-        expect(@session.find("//*[@id='divInPopupTwo']").text).to eq 'This is the text of divInPopupTwo'
-      end
-    end
-    it "should find the divs in both popups" do
-      @session.within_window("secondPopup") do
-        expect(@session.find("//*[@id='divInPopupTwo']").text).to eq 'This is the text of divInPopupTwo'
-      end
-      @session.within_window("firstPopup") do
-        expect(@session.find("//*[@id='divInPopupOne']").text).to eq 'This is the text of divInPopupOne'
-      end
-    end
-    it "should find the div in the main window after finding a div in a popup" do
-      @session.within_window("secondPopup") do
-        expect(@session.find("//*[@id='divInPopupTwo']").text).to eq 'This is the text of divInPopupTwo'
-      end
-      expect(@session.find("//*[@id='doesNotOpenWindows']").text).to eq 'Does not open windows'
-    end
-    it "should reset scope when switching windows" do
-      @session.within(:css, '#doesNotOpenWindows') do
-        @session.within_window("secondPopup") do
-          expect(@session.find("//*[@id='divInPopupTwo']").text).to eq 'This is the text of divInPopupTwo'
-        end
-      end
-    end
-    it "should switch back if exception was raised inside block" do
-      expect do
-        @session.within_window('secondPopup') do
-          raise 'some error'
-        end
-      end.to raise_error(StandardError, 'some error')
-      expect(@session.current_window).to eq(@window)
-    end
-  end
 end

data/lib/capybara/spec/session/within_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#within' do
   before do
     @session.visit('/with_scope')

data/lib/capybara/spec/spec_helper.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require "rspec"
 require "rspec/expectations"
 require "capybara"
@@ -6,15 +7,6 @@ require "capybara/rspec" # Required here instead of in rspec_spec to avoid RSpec
 require "capybara/spec/test_app"
 require "nokogiri"
 
-# Alias be_truthy/be_falsey if not already defined to be able to use in RSpec 2 and 3
-unless RSpec::Matchers.method_defined?(:be_truthy)
-  RSpec::Matchers.module_eval do
-    alias be_truthy be_true
-    alias be_falsey be_false
-    alias be_falsy be_false
-  end
-end
-
 module Capybara
   module SpecHelper
     class << self
@@ -33,11 +25,9 @@ module Capybara
         Capybara.default_max_wait_time = 1
         Capybara.ignore_hidden_elements = true
         Capybara.exact = false
-        Capybara.exact_options = false
         Capybara.raise_server_errors = true
         Capybara.visible_text_only = false
         Capybara.match = :smart
-        Capybara.wait_on_first_by_default = false
         Capybara.enable_aria_label = false
         reset_threadsafe
       end
@@ -52,12 +42,12 @@ module Capybara
         end
       end
 
-      def spec(name, options={}, &block)
+      def spec(name, *options, &block)
         @specs ||= []
         @specs << [name, options, block]
       end
 
-      def run_specs(session, name, options={})
+      def run_specs(session, name, **options)
         specs = @specs
         RSpec.describe Capybara::Session, name, options do
           include Capybara::SpecHelper
@@ -78,8 +68,12 @@ module Capybara
             SpecHelper.reset_threadsafe(false, @session)
           end
 
+          before :each, :exact_false do
+            Capybara.exact = false
+          end
+
           specs.each do |spec_name, spec_options, block|
-            describe spec_name, spec_options do
+            describe spec_name, *spec_options do
               class_eval(&block)
             end
           end
@@ -92,7 +86,7 @@ module Capybara
         session = session.current_session if session.respond_to?(:current_session)
         session.instance_variable_set(:@config, nil) if session
       end
-    end # class << self
+    end
 
     def silence_stream(stream)
       old_stream = stream.dup
@@ -117,11 +111,35 @@ module Capybara
     end
 
     def marionette?(session)
-      session.driver.respond_to?(:marionette?, true) && session.driver.send(:marionette?)
+      session.respond_to?(:driver) && session.driver.respond_to?(:marionette?, true) && session.driver.send(:marionette?)
+    end
+
+    def marionette_lt?(version, session)
+      marionette?(session) && (session.driver.browser.capabilities[:browser_version].to_f < version)
+    end
+
+    def marionette_gte?(version, session)
+      marionette?(session) && (session.driver.browser.capabilities[:browser_version].to_f >= version)
+    end
+
+    def chrome?(session)
+      session.respond_to?(:driver) && session.driver.respond_to?(:chrome?, true) && session.driver.send(:chrome?)
+    end
+
+    def chrome_lt?(version, session)
+      chrome?(session) && (session.driver.browser.capabilities[:version].to_f < version)
+    end
+
+    def chrome_gte?(version, session)
+      chrome?(session) && (session.driver.browser.capabilities[:version].to_f >= version)
+    end
+
+    def edge?(session)
+      session.respond_to?(:driver) && session.driver.respond_to?(:edge?, true) && session.driver.send(:edge?)
     end
 
-    def rspec2?
-      !defined?(::RSpec::Expectations::Version) || (Gem::Version.new(RSpec::Expectations::Version::STRING) < Gem::Version.new('3.0'))
+    def ie?(session)
+      session.respond_to?(:driver) && session.driver.respond_to?(:ie?, true) && session.driver.send(:ie?)
     end
   end
 end

data/lib/capybara/spec/test_app.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require 'sinatra/base'
 require 'tilt/erb'
 require 'rack'
@@ -21,7 +22,7 @@ class TestApp < Sinatra::Base
   # Also check lib/capybara/spec/views/*.erb for pages not listed here
 
   get '/' do
-    response.set_cookie('capybara', { value: 'root cookie', domain: request.host, path: request.path} )
+    response.set_cookie('capybara', value: 'root cookie', domain: request.host, path: request.path)
     'Hello world! <a href="with_html">Relative</a>'
   end
 
@@ -38,8 +39,8 @@ class TestApp < Sinatra::Base
   end
 
   get '/referer_base' do
-    '<a href="/get_referer">direct link</a>' +
-    '<a href="/redirect_to_get_referer">link via redirect</a>' +
+    '<a href="/get_referer">direct link</a>' \
+    '<a href="/redirect_to_get_referer">link via redirect</a>' \
     '<form action="/get_referer" method="get"><input type="submit"></form>'
   end
 
@@ -69,7 +70,7 @@ class TestApp < Sinatra::Base
   end
 
   get '/with-quotes' do
-    %q{"No," he said, "you can't do that."}
+    %q("No," he said, "you can't do that.")
   end
 
   get '/form/get' do
@@ -143,13 +144,22 @@ class TestApp < Sinatra::Base
     erb :with_html, locals: { referrer: request.referrer }
   end
 
+  get '/with_title' do
+    <<-HTML
+      <title>#{params[:title] || 'Test Title'}</title>
+      <body>
+        <svg><title>abcdefg</title></svg>
+      </body>
+    HTML
+  end
+
   get '/:view' do |view|
     erb view.to_sym, locals: { referrer: request.referrer }
   end
 
   post '/form' do
     @@form_post_count += 1
-    '<pre id="results">' + params[:form].merge({"post_count" => @@form_post_count}).to_yaml + '</pre>'
+    '<pre id="results">' + params[:form].merge("post_count" => @@form_post_count).to_yaml + '</pre>'
   end
 
   post '/upload_empty' do
@@ -173,7 +183,7 @@ class TestApp < Sinatra::Base
 
   post '/upload_multiple' do
     begin
-      buffer = ["#{params[:form][:multiple_documents].size}"]
+      buffer = [params[:form][:multiple_documents].size.to_s]
       params[:form][:multiple_documents].each do |doc|
         buffer << "Content-type: #{doc[:type]}"
         buffer << "File content: #{doc[:tempfile].read}"
@@ -185,6 +195,4 @@ class TestApp < Sinatra::Base
   end
 end
 
-if __FILE__ == $0
-  Rack::Handler::WEBrick.run TestApp, Port: 8070
-end
+Rack::Handler::Puma.run TestApp, Port: 8070 if $PROGRAM_NAME == __FILE__

data/lib/capybara/spec/views/form.erb

@@ -186,6 +186,7 @@ New line after and before textarea tag
     <input type="checkbox" value="pagani" name="form[cars][]" id="form_cars_pagani" style="position: absolute; left: -9999px"/>
     <label for="form_cars_pagani">Pagani</label>
     <input type="checkbox" value="ariel" name="form[cars][]" id="form_cars_ariel" style="display: none"/>
+    <input type="checkbox" value="porsche" name="form[cars][]" id="form_cars_porsche" checked="checked" style="display: none"/>
     <label>
       McLaren
       <input type="checkbox" value="mclaren" name="form[cars][]" id="form_cars_mclaren" style="display: none"/>
@@ -425,6 +426,12 @@ New line after and before textarea tag
     <input type="button" disabled="disabled" value="Disabled button"/>
   </p>
 
+  <p>
+    <input type="date" name="form[date]" id="form_date"/>
+    <input type="time" name="form[time]" id="form_time"/>
+    <input type="datetime-local" name="form[datetime]" id="form_datetime">
+  </p>
+
   <p>
     <input id="readonly" name="form[readonly_test]" readonly/>
     <input id="not_readonly" name="form[readonly_test]" />

data/lib/capybara/spec/views/initial_alert.erb

@@ -0,0 +1,10 @@
+<html xmlns="http://www.w3.org/1999/xhtml" lang="en">
+  <body>
+    <div>
+      Initial alert page
+    </div>
+    <script>
+      window.alert("Initial alert");
+    </script>
+  </body>
+</html>

data/lib/capybara/spec/views/with_fixed_header_footer.erb

@@ -0,0 +1,17 @@
+<html>
+<head>
+  <style>
+    header { height: 45px; position: fixed; top: 0; background-color: red; width: 100%;}
+    footer { height: 45px; position: fixed; bottom: 0; background-color: red; width: 100%;}
+    #main { margin: 45px;}
+    #tall { display: block; height: 2000px;}
+  </style>
+</head>
+<body>
+  <header>My headers</header>
+    <div id="main">
+      <div id="tall">A tall block</div>
+      <a href="/">Go to root</a>
+    </div>
+  <footer>My footer</footer>
+</body>

data/lib/capybara/spec/views/with_hover.erb

@@ -14,5 +14,10 @@
       Some text here so the wrapper has size
       <div class="hidden_until_hover">Here I am</div>
     </div>
+    <div style="display: block; height: 1000px; width: 100%"></div>
+    <div class="wrapper scroll_needed" >
+      Some text here so the wrapper has size
+      <div class="hidden_until_hover">Here I am</div>
+    </div>
   </body>
 </html>

data/lib/capybara/spec/views/with_html.erb

@@ -122,6 +122,10 @@ banana</textarea>
   <a id="link_blank_href" href="">Blank href</a>
 </div>
 
+<div id="uppercase" style="text-transform: uppercase;">
+  text here
+</div>
+
 <div id="ancestor3">
   Ancestor
   <div id="ancestor2">
@@ -146,4 +150,26 @@ banana</textarea>
     <div data-pre=true>Pre Sibling</div>
     <div data-post=true>Post Sibling</div>
   </div>
-</div>
+</div>
+
+<div id='1escape.me' class="2escape">needs escaping</div>
+
+<div id="normalized">
+  Some text<div>More   text</div>
+  <div> And more text</div>
+  Even more &nbsp;&nbsp; text
+
+  on multiple lines
+</div>
+
+<div id="non_visible_normalized" style="display: none">
+  Some text<div>More   text</div>
+  <div> And more text</div>
+  Even more &nbsp;&nbsp; text
+
+  on multiple lines
+</div>
+
+<div id="ws">
+&#x20;&#x1680;&#x2000;&#x2001;&#x2002; &#x2003;&#x2004;&nbsp;&#x2005; &#x2006;&#x2007;&#x2008;&#x2009;&#x200A;&#x202F;&#x205F;&#x3000;
+</div>

data/lib/capybara/spec/views/with_js.erb

@@ -96,6 +96,10 @@
       <a href="#" id="open-prompt">Open prompt</a>
     </p>
 
+    <p>
+      <a href="#" id="open-prompt-with-default">Open defaulted prompt</a>
+    </p>
+
     <p>
       <input id="disable-on-click"/>
     </p>
@@ -117,6 +121,13 @@
       <input type="file" id="hidden_file" style="opacity:0; display: none;">
     </p>
 
+    <div id="drag_scroll">
+      <p>This is a draggable element.</p>
+    </div>
+    <div id="drop_scroll">
+      <p>It should be dropped here.</p>
+    </div>
+
     <script type="text/javascript">
       // a javascript comment
       var aVar = 123;

data/lib/capybara/spec/views/within_frames.erb

@@ -4,7 +4,10 @@
         <title>With Frames</title>
     </head>
     <body>
-        <div id="divInMainWindow">This is the text for divInMainWindow</div>
+        <div id="divInMainWindow">
+          This is the text for divInMainWindow
+          <iframe src="/frame_parent" id="innerParentFrame"></iframe>
+        </div>
     <iframe src="/frame_one" id="frameOne" name="my frame one"></iframe>
     <iframe src="/frame_two" id="frameTwo"></iframe>
     <iframe src="/frame_parent" id="parentFrame"></iframe>

data/lib/capybara/version.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Capybara
-  VERSION = '2.15.0'
+  VERSION = '3.0.0'.freeze
 end

data/lib/capybara/window.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Capybara
   ##
   # The Window class represents a browser window.
@@ -101,7 +102,7 @@ module Capybara
     end
 
     def eql?(other)
-      other.kind_of?(self.class) && @session == other.session && @handle == other.handle
+      other.is_a?(self.class) && @session == other.session && @handle == other.handle
     end
     alias_method :==, :eql?
 
@@ -113,9 +114,9 @@ module Capybara
       "#<Window @handle=#{@handle.inspect}>"
     end
 
-    private
+  private
 
-    def wait_for_stable_size(seconds=session.config.default_max_wait_time)
+    def wait_for_stable_size(seconds = session.config.default_max_wait_time)
       res = yield if block_given?
       prev_size = size
       start_time = Capybara::Helpers.monotonic_time
@@ -125,16 +126,11 @@ module Capybara
         return res if cur_size == prev_size
         prev_size = cur_size
       end while (Capybara::Helpers.monotonic_time - start_time) < seconds
-      #TODO raise error in 3.0
-      #raise Capybara::WindowError, "Window size not stable."
-      warn "Window size not stable in #{seconds} seconds.  This will raise an exception in a future version of Capybara"
-      return res
+      raise Capybara::WindowError, "Window size not stable within #{seconds} seconds."
     end
 
     def raise_unless_current(what)
-      unless current?
-        raise Capybara::WindowError, "#{what} not current window is not possible."
-      end
+      raise Capybara::WindowError, "#{what} not current window is not possible." unless current?
     end
   end
 end