Don't test SMS Spam Collection on CI (#164)

* Remove test for SMS Spam

* Fix SMS-Spam

Author: Dsantra92

src/datasets/text/smsspamcollection.jl

@@ -75,7 +75,7 @@ function Base.getproperty(::Type{SMSSpamCollection}, s::Symbol)
         @warn "SMSSpamCollection.targets() is deprecated, use `SMSSpamCollection().targets` instead."
         return () -> SMSSpamCollection().targets
     else 
-        return getfield(Titanic, s)
+        return getfield(SMSSpamCollection, s)
     end
 end
 

test/datasets/text.jl

@@ -14,21 +14,6 @@
     end
 end
 
-
-@testset "SMSSpamCollection" begin
-    n_obs = 5574
-    n_targets = 1
-    n_features = ()
-    Tx=String
-    Ty=String
-    
-    d = SMSSpamCollection()
-    
-    test_supervised_array_dataset(d;
-        n_obs, n_targets, n_features,
-        Tx, Ty)
-end
-
 @testset "UD_English" begin
     n_features = ()
     Tx = Vector{Vector{String}}

test/datasets/text_no_ci.jl

@@ -0,0 +1,14 @@
+@testset "SMSSpamCollection" begin
+    n_obs = 5574
+    n_targets = 1
+    n_features = ()
+    Tx=String
+    Ty=String
+
+    d = SMSSpamCollection()
+    
+    test_supervised_array_dataset(d;
+        n_obs, n_targets, n_features,
+        Tx, Ty)
+end
+

test/runtests.jl

@@ -22,6 +22,7 @@ dataset_tests = [
 
 no_ci_dataset_tests = [
     "datasets/graphs_no_ci.jl",
+    "datasets/text_no_ci.jl",
     "datasets/vision/cifar10.jl",
     "datasets/vision/cifar100.jl",
     "datasets/vision/emnist.jl",