codecheck passed updates

Signed-off-by: henrykotze <[email protected]>

Author: henrykotze

src/systems/air_flow/AirFlow.cc

@@ -87,7 +87,9 @@ class gz::sim::systems::AirFlowPrivate
 
   /// \brief Update air flow sensor data based on physics data
   /// \param[in] _ecm Immutable reference to ECM.
-  public: void UpdateAirFlows(const EntityComponentManager &_ecm, gz::math::Vector3d wind);
+  /// \param[in] wind The wind conditions in the environment
+  public: void UpdateAirFlows(const EntityComponentManager &_ecm,
+                              gz::math::Vector3d wind);
 
   /// \brief Remove air flow sensors if their entities have been removed
   /// from simulation.
@@ -146,13 +148,13 @@ void AirFlow::PostUpdate(const UpdateInfo &_info,
   if (!_info.paused)
   {
 
-
-    gz::math::Vector3d wind{0,0,0};
+    gz::math::Vector3d wind{0, 0, 0};
     // get wind as a component from the _ecm
     if(_ecm.EntityByComponents(components::Wind()) != kNullEntity){
 
       Entity windEntity = _ecm.EntityByComponents(components::Wind());
-      auto windLinearVel = _ecm.Component<components::WorldLinearVelocity>(windEntity);
+      auto windLinearVel = _ecm.Component<components::WorldLinearVelocity>
+                            (windEntity);
 
       if (windLinearVel != nullptr){
         wind = windLinearVel->Data();
@@ -268,7 +270,8 @@ void AirFlowPrivate::CreateSensors(const EntityComponentManager &_ecm)
 }
 
 //////////////////////////////////////////////////
-void AirFlowPrivate::UpdateAirFlows(const EntityComponentManager &_ecm, math::Vector3d wind)
+void AirFlowPrivate::UpdateAirFlows(const EntityComponentManager &_ecm,
+                                    math::Vector3d wind)
 {
   GZ_PROFILE("AirFlowPrivate::UpdateAirFlows");
   _ecm.Each<components::AirFlowSensor, components::WorldPose>(
@@ -287,8 +290,7 @@ void AirFlowPrivate::UpdateAirFlows(const EntityComponentManager &_ecm, math::Ve
 
           if(wind.IsFinite()){
             it->second->SetWindVelocity(wind);
-          } else 
-          {
+          } else {
             gzwarn << "Wind velocity is ignored. Contains NaN";
             it->second->SetWindVelocity(gz::math::Vector3d::Zero);
           }

test/integration/air_flow_system.cc

@@ -37,7 +37,6 @@
 
 #include "../helpers/Relay.hh"
 #include "../helpers/EnvTestFixture.hh"
-#include "test_config.hh"
 
 using namespace gz;
 using namespace sim;
@@ -142,7 +141,7 @@ TEST_F(AirFlowTest, GZ_UTILS_TEST_DISABLED_ON_WIN32(AirFlow))
 
 /////////////////////////////////////////////////
 // The test checks if the sensor is capable of measuring
-// the 
+// the wind conditions when it is stationary
 TEST_F(AirFlowTest, GZ_UTILS_TEST_DISABLED_ON_WIN32(AirFlowWindy))
 {
   // Start server
@@ -232,8 +231,8 @@ TEST_F(AirFlowTest, GZ_UTILS_TEST_DISABLED_ON_WIN32(AirFlowWindy))
 }
 
 /////////////////////////////////////////////////
-// The test checks if the sensor is moving at the same speed as the 
-// wind, that it measure no airflow
+// The test checks that the sensor measures the correct relative airflow
+// when it is moving within windy condition.
 TEST_F(AirFlowTest, GZ_UTILS_TEST_DISABLED_ON_WIN32(AirFlowMoveWindy))
 {
   // Start server
@@ -277,9 +276,8 @@ TEST_F(AirFlowTest, GZ_UTILS_TEST_DISABLED_ON_WIN32(AirFlowMoveWindy))
         body = Link(bodyEntity);
         body.EnableVelocityChecks(_ecm);
 
-        body.SetLinearVelocity(_ecm, gz::math::Vector3d(1,5,0));
+        body.SetLinearVelocity(_ecm, gz::math::Vector3d(1, 5, 0));
 
-    
     }).OnPostUpdate([&](const UpdateInfo &_info,
                               const EntityComponentManager &_ecm)
       {