Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Milan Bouchet-Valat <nalimilan@club.fr>
  • Loading branch information
bkamins and nalimilan committed Oct 29, 2020
1 parent 3e95470 commit b710e70
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions test/join.jl
Expand Up @@ -887,7 +887,7 @@ end
@test_throws ArgumentError join(df1, df2, on=:id, kind=:inner)
end

@testset "Make DataFrameJoiner rely on AbstractDataFrame" begin
@testset "join mixing DataFrame and SubDataFrame" begin
df1 = DataFrame(a=[1, 2, 3], b=[4, 5, 6])
df1_copy = df1[df1.a .> 1, :]
df1_view1 = @view df1[df1.a .> 1, :]
Expand All @@ -896,7 +896,6 @@ end
@test innerjoin(df1_copy, df2, on=:a) ==
innerjoin(df1_view1, df2, on=:a) ==
innerjoin(df1_view2, df2, on=:a)

end

end # module

0 comments on commit b710e70

Please sign in to comment.