Fix Module scoping and add test. (#74)

* Fix Module scoping and add test.

* Add compat

* Remove unneeded test.

Author: odow

src/LinQuadOptInterface.jl

@@ -231,8 +231,8 @@ macro LinQuadOptimizerBase(inner_model_type=Any)
     qconstraint_primal_solution::Vector{Float64}
     qconstraint_dual_solution::Vector{Float64}
 
-    constraint_names::Dict{CI, String}
-    constraint_names_rev::Dict{String, Set{CI}}
+    constraint_names::Dict{LinQuadOptInterface.CI, String}
+    constraint_names_rev::Dict{String, Set{LinQuadOptInterface.CI}}
 
     objective_constant::Float64
 

test/runtests.jl

@@ -7,6 +7,16 @@ const MOIT = MathOptInterface.Test
 const MOIU = MathOptInterface.Utilities
 const LQOI = LinQuadOptInterface
 
+# Test that using LinQuadOptInterface passes without error. We have to do this
+# here because types can't be created in testsets. If it can be defined without
+# error, we're okay. This is most likely to error if fields are added to
+# @LinQuadOptimizerBase without proper module prefixes.
+using Compat  # For Nothing on v0.6
+mutable struct OptimizerTest <: LinQuadOptInterface.LinQuadOptimizer
+    LinQuadOptInterface.@LinQuadOptimizerBase
+    OptimizerTest() = new()
+end
+
 @testset "LinQuadOptInterface" begin
     solver = LQOI.MockLinQuadOptimizer()