invenia / Memento.jl

A flexible logging library for Julia
https://invenia.github.io/Memento.jl/latest
Other
88 stars 14 forks source link

@test_log not accepting string variables #55

Closed morris25 closed 6 years ago

morris25 commented 6 years ago

When running

query_id = response.query_id
warn_string = "Unable to find any results for your query $query_id. This could mean that your query is too restrictive or that something is wrong with the S3DB backend."
print(warn_string)
@test_warn(logger, warn_string, fetch(response))

I get

  Got an exception of type UndefVarError outside of a @test
  UndefVarError: warn_string not defined
  Stacktrace:
   [1] macro expansion at /Users/sam/.julia/v0.6/Memento/src/test.jl:18 [inlined]
   [2] macro expansion at /Users/sam/.julia/v0.6/S3DB/test/online.jl:78 [inlined]
   [3] macro expansion at ./test.jl:860 [inlined]
   [4] anonymous at ./<missing>:?
   [5] include_from_node1(::String) at ./loading.jl:576
   [6] include(::String) at ./sysimg.jl:14
   [7] macro expansion at /Users/sam/.julia/v0.6/S3DB/test/runtests.jl:26 [inlined]
   [8] macro expansion at ./test.jl:860 [inlined]
   [9] anonymous at ./<missing>:?
   [10] include_from_node1(::String) at ./loading.jl:576
   [11] include(::String) at ./sysimg.jl:14
   [12] process_options(::Base.JLOptions) at ./client.jl:305
   [13] _start() at ./client.jl:371

The string at the top is the correct output from print but @test_warn doesn't recognize warn_string.

Apparently esc is not being done on both "level" and "msg"