Skip to content

Add workaround for ConnectionStub's missing interface#1686

Merged
st0012 merged 3 commits intomasterfrom
fix-#1685
Jan 14, 2022
Merged

Add workaround for ConnectionStub's missing interface#1686
st0012 merged 3 commits intomasterfrom
fix-#1685

Conversation

@st0012
Copy link
Copy Markdown
Contributor

@st0012 st0012 commented Jan 14, 2022

ActionCable's ConnectionStub (for testing) doesn't implement the exact same interfaces as Connection::Base.
One thing that's missing is env. So calling connection.env direclty will fail in test environments when stub_connection is used.
See #1684 for more information.

Closes #1685.

ActionCable's ConnectionStub (for testing) doesn't implement the exact same interfaces as Connection::Base.
One thing that's missing is `env`. So calling `connection.env` direclty will fail in test environments when `stub_connection` is used.
See #1684 for more information.
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 14, 2022

Codecov Report

Merging #1686 (849f7a6) into master (6d312f8) will increase coverage by 0.09%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1686      +/-   ##
==========================================
+ Coverage   98.46%   98.55%   +0.09%     
==========================================
  Files         136      136              
  Lines        7667     7691      +24     
==========================================
+ Hits         7549     7580      +31     
+ Misses        118      111       -7     
Impacted Files Coverage Δ
sentry-rails/lib/sentry/rails/action_cable.rb 100.00% <100.00%> (+8.00%) ⬆️
...entry-rails/spec/sentry/rails/action_cable_spec.rb 100.00% <100.00%> (ø)
sentry-rails/lib/sentry/rails/railtie.rb 100.00% <0.00%> (+2.89%) ⬆️
sentry-ruby/lib/sentry/breadcrumb.rb 100.00% <0.00%> (+3.70%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d312f8...849f7a6. Read the comment docs.

@st0012 st0012 requested a review from sl0thentr0py January 14, 2022 12:47
@st0012 st0012 merged commit 7697af6 into master Jan 14, 2022
@st0012 st0012 deleted the fix-#1685 branch January 14, 2022 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

sentry-rails 4.9.0 breaks channels specs

3 participants