Skip to content

Commit

Permalink
Unignore tests for setting different cookies on different domain
Browse files Browse the repository at this point in the history
The issue has been fixed in ChromeDriver 90
https://bugs.chromium.org/p/chromedriver/issues/detail?id=3733
  • Loading branch information
p0deje committed Apr 21, 2021
1 parent dd8741a commit 25adcf1
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions rb/spec/integration/selenium/webdriver/manager_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -92,10 +92,7 @@ module WebDriver
}.to raise_error(Error::UnableToSetCookieError)
end

it 'should not allow setting on a different domain',
except: [{browser: %i[chrome edge],
reason: "https://bugs.chromium.org/p/chromedriver/issues/detail?id=3733"},
{browser: %i[safari safari_preview]}] do
it 'should not allow setting on a different domain', except: {browser: %i[safari safari_preview]} do
expect {
driver.manage.add_cookie name: 'domain',
value: 'different',
Expand Down

0 comments on commit 25adcf1

Please sign in to comment.