Improve docstring

Author: blegat

src/structures.jl

@@ -41,18 +41,23 @@ variables and their dual counterparts.
     this is the position in that vector.
 
   * `dual_constraint::Union{Nothing,MOI.ConstraintIndex}`: dual constraint
-    associated with the variable. If the variable is not constrained then the
-    set is EqualTo{T}(zero(T)). If the variable is a constrained variable then
-    the set is the dual set of the constrained variable set. If the dual set is
-    `Reals` then the field is kept as `nothing` as teh constraint is not added.
-
-  * `dual_function::Union{Nothing,MOI.ScalarAffineFunction{T}}`: if the
+    associated with the variable. If this is a variable constrained in
+    `MOI.Zeros` or `MOI.EqualTo`, then this is `nothing`. Otherwise, this is
+    a constraint with set corresponding to the dual set this variable is
+    constrained to.  If the variable is not constrained then this an equality
+    constrained, so the set is `MOI.EqualTo` even if strictly speaking, the dual
+    set of `MOI.Reals` is `MOI.Zeros`. If the variable is a constrained variable
+    in a given set `S`, this is a constraint on the dual set `MOI.dual_set(S)`.
+
+  * `dual_function::Union{Nothing,MOI.ScalarAffineFunction{T}}`: if it is a
     constrained variable is `VectorOfVariables`-in-`Zeros` or
     `VariableIndex`-in-`EqualTo(zero(T))` then the dual is `func`-in-`Reals`,
-    which is "irrelevant" to the model. But this information is cached for
-    completeness of the `DualOptimizer` for `get`ting `ConstraintDuals`.
+    which is "irrelevant" to the model. So the no constrained is added (hence
+    `dual_constraint` is `nothing` but the function is cached in this field for
+    completeness of the `DualOptimizer` for `get`ting `ConstraintDual`s.
+    Otherwise, `dual_function` is `nothing`.
 
-To got from the constrained variable constraint to the primal variable, use the
+To go from the constrained variable constraint to the primal variable, use the
 `primal_constrained_variables` field of `PrimalDualMap`.
 
 See also `PrimalDualMap` and `PrimalConstraintData`.

test/Tests/test_MOI_wrapper.jl

@@ -20,16 +20,16 @@
                 linear_config,
                 include = ["test_linear_"],
                 exclude = [
-                    "test_linear_FEASIBILITY_SENSE",
-                    "test_linear_INFEASIBLE_2",
-                    "test_linear_Interval_inactive",
-                    "test_linear_add_constraints",
-                    "test_linear_inactive_bounds",
-                    "test_linear_integration_2",
-                    "test_linear_integration_Interval",
-                    "test_linear_integration_delete_variables",
-                    "test_linear_complex_Zeros",
-                    "test_linear_complex_Zeros_duplicate",
+                    r"^test_linear_FEASIBILITY_SENSE$",
+                    r"^test_linear_INFEASIBLE_2$",
+                    r"^test_linear_Interval_inactive$",
+                    r"^test_linear_add_constraints$",
+                    r"^test_linear_inactive_bounds$",
+                    r"^test_linear_integration_2$",
+                    r"^test_linear_integration_Interval$",
+                    r"^test_linear_integration_delete_variables$",
+                    r"^test_linear_complex_Zeros$",
+                    r"^test_linear_complex_Zeros_duplicate$",
                 ],
             )
         end