const& and use functions in test

Author: pcarruscag

SU2_CFD/include/drivers/CDriverBase.hpp

@@ -124,13 +124,13 @@ class CDriverBase {
    * \brief Get the list of available outputs.
    * \return List of output names.
    */
-  inline vector<string> GetOutputNames() const { return output_container[MESH_0]->GetHistoryOutput_List(); }
+  inline vector<string> GetOutputNames() const { return output_container[MESH_0]->GetHistoryOutputList(); }
 
   /*!
    * \brief Get the value of one of the available history outputs.
    * \return Value of the output.
    */
-  inline passivedouble GetOutputValue(std::string output_name) const {
+  inline passivedouble GetOutputValue(const std::string& output_name) const {
     return SU2_TYPE::GetValue(output_container[MESH_0]->GetHistoryFieldValue(output_name));
   }
 
@@ -139,14 +139,15 @@ class CDriverBase {
    * \return List of surface output names.
    */
   inline vector<string> GetMarkerOutputNames() const {
-    return output_container[MESH_0]->GetHistoryOutputPerSurface_List();
+    return output_container[MESH_0]->GetHistoryOutputPerSurfaceList();
   }
 
   /*!
    * \brief Get the value of one of the available surface outputs at a given MARKER_MONITORING.
    * \return Value of the output.
    */
-  inline passivedouble GetMarkerMonitoringOutputValue(std::string output_name, std::string marker_monitoring) const {
+  inline passivedouble GetMarkerMonitoringOutputValue(const std::string& output_name,
+                                                      const std::string& marker_monitoring) const {
     for (auto iMarker = 0u; iMarker < main_config->GetnMarker_Monitoring(); ++iMarker) {
       if (marker_monitoring == main_config->GetMarker_Monitoring_TagBound(iMarker))
         return SU2_TYPE::GetValue(output_container[MESH_0]->GetHistoryFieldValuePerSurface(output_name, iMarker));
@@ -159,7 +160,8 @@ class CDriverBase {
    * \brief Get the value of one of the available surface outputs at a given MARKER_ANALYZE.
    * \return Value of the output.
    */
-  inline passivedouble GetMarkerAnalyzeOutputValue(std::string output_name, std::string marker_analyze) const {
+  inline passivedouble GetMarkerAnalyzeOutputValue(const std::string& output_name,
+                                                   const std::string& marker_analyze) const {
     for (auto iMarker = 0u; iMarker < main_config->GetnMarker_Analyze(); ++iMarker) {
       if (marker_analyze == main_config->GetMarker_Analyze_TagBound(iMarker))
         return SU2_TYPE::GetValue(output_container[MESH_0]->GetHistoryFieldValuePerSurface(output_name, iMarker));

TestCases/py_wrapper/deforming_bump_in_channel/run.py

@@ -88,6 +88,11 @@ def main():
     TimeIter += 1
     time += deltaT
 
+  # Check the value of an output to cover the functionality in a regression test.
+  assert 'DRAG' in SU2Driver.GetOutputNames()
+  assert abs(SU2Driver.GetOutputValue('DRAG') -
+             SU2Driver.GetMarkerMonitoringOutputValue('DRAG_ON_SURFACE', MarkerName)) < np.finfo(float).eps
+
   # Finalize the solver and exit cleanly
   SU2Driver.Finalize()