Skip to content
This repository was archived by the owner on Nov 30, 2024. It is now read-only.

fix ArgumentError: bad value for range #2515

Closed
wants to merge 1 commit into from
Closed

Conversation

Systho
Copy link
Contributor

@Systho Systho commented Feb 8, 2018

If the two arrays are the same, find() returns nil. This case must be accounted for

If the two arrays are the same, `find()` returns `nil`. This case must be accounted for
@myronmarston
Copy link
Member

Thanks for the fix! Can you add a spec for it? If you need to, you can call CommonBacktraceTruncater directly to test the behavior.

@benoittgt benoittgt mentioned this pull request Apr 24, 2018
@benoittgt
Copy link
Member

benoittgt commented Apr 24, 2018

I added test here: #2539

Thanks for the initial commit @Systho

xaviershay added a commit that referenced this pull request Apr 24, 2018
@xaviershay
Copy link
Member

Merged linked PR, closing this.

@xaviershay xaviershay closed this Apr 24, 2018
JonRowe added a commit that referenced this pull request Apr 25, 2018
MatheusRich pushed a commit to MatheusRich/rspec-core that referenced this pull request Oct 30, 2020
MatheusRich pushed a commit to MatheusRich/rspec-core that referenced this pull request Oct 30, 2020
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants