Fix Bridge test for non-invertible constraint bridge

Author: blegat

src/Bridges/Bridges.jl

@@ -142,7 +142,7 @@ MOI.attribute_value_type(::ListOfNonstandardBridges) = Vector
 
 MOI.is_copyable(::ListOfNonstandardBridges) = false
 
-MOI.get_fallback(model::MOI.ModelLike, ::ListOfNonstandardBridges) = Type[]
+MOI.get_fallback(::MOI.ModelLike, ::ListOfNonstandardBridges) = Type[]
 
 function _test_structural_identical(
     a::MOI.ModelLike,
@@ -191,12 +191,8 @@ function _test_structural_identical(
             f_b = try
                 MOI.get(b, MOI.ConstraintFunction(), ci)
             catch err
-                if cannot_unbridge &&
-                   err isa MOI.GetAttributeNotAllowed{MOI.ConstraintFunction}
-                    continue
-                else
-                    rethrow(err)
-                end
+                _runtests_error_handler(err, cannot_unbridge)
+                continue
             end
             f_b = MOI.Utilities.map_indices(x_map, f_b)
             s_b = MOI.get(b, MOI.ConstraintSet(), ci)
@@ -227,7 +223,10 @@ end
 _runtests_error_handler(err, ::Bool) = rethrow(err)
 
 function _runtests_error_handler(
-    err::MOI.GetAttributeNotAllowed{MOI.ConstraintFunction},
+    err::Union{
+        MOI.GetAttributeNotAllowed{MOI.ConstraintFunction},
+        MOI.GetAttributeNotAllowed{MOI.ConstraintPrimalStart},
+    },
     cannot_unbridge::Bool,
 )
     if cannot_unbridge
@@ -328,7 +327,15 @@ function runtests(
                     Test.@test MOI.get(model, attr, ci) === nothing
                     start = _fake_start(constraint_start, set)
                     MOI.set(model, attr, ci, start)
-                    Test.@test MOI.get(model, attr, ci) ≈ start
+                    returned_start = try
+                        MOI.get(model, attr, ci)
+                    catch err
+                        # For a Constraint bridge for which the map is not invertible, the constraint primal cannot
+                        # be inverted
+                        _runtests_error_handler(err, Bridge <: MOI.Bridges.Constraint.AbstractBridge && cannot_unbridge)
+                        continue
+                    end
+                    Test.@test returned_start ≈ start
                 end
             end
         end
@@ -416,7 +423,7 @@ _fake_start(value, ::MOI.AbstractScalarSet) = value
 
 _fake_start(value, set::MOI.AbstractVectorSet) = fill(value, MOI.dimension(set))
 
-_fake_start(value::AbstractVector, set::MOI.AbstractVectorSet) = value
+_fake_start(value::AbstractVector, ::MOI.AbstractVectorSet) = value
 
 function _bridged_model(Bridge::Type{<:Constraint.AbstractBridge}, inner)
     return Constraint.SingleBridgeOptimizer{Bridge}(inner)