capybara 2.7.0 → 3.35.3

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

@@ -1,113 +1,138 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#click_link_or_button' do
-  it "should click on a link" do
+  it 'should click on a link' do
     @session.visit('/with_html')
     @session.click_link_or_button('labore')
     expect(@session).to have_content('Bar')
   end
 
-  it "should click on a button" do
+  it 'should click on a button' do
     @session.visit('/form')
     @session.click_link_or_button('awe123')
     expect(extract_results(@session)['first_name']).to eq('John')
   end
 
-  it "should click on a button with no type attribute" do
+  it 'should click on a button with no type attribute' do
     @session.visit('/form')
     @session.click_link_or_button('no_type')
     expect(extract_results(@session)['first_name']).to eq('John')
   end
 
-  it "should be aliased as click_on" do
+  it 'should be aliased as click_on' do
     @session.visit('/form')
     @session.click_on('awe123')
     expect(extract_results(@session)['first_name']).to eq('John')
   end
 
-  it "should wait for asynchronous load", :requires => [:js] do
+  it 'should wait for asynchronous load', requires: [:js] do
+    Capybara.default_max_wait_time = 2
     @session.visit('/with_js')
     @session.click_link('Click me')
     @session.click_link_or_button('Has been clicked')
   end
 
-  it "casts to string" do
+  it 'casts to string' do
     @session.visit('/form')
-    @session.click_link_or_button(:'awe123')
+    @session.click_link_or_button(:awe123)
     expect(extract_results(@session)['first_name']).to eq('John')
   end
 
-  context "with :exact option" do
-    context "when `true`" do
-      it "clicks on approximately matching link" do
+  context 'with test_id' do
+    it 'should click on a button' do
+      Capybara.test_id = 'data-test-id'
+      @session.visit('/form')
+      @session.click_link_or_button('test_id_button')
+      expect(extract_results(@session)['first_name']).to eq('John')
+    end
+  end
+
+  context 'with :exact option' do
+    context 'when `false`' do
+      it 'clicks on approximately matching link' do
         @session.visit('/with_html')
-        @session.click_link_or_button('abore', :exact => false)
+        @session.click_link_or_button('abore', exact: false)
         expect(@session).to have_content('Bar')
       end
 
-      it "clicks on approximately matching button" do
+      it 'clicks on approximately matching button' do
         @session.visit('/form')
-        @session.click_link_or_button('awe')
+        @session.click_link_or_button('awe', exact: false)
         expect(extract_results(@session)['first_name']).to eq('John')
       end
     end
 
-    context "when `false`" do
-      it "does not click on link which matches approximately" do
+    context 'when `true`' do
+      it 'does not click on link which matches approximately' do
         @session.visit('/with_html')
-        msg = "Unable to find link or button \"abore\""
+        msg = 'Unable to find link or button "abore"'
         expect do
-          @session.click_link_or_button('abore', :exact => true)
+          @session.click_link_or_button('abore', exact: true)
         end.to raise_error(Capybara::ElementNotFound, msg)
       end
 
-      it "does not click on approximately matching button" do
+      it 'does not click on approximately matching button' do
         @session.visit('/form')
-        msg = "Unable to find link or button \"awe\""
+        msg = 'Unable to find link or button "awe"'
 
         expect do
-          @session.click_link_or_button('awe', :exact => true)
+          @session.click_link_or_button('awe', exact: true)
         end.to raise_error(Capybara::ElementNotFound, msg)
       end
     end
   end
 
   context "with a locator that doesn't exist" do
-    it "should raise an error" do
+    it 'should raise an error' do
       @session.visit('/with_html')
-      msg = "Unable to find link or button \"does not exist\""
+      msg = 'Unable to find link or button "does not exist"'
       expect do
         @session.click_link_or_button('does not exist')
       end.to raise_error(Capybara::ElementNotFound, msg)
     end
   end
 
-  context "with :disabled option" do
-    it "ignores disabled buttons when false" do
+  context 'with :disabled option' do
+    it 'ignores disabled buttons when false' do
       @session.visit('/form')
       expect do
