matrixfunctions / GraphMatFun.jl

Computation graphs for matrix functions
MIT License
13 stars 0 forks source link

unit test error #12

Closed jarlebring closed 3 years ago

jarlebring commented 3 years ago

When we add a unit test, the corresponding file needs to be added in test/runtests.jl. (Yes. This type of hardcoding is the most common way of doing it in julia packages.) There were a number of files missing. When I added the files I got quite a few errors. I fixed some myself, but I got one error which I did not see how to fix directly:

compress: Error During Test at /home/jarl/jobb_synced/src/matfun/test/compress.jl:2
  Got exception outside of a @test
  MethodError: no method matching replace_node!(::Compgraph{Complex{Float64}}, ::Symbol, ::Symbol)
  Closest candidates are:
    replace_node!(::Any, ::Any, ::Any, !Matched::Any) at /home/jarl/jobb_synced/src/matfun/src/compress_graph.jl:184
  Stacktrace:
   [1] compress_graph_trivial!(::Compgraph{Complex{Float64}}, ::Array{Tuple{Symbol,Int64},1}) at /home/jarl/jobb_synced/src/matfun/src/compress_graph.jl:252
   [2] top-level scope at /home/jarl/jobb_synced/src/matfun/test/compress.jl:76
   [3] top-level scope at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.5/Test/src/Test.jl:1115
   [4] top-level scope at /home/jarl/jobb_synced/src/matfun/test/compress.jl:4