capybara 2.18.0 → 3.40.0

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

@@ -1,94 +1,127 @@
 # frozen_string_literal: true
-Capybara::SpecHelper.spec "#choose" do
+
+Capybara::SpecHelper.spec '#choose' do
   before do
     @session.visit('/form')
   end
 
-  it "should choose a radio button by id" do
-    @session.choose("gender_male")
+  it 'should choose a radio button by id' do
+    @session.choose('gender_male')
     @session.click_button('awesome')
     expect(extract_results(@session)['gender']).to eq('male')
   end
 
-  it "should choose a radio button by label" do
-    @session.choose("Both")
+  it 'ignores readonly attribute on radio buttons' do
+    @session.choose('gender_both')
+    @session.click_button('awesome')
+    expect(extract_results(@session)['gender']).to eq('both')
+  end
+
+  it 'should choose a radio button by label' do
+    @session.choose('Both')
     @session.click_button('awesome')
     expect(extract_results(@session)['gender']).to eq('both')
   end
 
-  it "casts to string" do
-    @session.choose("Both")
-    @session.click_button(:'awesome')
+  it 'should work without a locator string' do
+    @session.choose(id: 'gender_male')
+    @session.click_button('awesome')
+    expect(extract_results(@session)['gender']).to eq('male')
+  end
+
+  it 'should be able to choose self when no locator string specified' do
+    rb = @session.find(:id, 'gender_male')
+    rb.choose
+    @session.click_button('awesome')
+    expect(extract_results(@session)['gender']).to eq('male')
+  end
+
+  it 'casts to string' do
+    @session.choose('Both')
+    @session.click_button(:awesome)
     expect(extract_results(@session)['gender']).to eq('both')
   end
 
   context "with a locator that doesn't exist" do
-    it "should raise an error" do
-      msg = "Unable to find visible radio button \"does not exist\" that is not disabled"
+    it 'should raise an error' do
+      msg = /Unable to find radio button "does not exist"/
       expect do
         @session.choose('does not exist')
       end.to raise_error(Capybara::ElementNotFound, msg)
     end
   end
 
-  context "with a disabled radio button" do
-    it "should raise an error" do
+  context 'with a disabled radio button' do
+    it 'should raise an error' do
       expect do
         @session.choose('Disabled Radio')
       end.to raise_error(Capybara::ElementNotFound)
     end
   end
 
-  context "with :exact option" do
-    it "should accept partial matches when false" do
-      @session.choose("Mal", exact:  false)
+  context 'with :exact option' do
+    it 'should accept partial matches when false' do
+      @session.choose('Mal', exact: false)
       @session.click_button('awesome')
       expect(extract_results(@session)['gender']).to eq('male')
     end
 
-    it "should not accept partial matches when true" do
+    it 'should not accept partial matches when true' do
       expect do
-        @session.choose("Mal", exact:  true)
+        @session.choose('Mal', exact: true)
       end.to raise_error(Capybara::ElementNotFound)
     end
   end
 
-  context "with `option` option" do
-    it "can check radio buttons by their value" do
-      @session.choose('form[gender]', option: "male")
+  context 'with `option` option' do
+    it 'can check radio buttons by their value' do
+      @session.choose('form[gender]', option: 'male')
       @session.click_button('awesome')
-      expect(extract_results(@session)['gender']).to eq("male")
+      expect(extract_results(@session)['gender']).to eq('male')
     end
 
-    it "should raise an error if option not found" do
+    it 'should alias `:with` option' do
+      @session.choose('form[gender]', with: 'male')
+      @session.click_button('awesome')
+      expect(extract_results(@session)['gender']).to eq('male')
+    end
+
+    it 'should raise an error if option not found' do
       expect do
-        @session.choose('form[gender]', option: "hermaphrodite")
+        @session.choose('form[gender]', option: 'hermaphrodite')
       end.to raise_error(Capybara::ElementNotFound)
     end
   end
 
-  context "with hidden radio buttons" do
-    context "with Capybara.automatic_label_click == true" do
+  context 'with hidden radio buttons' do
+    context 'with Capybara.automatic_label_click == true' do
       around do |spec|
         old_click_label, Capybara.automatic_label_click = Capybara.automatic_label_click, true
         spec.run
         Capybara.automatic_label_click = old_click_label
       end
 
-      it "should select by clicking the link if available" do
-        @session.choose("party_democrat")
+      it 'should select by clicking the label if available' do
+        @session.choose('party_democrat')
+        @session.click_button('awesome')
+        expect(extract_results(@session)['party']).to eq('democrat')
+      end
+
+      it 'should select self by clicking the label if no locator specified' do
+        cb = @session.find(:id, 'party_democrat', visible: :hidden)
+        cb.choose
         @session.click_button('awesome')
         expect(extract_results(@session)['party']).to eq('democrat')
       end
 
-      it "should raise error if not allowed to click label" do
-        expect{@session.choose("party_democrat", allow_label_click: false)}.to raise_error(Capybara::ElementNotFound, 'Unable to find visible radio button "party_democrat" that is not disabled')
+      it 'should raise error if not allowed to click label' do
+        expect { @session.choose('party_democrat', allow_label_click: false) }.to raise_error(Capybara::ElementNotFound, /Unable to find visible radio button "party_democrat"/)
       end
     end
   end
 
-  it "should return the chosen radio button" do
+  it 'should return the chosen radio button' do
     el = @session.find(:radio_button, 'gender_male')
-    expect(@session.choose("gender_male")).to eq el
+    expect(@session.choose('gender_male')).to eq el
   end
 end