summary throws error when an unknown method name is provided

Author: jbytecode

CHANGELOG.md

@@ -5,6 +5,8 @@
 - Initial attempt to reduce memory allocations in `lts()`, `lms()`, `hadi92()`, `hadi94()`, `hs93()`, `robcov()`
 - Replace `@assert` macro with `throw(ErrorException())` in whole code
 - `depestregression` returns `Dict` instead of a `vector` of betas like other regression methods.
+- `summary()` throws `ErrorException` rather than simply prompting with `@error` macro.
+
 
 
 # v0.11.3 

src/summary.jl

@@ -118,8 +118,7 @@ function detectOutliers(X::AbstractMatrix{Float64}, y::AbstractVector{Float64};
                 result = Int[]
             end
         else
-            @error "Method not found " method
-            result = Int[]
+            throw(ErrorException("Method not found: $method"))
         end
         outlier_matrix[:, method] = makeColorColumn(result, n)
     end

test/testsummary.jl

@@ -1,6 +1,8 @@
 @testset "Summary" begin
-    smiley = " 😔 "
-    methods = [
+
+    @testset "Withour errors" begin
+        smiley = " 😔 "
+        methods = [
             "hs93",
             "ks89",
             "smr98",
@@ -10,25 +12,40 @@
             "asm20",
             "bch",
             "bacon",
-            "imon2005",
+            "imon2005"
+        ]
+        sett = LinRegOutliers.createRegressionSetting(@formula(calls ~ year), phones)
+        result = LinRegOutliers.detectOutliers(sett, methods=methods)
+
+        for i in 15:21
+            @test result[:, "hs93"][i] == smiley
+        end
+
+        for i in 18:20
+            @test result[:, "ks89"][i] == smiley
+        end
+
+        for i in 15:24
+            @test result[:, "smr98"][i] == smiley
+        end
+
+        for i in 15:21
+            @test result[:, "lts"][i] == smiley
+        end
+
+    end
+
+
+
+    @testset "Withour errors" begin
+        
+        methods = [
             "unknown"
         ]
-    sett = LinRegOutliers.createRegressionSetting(@formula(calls ~ year), phones)
-    result = LinRegOutliers.detectOutliers(sett, methods = methods)
-    
-    for i in 15:21
-        @test result[:, "hs93"][i] == smiley
-    end 
-
-    for i in 18:20
-        @test result[:, "ks89"][i] == smiley
-    end 
-
-    for i in 15:24
-        @test result[:, "smr98"][i] == smiley
-    end 
-
-    for i in 15:21
-        @test result[:, "lts"][i] == smiley
-    end 
+        
+        sett = LinRegOutliers.createRegressionSetting(@formula(calls ~ year), phones)
+        
+        Test.@test_throws ErrorException LinRegOutliers.detectOutliers(sett, methods=methods)
+
+    end
 end