Update

Author: odow

src/attributes.jl

@@ -1273,7 +1273,7 @@ function Base.showerror(io::IO, err::OptimizeInProgress)
         This error occurs when you try to query an unsupported attribute from \
         inside a callback. Consult the solver's documentation to learn which \
         attributes may be queried inside a callback.
-        """
+        """,
     )
 end
 

test/Bridges/General/test_lazy_bridge_optimizer.jl

@@ -1772,17 +1772,18 @@ function test_UnsupportedConstraint_when_it_cannot_be_bridged()
     mock = MOI.Utilities.MockOptimizer(NoRSOCModel{Float64}())
     bridged_mock = MOI.Bridges.LazyBridgeOptimizer(mock)
     x = MOI.add_variables(bridged_mock, 4)
-    err = MOI.UnsupportedConstraint{
-        MOI.VectorOfVariables,
-        MOI.RotatedSecondOrderCone,
-    }()
-    @test_throws err begin
+    @test_throws(
+        MOI.UnsupportedConstraint{
+            MOI.VectorOfVariables,
+            MOI.RotatedSecondOrderCone,
+        },
         MOI.add_constraint(
             bridged_mock,
             MOI.VectorOfVariables(x),
             MOI.RotatedSecondOrderCone(4),
-        )
-    end
+        ),
+    )
+    return
 end
 
 function test_MOI_runtests_No_RSOCModel()
@@ -1839,17 +1840,17 @@ function test_bridge_selection()
         MOI.LogDetConeTriangle,
     ))
     x = MOI.add_variables(bridged_mock, 3)
-    err = MOI.UnsupportedConstraint{
-        MOI.VectorAffineFunction{Float64},
-        MOI.LogDetConeTriangle,
-    }()
-    @test_throws err begin
+    @test_throws(
+        MOI.UnsupportedConstraint{
+            MOI.VectorAffineFunction{Float64},
+            MOI.LogDetConeTriangle,
+        },
         MOI.Bridges.bridge_type(
             bridged_mock,
             MOI.VectorAffineFunction{Float64},
             MOI.LogDetConeTriangle,
-        )
-    end
+        ),
+    )
     c = MOI.add_constraint(
         bridged_mock,
         MOI.VectorOfVariables(x),

test/General/test_attributes.jl

@@ -392,15 +392,15 @@ end
 
 function test_showerror_OptimizeInProgress()
     err = MOI.OptimizeInProgress(MOI.VariablePrimal())
-    @test sprint(showerror, err) ==
-          """
-          Unable to get the attribute MathOptInterface.VariablePrimal(1) \
-          because `MOI.optimize!` in progress.
-
-          This error occurs when you try to query an unsupported attribute from \
-          inside a callback. Consult the solver's documentation to learn which \
-          attributes may be queried inside a callback.
-          """
+    ret = """
+    Unable to get the attribute MathOptInterface.VariablePrimal(1) \
+    because `MOI.optimize!` in progress.
+
+    This error occurs when you try to query an unsupported attribute from \
+    inside a callback. Consult the solver's documentation to learn which \
+    attributes may be queried inside a callback.
+    """
+    @test sprint(showerror, err) == ret
     return
 end
 

test/General/test_constraints.jl

@@ -46,17 +46,17 @@ function test_LowerBoundAlreadySet_error()
     S2 = MOI.Interval{Int}
     err = MOI.LowerBoundAlreadySet{S1,S2}(x)
     @test err isa Exception
-    @test sprint(showerror, err) ==
-          """
-          LowerBoundAlreadySet{
-              $S1,
-              $S2,
-          }
+    ret = """
+    LowerBoundAlreadySet{
+        $S1,
+        $S2,
+    }
 
-          You cannot add a `VariableIndex`-in-`$(S2)` constraint for variable \
-          $(x) because a `VariableIndex`-in-`$(S1)` constraint was already set \
-          for this variable and both constraints define a lower bound.
-          """
+    You cannot add a `VariableIndex`-in-`$(S2)` constraint for variable \
+    $(x) because a `VariableIndex`-in-`$(S1)` constraint was already set \
+    for this variable and both constraints define a lower bound.
+    """
+    @test sprint(showerror, err) == ret
     return
 end
 
@@ -66,17 +66,17 @@ function test_UpperBoundAlreadySet_error()
     S2 = MOI.Interval{Int}
     err = MOI.UpperBoundAlreadySet{S1,S2}(x)
     @test err isa Exception
-    @test sprint(showerror, err) ==
-          """
-          UpperBoundAlreadySet{
-              $S1,
-              $S2,
-          }
+    ret = """
+    UpperBoundAlreadySet{
+        $S1,
+        $S2,
+    }
 
-          You cannot add a `VariableIndex`-in-`$(S2)` constraint for variable \
-          $(x) because a `VariableIndex`-in-`$(S1)` constraint was already set \
-          for this variable and both constraints define an upper bound.
-          """
+    You cannot add a `VariableIndex`-in-`$(S2)` constraint for variable \
+    $(x) because a `VariableIndex`-in-`$(S1)` constraint was already set \
+    for this variable and both constraints define an upper bound.
+    """
+    @test sprint(showerror, err) == ret
     return
 end
 

test/General/test_errors.jl

@@ -66,7 +66,7 @@ function test_errors_UnsupportedConstraint()
         MOI.UnsupportedConstraint,
         MOI.add_constraint(model, vi, MOI.EqualTo(0)),
     )
