-
Notifications
You must be signed in to change notification settings - Fork 556
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
undefined method "write" #999
Comments
@zsan thank you for reporting. Are you able to provide an example code to reproduce it or show more stacktrace to understand where this Would you mind trying the following two commits and see if it works on any of them? gem 'webmock', github: 'bblimke/webmock', branch: '102dd56' @rzane any thoughts? |
I have released 3.18.1 with reverted 2c10bd9 It would be good to get a test or sample code to reproduce the issue or at least a stacktrace from the error. |
@bblimke , I have a test branch that started failing with 3.18.0: Sample stacktrace:
|
Hello, I just put require "rails_helper"
RSpec.feature "LoginMethods", type: :feature, js: true do
before do
@google_user = create(:user, login_method: "external", provider: "google_oauth2")
OmniAuth.config.test_mode = true
end
scenario "reject internal login" do
visit new_user_session_path
byebug
fill_in "Email", with: @google_user.email
fill_in "Password", with: @google_user.password
click_button "Sign in"
expect(page).to have_content("This account is for login using Google account.")
end
end As you can see i put Just FYI, the error is gone when i tried with |
Version |
Sorry for the trouble everyone. I'll investigate further and try to figure out what went wrong here. The core problem seems to be that StubSocket isn't a very convincing stub of a socket. |
Hello,
Just FYI, i have this error message when i run
rspec
webmock's version
3.18.0
Downgraded to
3.17.1
solve the issueThe text was updated successfully, but these errors were encountered: