Fix some tests and suppress warnings in Julia 0.7

[tkf]

First two commits for make test working. The rest is for suppressing deprecated warnings.

[codecov-io]

Codecov Report

Merging #31 into master will increase coverage by 2.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
+ Coverage   94.02%   96.09%   +2.06%     
==========================================
  Files           5        4       -1     
  Lines         201      205       +4     
==========================================
+ Hits          189      197       +8     
+ Misses         12        8       -4

Impacted Files	Coverage Δ
src/settable.jl	98.38% <100%> (+0.02%)	⬆️
src/sugar.jl	94.44% <0%> (+0.44%)	⬆️
src/macrotools.jl	90.9% <0%> (+6.81%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d661e8...47bcc7d. Read the comment docs.

[jw3126]

Thanks for doing this!
Off topic: I am curious what functionality from this package you are actually using. Personally I only use it to manipulate immutable types and never used it to do actual mutations (e.g. v =[1]; @set! v[1]=2). Initially I thought it was useful for generic programming that supports immutable and mutable types simultaneously. But I never needed it in practice. OTOH I implement custom lenses from time to time and then the MutationPolicy machinery just adds annoying complexity. I am thinking about removing it, do you have an opinion about that?

[jw3126]

Is there a reason, why you use try instead of if VERSION < v"0.7-"?

[tkf]

I was thinking it'd be safer for earlier revisions of Julia master and I was lazy to find at which revision __module__ was introduced. But I used if VERSION < v"0.7-" elsewhere so it's better to stick with it for consistency. I'll fix it.

[jw3126]

Is the problem QuickTypes not working with 0.7 or is there a problem with our macros, do you know?

[tkf]

QuickTypes.jl itself doesn't work with the nighly at the moment. I don't know if our macro has any problems. I tried to fix QuickTypes.jl but it's tricky so I thought I'd skip testing for now.

[jw3126]

Why @test_skip and not @test_broken?

[tkf]

@test_broken requires the expression to return false but @set! v[:] .= 0 actually throws at the moment. @test_skip doesn't execute the body so effectively it's commenting out the expression. I thought it's slightly better than commenting out since it shows up in the test result and we can remember that there is something to fix. But @test_skip is not super correct to use it here since @set! v[:] .= 0 will not return true. I suppose I should have leave informative comments there.