-    msg = """
+    ret = """
     UnsupportedConstraint{
         $(MOI.VariableIndex),
         $(MOI.EqualTo{Int}),
@@ -79,7 +79,7 @@ function test_errors_UnsupportedConstraint()
     try
         MOI.add_constraint(model, vi, MOI.EqualTo(0))
     catch err
-        @test sprint(showerror, err) == msg
+        @test sprint(showerror, err) == ret
     end
     return
 end
@@ -299,23 +299,23 @@ end
 
 function test_errors_ResultIndexBoundsError()
     err = MOI.ResultIndexBoundsError(MOI.VariablePrimal(1), 0)
-    @test sprint(showerror, err) ==
-          """
-          Result index of attribute MathOptInterface.VariablePrimal(1) is out of bounds.
+    ret = """
+    Result index of attribute MathOptInterface.VariablePrimal(1) is out of bounds.
 
-          There are currently 0 solution(s) in the model.
-          """
+    There are currently 0 solution(s) in the model.
+    """
+    @test sprint(showerror, err) == ret
     return
 end
 
 function test_errors_ConflictIndexBoundsError()
     err = MOI.ConflictIndexBoundsError(MOI.ConstraintConflictStatus(1), 0)
-    @test sprint(showerror, err) ==
-          """
-          Conflict index of attribute MathOptInterface.ConstraintConflictStatus(1) is out of bounds.
+    ret = """
+    Conflict index of attribute MathOptInterface.ConstraintConflictStatus(1) is out of bounds.
 
-          There are currently 0 conflict(s) in the model.
-          """
+    There are currently 0 conflict(s) in the model.
+    """
+    @test sprint(showerror, err) == ret
     return
 end
 
@@ -396,13 +396,13 @@ end
 
 function test_showerror_InvalidIndex()
     x = MOI.VariableIndex(1)
-    @test sprint(showerror, MOI.InvalidIndex(x)) ==
-          """
-          The index $x is invalid.
+    err = """
+    The index $x is invalid.
 
-          An index becomes invalid after it has been deleted. Alternatively, this \
-          error might mean that you have re-used the index from a different model.
-          """
+    An index becomes invalid after it has been deleted. Alternatively, this \
+    error might mean that you have re-used the index from a different model.
+    """
+    @test sprint(showerror, MOI.InvalidIndex(x)) == err
     return
 end