-        @session.click_link_or_button('Disabled button', :disabled => false)
+        @session.click_link_or_button('Disabled button', disabled: false)
       end.to raise_error(Capybara::ElementNotFound)
     end
 
-    it "ignores disabled buttons by default" do
+    it 'ignores disabled buttons by default' do
       @session.visit('/form')
       expect do
         @session.click_link_or_button('Disabled button')
       end.to raise_error(Capybara::ElementNotFound)
     end
 
-    it "happily clicks on links which incorrectly have the disabled attribute" do
+    it 'happily clicks on links which incorrectly have the disabled attribute' do
       @session.visit('/with_html')
       @session.click_link_or_button('Disabled link')
-      expect(@session).to have_content("Bar")
+      expect(@session).to have_content('Bar')
     end
+  end
 
-    it "does nothing when button is disabled" do
-      @session.visit('/form')
+  it 'should return the element clicked' do
+    @session.visit('/with_html')
+    link = @session.find(:link, 'Blank Anchor')
+    expect(@session.click_link_or_button('Blank Anchor')).to eq link
+  end
+
+  context 'with enable_aria_label' do
+    it 'should click on link' do
+      @session.visit('/with_html')
       expect do
-        @session.click_link_or_button('Disabled button', :disabled => false)
-      end.to raise_error(Capybara::ElementNotFound)
+        @session.click_link_or_button('Go to simple', enable_aria_label: true)
+      end.not_to raise_error
     end
 
+    it 'should click on button' do
+      @session.visit('/form')
+      expect do
+        @session.click_link_or_button('Aria button', enable_aria_label: true)
+      end.not_to raise_error
+    end
   end
 end

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

@@ -1,182 +1,229 @@
 # frozen_string_literal: true
+
 Capybara::SpecHelper.spec '#click_link' do
   before do
     @session.visit('/with_html')
   end
 
-  it "should wait for asynchronous load", :requires => [:js] do
+  it 'should wait for asynchronous load', requires: [:js] do
+    Capybara.default_max_wait_time = 2
     @session.visit('/with_js')
     @session.click_link('Click me')
     @session.click_link('Has been clicked')
   end
 
-  it "casts to string" do
-    @session.click_link(:'foo')
+  it 'casts to string' do
+    @session.click_link(:foo)
     expect(@session).to have_content('Another World')
   end
 
-  it "raises any errors caught inside the server", :requires => [:server] do
-    quietly { @session.visit("/error") }
+  it 'raises any errors caught inside the server', requires: [:server] do
+    quietly { @session.visit('/error') }
     expect do
       @session.click_link('foo')
     end.to raise_error(TestApp::TestAppError)
   end
 
-  context "with id given" do
-    it "should take user to the linked page" do
+  context 'with id given' do
+    it 'should take user to the linked page' do
       @session.click_link('foo')
       expect(@session).to have_content('Another World')
     end
   end
 
-  context "with text given" do
-    it "should take user to the linked page" do
+  context 'with text given' do
+    it 'should take user to the linked page' do
       @session.click_link('labore')
       expect(@session).to have_content('Bar')
     end
 
-    it "should accept partial matches" do
+    it 'should accept partial matches', :exact_false do
       @session.click_link('abo')
       expect(@session).to have_content('Bar')
     end
   end
 
-  context "with title given" do
-    it "should take user to the linked page" do
+  context 'with title given' do
+    it 'should take user to the linked page' do
       @session.click_link('awesome title')
       expect(@session).to have_content('Bar')
     end
 
-    it "should accept partial matches" do
+    it 'should accept partial matches', :exact_false do
       @session.click_link('some titl')
       expect(@session).to have_content('Bar')
     end
   end
 
-  context "with alternative text given to a contained image" do
-    it "should take user to the linked page" do
+  context 'with alternative text given to a contained image' do
+    it 'should take user to the linked page' do
       @session.click_link('awesome image')
       expect(@session).to have_content('Bar')
     end
 
