Merge #1470

1470: remove dataset tests r=DhairyaLGandhi a=CarloLucibello

since the deprecation warnings are annoying, but most importantly because the download links have become unreliable and occasionally fail (as in #1469).


Co-authored-by: Carlo Lucibello <[email protected]>

Author: bors[bot]

test/data.jl

@@ -75,52 +75,3 @@
     Flux.train!(loss, [θ], ncycle(d, 10), Descent(0.1))
     @test norm(θ .- 1) < 1e-10
 end
-
-
-@testset "CMUDict" begin
-    @test cmudict()["CATASTROPHE"] == :[K,AH0,T,AE1,S,T,R,AH0,F,IY0].args
-
-    @test length(CMUDict.phones()) == 39
-
-    @test length(CMUDict.symbols()) == 84
-end
-
-@testset "MNIST" begin
-    @test MNIST.images()[1] isa Matrix
-    @test MNIST.labels() isa Vector{Int64}
-end
-
-@testset "FashionMNIST" begin
-    @test FashionMNIST.images()[1] isa Matrix
-    @test FashionMNIST.labels() isa Vector{Int64}
-end
-
-@testset "Sentiment" begin
-    @test Data.Sentiment.train() isa Vector{Data.Tree{Any}}
-end
-
-@testset "Iris" begin
-    @test Iris.features() isa Matrix
-    @test size(Iris.features()) == (4,150)
-
-    @test Iris.labels() isa Vector{String}
-    @test size(Iris.labels()) == (150,)
-end
-
-
-@testset "Housing" begin
-    @test Housing.features() isa Matrix # test broken due to SSL certifate expiration problem
-    @test size(Housing.features()) == (506, 13)
-
-    @test Housing.targets() isa Array{Float64}
-    @test size(Housing.targets()) == (506, 1)
-end
-
-@testset "Tree show" begin
-    # testcase for issue #1354
-    # testing that methods(Base.show) does not throw. Having something more specific would be too fragile
-    buf = IOBuffer()
-    Base.show(buf, filter(x->x.module == Flux, methods(Base.show).ms))
-    str_repr = String(take!(buf))
-    @test !isempty(str_repr)
-end