Skip to content

Commit 1075428

Browse files
committed
fix a bug in logic handling failed application of operations
1 parent 77428a4 commit 1075428

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/test.jl

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,6 +209,7 @@ function test(model_proxies, data...; mod=Main, level=2, throw=false, verbosity=
209209
function update(row, i, test, value_or_exception, outcome)
210210
outcome_nt = NamedTuple{(test,)}((outcome,))
211211
updated_row = merge(row, outcome_nt)
212+
@show updated_row
212213
summary[i] = updated_row
213214
if outcome == "×"
214215
failures_row = (
@@ -266,7 +267,7 @@ function test(model_proxies, data...; mod=Main, level=2, throw=false, verbosity=
266267
operations, outcome =
267268
MLJTestIntegration.operations(fitted_machine, data...; throw, verbosity)
268269
# special treatment to get list of operations in `summary`:
269-
if operations == "×"
270+
if outcome == "×"
270271
row = update(row, i, :operations, operations, outcome)
271272
continue
272273
else

0 commit comments

Comments
 (0)