[UnitTests] Update/add tests related to electrochemistry

Cantera · Mar 15, 2022 · 093af64 · 093af64
1 parent 874b85b
commit 093af64
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/interfaces/cython/cantera/test/test_reaction.py b/interfaces/cython/cantera/test/test_reaction.py
@@ -1757,7 +1757,7 @@ def setUpClass(cls):
             args = list(cls._rate.values())
             cls._rate_obj = ct.Arrhenius(*args)
             cls._cls = ct.InterfaceReaction
-            cls._rxn_type = "interface"
+            cls._rxn_type = "interface-legacy"
             cls._rate_type = None
             cls._rate_cls = None
             cls._legacy_uses_rate = True
@@ -1816,6 +1816,16 @@ def test_rate(self):
             pytest.skip("Legacy: interface rate does not include coverage terms")
         super().test_rate()
 
+    def test_electrochemistry(self):
+        if self._legacy:
+            pytest.skip("Legacy: property uses_electrochemisty not implemented")
+        rxn = self.from_yaml()
+        sol2 = ct.Interface(thermo="Surface", kinetics="interface",
+                    species=self.species, reactions=[rxn], adjacent=self.adj)
+        rate2 = sol2.reaction(0).rate
+        assert not rate2.uses_electrochemistry
+        assert np.isnan(rate2.beta)
+
 
 class TestArrheniusInterfaceReaction(InterfaceReactionTests, utilities.CanteraTest):
     # test interface reaction without coverages

diff --git a/test/kinetics/kineticsFromYaml.cpp b/test/kinetics/kineticsFromYaml.cpp
@@ -631,7 +631,7 @@ TEST_F(ReactionToYaml, electrochemical)
     oxide_surf->thermo()->setElectricPotential(-3.4);
     ox_surf->setCoveragesByName("O''(ox):0.2, OH'(ox):0.3, H2O(ox):0.5");
     duplicateReaction(0);
-    EXPECT_TRUE(std::dynamic_pointer_cast<ElectrochemicalReaction2>(duplicate));
+    EXPECT_TRUE(std::dynamic_pointer_cast<InterfaceArrheniusRate>(duplicate->rate()));
     compareReactions();
     compareReactions();
 }