-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
irb: warn: can't alias context from irb_context. (3.5.0.beta4 and 3.5.0 final) #1645
Comments
That did not help - i know |
That warning is coming from IRB here: It's apparently happening because rspec-core, when loaded, defines Can you provide a repo that repros this? I tried reproing it in isolation and have failed so far. |
@myronmarston ah thanks, I'll try to create a repro the next days. Do you mean this line |
Correct, that's what I meant, but I just realized why that doesn't solve the problem here: because you have to opt-in to disabling monkey patching (instead of opting in to monkey patching), the monkey patches, including |
hey @marcusg could you provide us with a reproduction case for this issue? Just a repository we can clone that demonstrates the problem. With that we can debug :) |
@samphippen yes, of course. I'll add repro steps as soon as possible. |
I've created a repro app https://github.com/marcusg/rspec-rails-issue-1645
|
Repro confirmed. |
I've got the same warning after upgrading |
This reverts commit 994cb7a. The original fix was never shown to have fixed anything, and looks to have triggered #1645 and rspec/rspec-core#2301. Fixes #1645.
There's an attempted fix in #1678 -- can one of you who has ran into this issue give that a try? |
This reverts commit 994cb7a. The original fix was never shown to have fixed anything, and looks to have triggered #1645 and rspec/rspec-core#2301. Fixes #1645.
Can you merge this into the 3-5 branch as well? Sent from my iPhone
|
This reverts commit 994cb7a. The original fix was never shown to have fixed anything, and looks to have triggered #1645 and rspec/rspec-core#2301. Fixes #1645.
Done. |
Works for me with |
I'm still having issues with:
[EDIT] Just learned many other gems could cause this — I will investigate this further. |
This reverts commit 994cb7a. The original fix was never shown to have fixed anything, and looks to have triggered rspec#1645 and rspec/rspec-core#2301. Fixes rspec#1645.
Don’t require rspec_junit_formatter gem It’s only needed when running tests (for circleci). It messes with the console, when running `rails c` it causes this warning: ``` irb: warn: can't alias context from irb_context ``` This is related to rspec monkey_patching a `context` method on `Object` (rspec/rspec-rails#1645)
Don’t require rspec_junit_formatter gem It’s only needed when running tests (for circleci). It messes with the console, when running `rails c` it causes this warning: ``` irb: warn: can't alias context from irb_context ``` This is related to rspec monkey_patching a `context` method on `Object` (rspec/rspec-rails#1645)
Hi! This solve warning for me:
|
Also, |
P.S. Here's where the methods (including |
Happened via
|
Hey,
After upgrading from
3.5.0.beta3
to3.5.0.beta4
I receive a warning when using therails console
command like:Version beta3 works well, the warning is only visible in beta4.
The text was updated successfully, but these errors were encountered: