Skip to content

Commit 80c370f

Browse files
committed
Test more parts of Optim interface
1 parent c645e11 commit 80c370f

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

ext/DynamicExpressionsOptimExt.jl

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,6 @@ end
2727
function Base.propertynames(r::ExpressionOptimizationResults)
2828
return (:tree, propertynames(getfield(r, :_results))...)
2929
end
30-
function base_results(r::ExpressionOptimizationResults)
31-
return getfield(r, :_results)
32-
end
3330
function Optim.minimizer(r::ExpressionOptimizationResults)
3431
return r.tree
3532
end

test/test_optim.jl

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,10 @@ end
4545

4646
res = optimize(f, g!, tree, BFGS())
4747
@test did_i_run[]
48+
@test res.f_calls > 0
4849
@test isapprox(get_constants(res.minimizer), get_constants(target_tree); atol=0.01)
50+
@test Optim.minimizer(res) === res.minimizer
51+
@test propertynames(res) == (:tree, propertynames(getfield(res, :_results))...)
4952

5053
@testset "Hessians not implemented" begin
5154
@test_throws ArgumentError optimize(f, g!, t -> t, tree, BFGS())

0 commit comments

Comments
 (0)