-
Notifications
You must be signed in to change notification settings - Fork 125
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
Wrap join on with list #866
Conversation
@@ -2291,6 +2291,19 @@ defmodule Explorer.DataFrameTest do | |||
assert_raise ArgumentError, msg, fn -> DF.join(left, right, how: :inner_join) end | |||
end | |||
|
|||
test "with matching column indexes as single value" do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pulled the simplest test from within the existing tests to create a base case covering the expected behavior. Not sure if there are any edge cases worth covering, but this clearly demonstrates a red/green on the desired case from my end.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is enough! Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! ❤️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! Thank you!
@@ -2291,6 +2291,19 @@ defmodule Explorer.DataFrameTest do | |||
assert_raise ArgumentError, msg, fn -> DF.join(left, right, how: :inner_join) end | |||
end | |||
|
|||
test "with matching column indexes as single value" do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is enough! Thanks!
Overview
Use
List.wrap/1
to wrap the:on
option tojoin/3
allowing users to give a single column without raising a match error from the case withinjoin/3
.Fixes #865