Skip to content

Commit 52e5417

Browse files
authored
Merge branch 'trunk' into credential
2 parents 5255a43 + 44a87ed commit 52e5417

File tree

7 files changed

+30
-50
lines changed

7 files changed

+30
-50
lines changed

.github/ISSUE_TEMPLATE/bug-report.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,14 @@ body:
3535
label: Debugging Logs
3636
placeholder: |
3737
Note: the stack trace should be in the explanation section above
38-
Instructions for enabling logging can be found in the link below
38+
Instructions for enabling and referencing logs can be found in the links below
3939
render: logs
4040
- type: markdown
4141
id: link
4242
attributes:
4343
value: |
4444
**Read our [logging documentation](https://www.selenium.dev/documentation/webdriver/troubleshooting/logging/)**
45+
**Link to a [gist](https://gist.github.com/) if logs are too long**
4546
 
4647
4748
## Help us Address Your Issue Faster!

rb/.rubocop.yml

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,14 @@
1-
require:
2-
- rubocop-rake
3-
- rubocop-rspec
4-
51
plugins:
62
- rubocop-performance
3+
- rubocop-rake
4+
- rubocop-rspec
75

86
AllCops:
97
TargetRubyVersion: 3.1
108
NewCops: enable
119
Exclude:
1210
- !ruby/regexp /lib\/selenium\/devtools\/v\d+/
1311

14-
Capybara/RSpec/PredicateMatcher:
15-
Enabled: false
16-
17-
FactoryBot/CreateList:
18-
Enabled: false
19-
2012
Gemspec/DevelopmentDependencies:
2113
EnforcedStyle: gemspec
2214

@@ -155,10 +147,8 @@ Style/OptionalBooleanParameter:
155147
Enabled: false
156148

157149
Lint/Debugger:
158-
Enabled: true
159-
DebuggerMethods:
160-
Capybara:
161-
- save_screenshot: ~
150+
IgnoredMethods:
151+
- save_screenshot
162152

163153
Lint/UselessConstantScoping:
164154
Enabled: false

rb/Gemfile.lock

Lines changed: 10 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -134,25 +134,16 @@ GEM
134134
rubocop-ast (1.44.0)
135135
parser (>= 3.3.7.2)
136136
prism (~> 1.4)
137-
rubocop-capybara (2.22.1)
138-
lint_roller (~> 1.1)
139-
rubocop (~> 1.72, >= 1.72.1)
140-
rubocop-factory_bot (2.27.1)
141-
lint_roller (~> 1.1)
142-
rubocop (~> 1.72, >= 1.72.1)
143137
rubocop-performance (1.25.0)
144138
lint_roller (~> 1.1)
145139
rubocop (>= 1.75.0, < 2.0)
146140
rubocop-ast (>= 1.38.0, < 2.0)
147-
rubocop-rake (0.6.0)
148-
rubocop (~> 1.0)
149-
rubocop-rspec (2.31.0)
150-
rubocop (~> 1.40)
151-
rubocop-capybara (~> 2.17)
152-
rubocop-factory_bot (~> 2.22)
153-
rubocop-rspec_rails (~> 2.28)
154-
rubocop-rspec_rails (2.29.1)
155-
rubocop (~> 1.61)
141+
rubocop-rake (0.7.1)
142+
lint_roller (~> 1.1)
143+
rubocop (>= 1.72.1)
144+
rubocop-rspec (3.5.0)
145+
lint_roller (~> 1.1)
146+
rubocop (~> 1.72, >= 1.72.1)
156147
ruby-progressbar (1.13.0)
157148
rubyzip (2.4.1)
158149
securerandom (0.4.1)
@@ -209,10 +200,10 @@ DEPENDENCIES
209200
rack (~> 2.0)
210201
rake (~> 13.0)
211202
rspec (~> 3.0)
212-
rubocop (~> 1.60, >= 1.60.2)
213-
rubocop-performance (~> 1.15)
214-
rubocop-rake (~> 0.6.0)
215-
rubocop-rspec (~> 2.16)
203+
rubocop (~> 1.75)
204+
rubocop-performance (~> 1.25)
205+
rubocop-rake (~> 0.7)
206+
rubocop-rspec (~> 3.5)
216207
selenium-devtools!
217208
selenium-webdriver!
218209
steep (~> 1.5.0)

rb/lib/selenium/webdriver/common/options.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def safari(**opts)
5252
end
5353

5454
def set_capabilities
55-
(W3C_OPTIONS + self::CAPABILITIES.keys).each do |key|
55+
(W3C_OPTIONS + GRID_OPTIONS + self::CAPABILITIES.keys).each do |key|
5656
next if method_defined? key
5757

5858
define_method key do

rb/lib/selenium/webdriver/support/guards.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ def initialize(example, bug_tracker: '', conditions: nil)
4040
def add_condition(name, condition = false, &block)
4141
condition = false if condition.nil?
4242
@guard_conditions << GuardCondition.new(name, condition, &block)
43-
WebDriver.logger.info "Running with Guard '#{name}' set to: #{condition}"
43+
WebDriver.logger.debug "Running with Guard '#{name}' set to: #{condition}"
4444
end
4545

4646
def add_message(name, message)

rb/selenium-webdriver.gemspec

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,10 @@ Gem::Specification.new do |s|
5959
s.add_development_dependency 'rack', ['~> 2.0']
6060
s.add_development_dependency 'rake', ['~> 13.0']
6161
s.add_development_dependency 'rspec', ['~> 3.0']
62-
s.add_development_dependency 'rubocop', ['~> 1.60', '>=1.60.2']
63-
s.add_development_dependency 'rubocop-performance', ['~> 1.15']
64-
s.add_development_dependency 'rubocop-rake', ['~> 0.6.0']
65-
s.add_development_dependency 'rubocop-rspec', ['~> 2.16']
62+
s.add_development_dependency 'rubocop', ['~> 1.75']
63+
s.add_development_dependency 'rubocop-performance', ['~> 1.25']
64+
s.add_development_dependency 'rubocop-rake', ['~> 0.7']
65+
s.add_development_dependency 'rubocop-rspec', ['~> 3.5']
6666
s.add_development_dependency 'webmock', ['~> 3.5']
6767
s.add_development_dependency 'webrick', ['~> 1.7']
6868
s.add_development_dependency 'yard', ['~> 0.9.11', '>= 0.9.36']

rb/spec/unit/selenium/webdriver/common/print_options_spec.rb

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -17,42 +17,40 @@
1717
# specific language governing permissions and limitations
1818
# under the License.
1919

20-
2120
require File.expand_path('../spec_helper', __dir__)
2221
require 'selenium/webdriver/common/print_options'
2322

24-
2523
module Selenium
2624
module WebDriver
2725
describe PrintOptions do
28-
let(:options) { PrintOptions.new }
26+
let(:options) { described_class.new }
2927

3028
it 'has default values' do
3129
expect(options.orientation).to eq('portrait')
3230
expect(options.scale).to eq(1.0)
3331
expect(options.background).to be(false)
34-
expect(options.page_size).to eq({ width: 21.0, height: 29.7 })
35-
expect(options.margins).to eq({ top: 1.0, bottom: 1.0, left: 1.0, right: 1.0 })
32+
expect(options.page_size).to eq({width: 21.0, height: 29.7})
33+
expect(options.margins).to eq({top: 1.0, bottom: 1.0, left: 1.0, right: 1.0})
3634
end
3735

3836
it 'can set custom page size' do
39-
custom_size = { width: 25.0, height: 30.0 }
37+
custom_size = {width: 25.0, height: 30.0}
4038
options.page_size = custom_size
4139
expect(options.page_size).to eq(custom_size)
4240
end
4341

4442
it 'can set predefined page sizes using symbols' do
4543
options.page_size = :a4
46-
expect(options.page_size).to eq({ width: 21.0, height: 29.7 })
44+
expect(options.page_size).to eq({width: 21.0, height: 29.7})
4745

4846
options.page_size = :legal
49-
expect(options.page_size).to eq({ width: 21.59, height: 35.56 })
47+
expect(options.page_size).to eq({width: 21.59, height: 35.56})
5048

5149
options.page_size = :tabloid
52-
expect(options.page_size).to eq({ width: 27.94, height: 43.18 })
50+
expect(options.page_size).to eq({width: 27.94, height: 43.18})
5351

5452
options.page_size = :letter
55-
expect(options.page_size).to eq({ width: 21.59, height: 27.94 })
53+
expect(options.page_size).to eq({width: 21.59, height: 27.94})
5654
end
5755

5856
it 'raises an error for unsupported predefined page size symbols' do

0 commit comments

Comments
 (0)