Organize tests (#120)

* Organize tests

* Move 'rng' to runtests.jl

Author: eliascarv

test/colspec.jl

@@ -104,59 +104,3 @@
   @inferred TableTransforms.choose(colspec, vecnames)
   @inferred TableTransforms.choose(colspec, tupnames)
 end
-
-@testset "TableSelection" begin
-  a = rand(4000)
-  b = rand(4000)
-  c = rand(4000)
-  d = rand(4000)
-  e = rand(4000)
-  f = rand(4000)
-  t = Table(; a, b, c, d, e, f)
-
-  # Tables.jl interface
-  select = [:a, :b, :e]
-  newnames = select
-  s = TableTransforms.TableSelection(t, newnames, select)
-  @test Tables.istable(s) == true
-  @test Tables.columnaccess(s) == true
-  @test Tables.rowaccess(s) == false
-  @test Tables.columns(s) === s
-  @test Tables.columnnames(s) == [:a, :b, :e]
-  @test Tables.schema(s).names == (:a, :b, :e)
-  @test Tables.schema(s).types == (Float64, Float64, Float64)
-  @test Tables.materializer(s) == Tables.materializer(t)
-
-  # getcolumn
-  cols = Tables.columns(t)
-  @test Tables.getcolumn(s, :a) == Tables.getcolumn(cols, :a)
-  @test Tables.getcolumn(s, 1) == Tables.getcolumn(cols, 1)
-  @test Tables.getcolumn(s, 3) == Tables.getcolumn(cols, :e)
-
-  # selectin with renaming
-  select = [:c, :d, :f]
-  newnames = [:x, :y, :z]
-  s = TableTransforms.TableSelection(t, newnames, select)
-  @test Tables.columnnames(s) == [:x, :y, :z]
-  @test Tables.getcolumn(s, :x) == t.c
-  @test Tables.getcolumn(s, :y) == t.d
-  @test Tables.getcolumn(s, :z) == t.f
-
-  # row table
-  select = [:a, :b, :e]
-  newnames = select
-  rt = Tables.rowtable(t)
-  s = TableTransforms.TableSelection(rt, newnames, select)
-  cols = Tables.columns(rt)
-  @test Tables.getcolumn(s, :a) == Tables.getcolumn(cols, :a)
-  @test Tables.getcolumn(s, 1) == Tables.getcolumn(cols, 1)
-  @test Tables.getcolumn(s, 3) == Tables.getcolumn(cols, :e)
-
-  # throws
-  @test_throws AssertionError TableTransforms.TableSelection(t, [:a, :b, :z], [:a, :b, :z])
-  @test_throws AssertionError TableTransforms.TableSelection(t, [:x, :y, :z], [:c, :d, :k])
-  s = TableTransforms.TableSelection(t, [:a, :b, :e], [:a, :b, :e])
-  @test_throws ErrorException Tables.getcolumn(s, :f)
-  s = TableTransforms.TableSelection(t, [:x, :y, :z], [:c, :d, :f])
-  @test_throws ErrorException Tables.getcolumn(s, :c)
-end

test/runtests.jl

@@ -23,6 +23,10 @@ islinux = Sys.islinux()
 visualtests = !isCI || (isCI && islinux)
 datadir = joinpath(@__DIR__,"data")
 
+# using MersenneTwister for backward
+# compatibility with old Julia versions
+rng = MersenneTwister(42)
+
 # for functor tests in Functional testset
 struct Polynomial{T<:Real}
   coeffs::Vector{T}
@@ -43,6 +47,7 @@ testfiles = [
   "distributions.jl",
   "colspec.jl",
   "transforms.jl",
+  "tableselection.jl",
   "shows.jl"
 ]
 

test/tableselection.jl

@@ -0,0 +1,55 @@
+@testset "TableSelection" begin
+  a = rand(4000)
+  b = rand(4000)
+  c = rand(4000)
+  d = rand(4000)
+  e = rand(4000)
+  f = rand(4000)
+  t = Table(; a, b, c, d, e, f)
+
+  # Tables.jl interface
+  select = [:a, :b, :e]
+  newnames = select
+  s = TableTransforms.TableSelection(t, newnames, select)
+  @test Tables.istable(s) == true
+  @test Tables.columnaccess(s) == true
+  @test Tables.rowaccess(s) == false
+  @test Tables.columns(s) === s
+  @test Tables.columnnames(s) == [:a, :b, :e]
+  @test Tables.schema(s).names == (:a, :b, :e)
+  @test Tables.schema(s).types == (Float64, Float64, Float64)
+  @test Tables.materializer(s) == Tables.materializer(t)
+
+  # getcolumn
+  cols = Tables.columns(t)
+  @test Tables.getcolumn(s, :a) == Tables.getcolumn(cols, :a)
+  @test Tables.getcolumn(s, 1) == Tables.getcolumn(cols, 1)
+  @test Tables.getcolumn(s, 3) == Tables.getcolumn(cols, :e)
+
+  # selectin with renaming
+  select = [:c, :d, :f]
+  newnames = [:x, :y, :z]
+  s = TableTransforms.TableSelection(t, newnames, select)
+  @test Tables.columnnames(s) == [:x, :y, :z]
+  @test Tables.getcolumn(s, :x) == t.c
+  @test Tables.getcolumn(s, :y) == t.d
+  @test Tables.getcolumn(s, :z) == t.f
+
+  # row table
+  select = [:a, :b, :e]
+  newnames = select
+  rt = Tables.rowtable(t)
+  s = TableTransforms.TableSelection(rt, newnames, select)
+  cols = Tables.columns(rt)
+  @test Tables.getcolumn(s, :a) == Tables.getcolumn(cols, :a)
+  @test Tables.getcolumn(s, 1) == Tables.getcolumn(cols, 1)
+  @test Tables.getcolumn(s, 3) == Tables.getcolumn(cols, :e)
+
+  # throws
+  @test_throws AssertionError TableTransforms.TableSelection(t, [:a, :b, :z], [:a, :b, :z])
+  @test_throws AssertionError TableTransforms.TableSelection(t, [:x, :y, :z], [:c, :d, :k])
+  s = TableTransforms.TableSelection(t, [:a, :b, :e], [:a, :b, :e])
+  @test_throws ErrorException Tables.getcolumn(s, :f)
+  s = TableTransforms.TableSelection(t, [:x, :y, :z], [:c, :d, :f])
+  @test_throws ErrorException Tables.getcolumn(s, :c)
+end