better test coverage

Author: joshday

src/PlotlyLight.jl

@@ -263,7 +263,10 @@ function Base.show(io::IO, M::MIME"text/html", o::Plot; setting::Settings = DEFA
 end
 
 #-----------------------------------------------------------------------------# collectrows
-collectrows(x::AbstractMatrix) = collect.(eachrow(x))
+function collectrows(x::AbstractMatrix)
+    Base.depwarn("collectrows is deprecated.  PlotlyLight now automatically applies this to matrices", :collectrows; force=true)
+    collect.(eachrow(x))
+end
 
 fix_matrix(x::Config) = Config(k => fix_matrix(v) for (k,v) in pairs(x))
 fix_matrix(x) = x

test/runtests.jl

@@ -77,4 +77,13 @@ end
 
     settings!(fix_matrix = false)
     @test occursin("[1,2,3,4]", html(p))
+
+    @test occursin("Settings", repr(settings!()))
+end
+
+#-----------------------------------------------------------------------------# other
+@testset "other" begin
+    @test occursin("{}", repr("text/html", PlotlyLight.page(Plot())))
+
+    @test_warn "deprecated" PlotlyLight.collectrows(rand(2,2))
 end