allow passing AbstractString to Between (#17)

allow passing AbstractString to Between

Author: bkamins

Project.toml

@@ -1,7 +1,7 @@
 name = "DataAPI"
 uuid = "9a962f9c-6df0-11e9-0e5d-c546b8b5ee8a"
 authors = ["quinnj <[email protected]>"]
-version = "1.1.0"
+version = "1.2.0"
 
 [compat]
 julia = "1"

src/DataAPI.jl

@@ -103,6 +103,10 @@ struct Between{T1 <: Union{Int, Symbol}, T2 <: Union{Int, Symbol}}
     last::T2
 end
 
+Between(x::AbstractString, y::AbstractString) = Between(Symbol(x), Symbol(y))
+Between(x::Union{Int, Symbol}, y::AbstractString) = Between(x, Symbol(y))
+Between(x::AbstractString, y::Union{Int, Symbol}) = Between(Symbol(x), y)
+
 """
     All(cols...)
 

test/runtests.jl

@@ -59,10 +59,10 @@ end
 
 @testset "Between" begin
 
-    for x in (1, :a), y in (1, :a)
+    for x in (1, :a, "a"), y in (1, :a, "a")
         b = DataAPI.Between(x, y)
-        @test b.first == x
-        @test b.last == y
+        @test b.first == (x isa Int ? x : Symbol(x))
+        @test b.last == (y isa Int ? y : Symbol(y))
     end
 
     @test_throws MethodError DataAPI.Between(true, 1)