-    it "should accept partial matches" do
+    it 'should accept partial matches', :exact_false do
       @session.click_link('some imag')
       expect(@session).to have_content('Bar')
     end
   end
 
   context "with a locator that doesn't exist" do
-    it "should raise an error" do
-      msg = "Unable to find link \"does not exist\""
+    it 'should raise an error' do
+      msg = 'Unable to find link "does not exist"'
       expect do
         @session.click_link('does not exist')
       end.to raise_error(Capybara::ElementNotFound, msg)
     end
   end
 
-  context "with :href option given" do
-    it "should find links with valid href" do
-      @session.click_link('labore', :href => '/with_simple_html')
+  context 'with :href option given' do
+    it 'should find links with valid href' do
+      @session.click_link('labore', href: '/with_simple_html')
       expect(@session).to have_content('Bar')
     end
 
     it "should raise error if link wasn't found" do
-      expect { @session.click_link('labore', :href => 'invalid_href') }.to raise_error(Capybara::ElementNotFound)
+      expect { @session.click_link('labore', href: 'invalid_href') }.to raise_error(Capybara::ElementNotFound, /with href "invalid_href/)
     end
   end
 
-  context "with a regex :href option given"  do
-    it "should find a link matching an all-matching regex pattern" do
-      @session.click_link('labore', :href => /.+/)
+  context 'with a regex :href option given' do
+    it 'should find a link matching an all-matching regex pattern' do
+      @session.click_link('labore', href: /.+/)
       expect(@session).to have_content('Bar')
     end
 
-    it "should find a link matching an exact regex pattern" do
-      @session.click_link('labore', :href => /\/with_simple_html/)
+    it 'should find a link matching an exact regex pattern' do
+      @session.click_link('labore', href: %r{/with_simple_html})
       expect(@session).to have_content('Bar')
     end
 
-    it "should find a link matching a partial regex pattern" do
-      @session.click_link('labore', :href => /\/with_simple/)
+    it 'should find a link matching a partial regex pattern' do
+      @session.click_link('labore', href: %r{/with_simple})
       expect(@session).to have_content('Bar')
     end
 
     it "should raise an error if no link's href matched the pattern" do
-      expect { @session.click_link('labore', :href => /invalid_pattern/) }.to raise_error(Capybara::ElementNotFound)
-      expect { @session.click_link('labore', :href => /.+d+/) }.to raise_error(Capybara::ElementNotFound)
+      expect { @session.click_link('labore', href: /invalid_pattern/) }.to raise_error(Capybara::ElementNotFound, %r{with href matching /invalid_pattern/})
+      expect { @session.click_link('labore', href: /.+d+/) }.to raise_error(Capybara::ElementNotFound, /#{Regexp.quote "with href matching /.+d+/"}/)
+    end
+
+    context 'href: nil' do
+      it 'should not raise an error on links with no href attribute' do
+        expect { @session.click_link('No Href', href: nil) }.not_to raise_error
+      end
+
+      it 'should raise an error if href attribute exists' do
+        expect { @session.click_link('Blank Href', href: nil) }.to raise_error(Capybara::ElementNotFound, /with no href attribute/)
+        expect { @session.click_link('Normal Anchor', href: nil) }.to raise_error(Capybara::ElementNotFound, /with no href attribute/)
+      end
+    end
+
+    context 'href: false' do
+      it 'should not raise an error on links with no href attribute' do
+        expect { @session.click_link('No Href', href: false) }.not_to raise_error
+      end
+
+      it 'should not raise an error if href attribute exists' do
+        expect { @session.click_link('Blank Href', href: false) }.not_to raise_error
+        expect { @session.click_link('Normal Anchor', href: false) }.not_to raise_error
+      end
     end
   end
 
-  it "should follow relative links" do
+  it 'should follow relative links' do
     @session.visit('/')
     @session.click_link('Relative')
     expect(@session).to have_content('This is a test')
   end
 
-  it "should follow protocol relative links" do
+  it 'should follow protocol relative links' do
     @session.click_link('Protocol')
     expect(@session).to have_content('Another World')
   end
 
-  it "should follow redirects" do
+  it 'should follow redirects' do
     @session.click_link('Redirect')
     expect(@session).to have_content('You landed')
   end
 
-  it "should follow redirects back to itself" do
+  it 'should follow redirects back to itself' do
     @session.click_link('BackToMyself')
-    expect(@session).to have_css('#referrer', text: /\/with_html$/)
+    expect(@session).to have_css('#referrer', text: %r{/with_html$})
     expect(@session).to have_content('This is a test')
   end
 
-  it "should add query string to current URL with naked query string" do
+  it 'should add query string to current URL with naked query string' do
     @session.click_link('Naked Query String')
     expect(@session).to have_content('Query String sent')
   end
 
-  it "should do nothing on anchor links" do
-    @session.fill_in("test_field", :with => 'blah')
+  it 'should do nothing on anchor links' do
+    @session.fill_in('test_field', with: 'blah')
     @session.click_link('Normal Anchor')
-    expect(@session.find_field("test_field").value).to eq('blah')
+    expect(@session.find_field('test_field').value).to eq('blah')
     @session.click_link('Blank Anchor')
-    expect(@session.find_field("test_field").value).to eq('blah')
+    expect(@session.find_field('test_field').value).to eq('blah')
     @session.click_link('Blank JS Anchor')
-    expect(@session.find_field("test_field").value).to eq('blah')
+    expect(@session.find_field('test_field').value).to eq('blah')
   end
 
-  it "should do nothing on URL+anchor links for the same page" do
-    @session.fill_in("test_field", :with => 'blah')
+  it 'should do nothing on URL+anchor links for the same page' do
+    @session.fill_in('test_field', with: 'blah')
     @session.click_link('Anchor on same page')
-    expect(@session.find_field("test_field").value).to eq('blah')
+    expect(@session.find_field('test_field').value).to eq('blah')
   end
 
-  it "should follow link on URL+anchor links for a different page" do
+  it 'should follow link on URL+anchor links for a different page' do
     @session.click_link('Anchor on different page')
     expect(@session).to have_content('Bar')
   end
 
-  it "should follow link on anchor if the path has regex special characters" do
+  it 'should follow link on anchor if the path has regex special characters' do
     @session.visit('/with.*html')
     @session.click_link('Anchor on different page')
     expect(@session).to have_content('Bar')
   end
 
-  it "raise an error with links with no href" do
+  it 'should raise an error with links with no href' do
     expect do
       @session.click_link('No Href')
     end.to raise_error(Capybara::ElementNotFound)
   end
 
-  context "with :exact option" do
-    it "should accept partial matches when false" do
-      @session.click_link('abo', :exact => false)
+  context 'with :exact option' do
+    it 'should accept partial matches when false' do
+      @session.click_link('abo', exact: false)
       expect(@session).to have_content('Bar')
     end
 
-    it "should not accept partial matches when true" do
+    it 'should not accept partial matches when true' do
       expect do
-        @session.click_link('abo', :exact => true)
+        @session.click_link('abo', exact: true)
       end.to raise_error(Capybara::ElementNotFound)
     end
   end
+
+  context 'without locator' do
+    it 'uses options' do
+      @session.click_link(href: '/foo')
+      expect(@session).to have_content('Another World')
+    end
+  end
+
+  it 'should return element clicked' do
+    el = @session.find(:link, 'Normal Anchor')
+    expect(@session.click_link('Normal Anchor')).to eq el
+  end
+
+  it 'can download a file', requires: [:download] do
+    # This requires the driver used for the test to be configured
+    # to download documents with the mime type "text/csv"
+    download_file = File.join(Capybara.save_path, 'download.csv')
+    expect(File).not_to exist(download_file)
+    @session.click_link('Download Me')
+    sleep 2 # allow time for file to download
+    expect(File).to exist(download_file)
+    FileUtils.rm_rf download_file
+  end
 end