cleanup a bit

tests/transform/test_transform_split_reversible_reactions.py

@@ -138,20 +138,21 @@ def test_skipping_submodels(self):
         self.assertNotEqual(rxn_ids, set([rxn.id for rxn in model.reactions]))
 
         self.submodel.framework = onto['WC:dynamic_flux_balance_analysis']
-        model = self.model.copy()
-        SplitReversibleReactionsTransform(excluded_frameworks=dfba_framework).run(model)
-        self.assertEqual(rxn_ids, set([rxn.id for rxn in model.reactions]))
+        SplitReversibleReactionsTransform(excluded_frameworks=dfba_framework).run(self.model)
+        self.assertEqual(rxn_ids, set([rxn.id for rxn in self.model.reactions]))
 
-    def test_dfba_submodel(self):
+    def test_dfba_flux_bounds(self):
         self.submodel.framework = onto['WC:dynamic_flux_balance_analysis']
 
+        # test no flux bounds
         self.r1.reversible = True
         model = self.model.copy()
         SplitReversibleReactionsTransform().run(model)
         for id in ['r0_forward', 'r0_backward', 'r1_forward', 'r1_backward']:
             rxn = model.reactions.get_one(id=id)
             self.assertEqual(rxn.flux_bounds, None)
 
+        # test bad flux bounds
         self.r0.flux_bounds = FluxBounds(min=3.,
                                          max=2.,
                                          units=unit_registry.parse_units('M s^-1'))