From a00e4bbe945114d558c5ad6ccc8323d9d36c86c4 Mon Sep 17 00:00:00 2001 From: Ingmar Schoegl Date: Sun, 13 Mar 2022 22:10:22 -0500 Subject: [PATCH] [UnitTests] Update/add tests related to electrochemistry --- interfaces/cython/cantera/test/test_reaction.py | 7 +++++++ test/kinetics/kineticsFromYaml.cpp | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/interfaces/cython/cantera/test/test_reaction.py b/interfaces/cython/cantera/test/test_reaction.py index 0ae34032a91..026c07f0dcf 100644 --- a/interfaces/cython/cantera/test/test_reaction.py +++ b/interfaces/cython/cantera/test/test_reaction.py @@ -1816,6 +1816,13 @@ 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() + assert not rxn.rate.uses_electrochemistry + assert np.isnan(rxn.rate.beta) + class TestArrheniusInterfaceReaction(InterfaceReactionTests, utilities.CanteraTest): # test interface reaction without coverages diff --git a/test/kinetics/kineticsFromYaml.cpp b/test/kinetics/kineticsFromYaml.cpp index 5e8dbf742b3..d7923b279b5 100644 --- 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(duplicate)); + EXPECT_TRUE(std::dynamic_pointer_cast(duplicate->rate())); compareReactions(); compareReactions(); }