Do not error on duplicate boundary addition (#1401)

* Do not error on duplicate boundary addition

* add log message when adding duplicate boundary reaction

Author: oxinabox

release-notes/next-release.md

@@ -9,6 +9,7 @@
 * Fixed a bug with SBML group parsing that affects the Debian package.
 
 ## Other
+* Adding a duplicate boundary reaction (with `add_boundary`) no longer errors, but instead just returns the existing reaction.
 
 ## Deprecated features
 

src/cobra/core/model.py

@@ -676,7 +676,9 @@ def add_boundary(
                 "identifier. Please set the `reaction_id`."
             )
         if reaction_id in self.reactions:
-            raise ValueError(f"Boundary reaction '{reaction_id}' already exists.")
+            # It already exists so just retrieve it.
+            logger.info(f"Boundary reaction '{reaction_id}' already exists.")
+            return self.reactions.get_by_id(reaction_id)
         name = f"{metabolite.name} {type}"
         rxn = Reaction(id=reaction_id, name=name, lower_bound=lb, upper_bound=ub)
         rxn.add_metabolites({metabolite: -1})

tests/test_core/test_model.py

@@ -489,9 +489,9 @@ def test_add_existing_boundary(
         The allowed types for boundary, see add_boundary() for types.
     """
     for metabolite in metabolites:
-        model.add_boundary(metabolite, reaction_type)
-        with pytest.raises(ValueError):
-            model.add_boundary(metabolite, reaction_type)
+        rxn_added = model.add_boundary(metabolite, reaction_type)
+        rxn_dup = model.add_boundary(metabolite, reaction_type)
+        assert rxn_dup is rxn_added
 
 
 @pytest.mark.parametrize("solver", optlang_solvers)