make lint

Author: Gui-FernandesBR

tests/fixtures/controller/controller_fixtures.py

@@ -65,24 +65,24 @@ def controller_function(  # pylint: disable=unused-argument
         environment,
     ):
         # state = [x, y, z, vx, vy, vz, e0, e1, e2, e3, wx, wy, wz]
-        altitude_ASL = state[2]  # altitude above sea level
-        altitude_AGL = (
-            altitude_ASL - environment.elevation
+        altitude_asl = state[2]  # altitude above sea level
+        altitude_agl = (
+            altitude_asl - environment.elevation
         )  # altitude above ground level
         vx, vy, vz = state[3], state[4], state[5]
 
         # Use environment parameter instead of global variable
-        wind_x = environment.wind_velocity_x(altitude_ASL)
-        wind_y = environment.wind_velocity_y(altitude_ASL)
+        wind_x = environment.wind_velocity_x(altitude_asl)
+        wind_y = environment.wind_velocity_y(altitude_asl)
 
         # Calculate Mach number using environment data
         free_stream_speed = ((wind_x - vx) ** 2 + (wind_y - vy) ** 2 + (vz) ** 2) ** 0.5
-        mach_number = free_stream_speed / environment.speed_of_sound(altitude_ASL)
+        mach_number = free_stream_speed / environment.speed_of_sound(altitude_asl)
 
         if time < 3.9:
             return None
 
-        if altitude_AGL < 1500:
+        if altitude_agl < 1500:
             air_brakes.deployment_level = 0
         else:
             previous_vz = state_history[-1][5] if state_history else vz

tests/integration/simulation/test_flight.py

@@ -604,7 +604,7 @@ def test_7_parameter_controller_with_sensors(calisto_robust, example_plain_env):
     """
 
     # Define a 7-parameter controller
-    def controller_7_params(
+    def controller_7_params(  # pylint: disable=unused-argument
         time,
         sampling_rate,
         state,
@@ -662,14 +662,14 @@ def test_invalid_controller_parameter_count(calisto_robust):
     """
 
     # Define controller with wrong number of parameters (5)
-    def invalid_controller_5_params(
+    def invalid_controller_5_params(  # pylint: disable=unused-argument
         time, sampling_rate, state, state_history, observed_variables
     ):
         """Invalid controller with only 5 parameters."""
         return None
 
     # Define controller with wrong number of parameters (9)
-    def invalid_controller_9_params(
+    def invalid_controller_9_params(  # pylint: disable=unused-argument
         time,
         sampling_rate,
         state,
@@ -729,7 +729,7 @@ def test_environment_methods_accessible_in_controller(
         "temperature": False,
     }
 
-    def controller_test_environment_access(
+    def controller_test_environment_access(  # pylint: disable=unused-argument
         time,
         sampling_rate,
         state,
@@ -740,7 +740,7 @@ def controller_test_environment_access(
         environment,
     ):
         """Controller that tests access to various environment methods."""
-        altitude_ASL = state[2]
+        altitude_asl = state[2]
 
         if time < 3.9:
             return None
@@ -750,25 +750,25 @@ def controller_test_environment_access(
             _ = environment.elevation
             methods_called["elevation"] = True
 
-            _ = environment.wind_velocity_x(altitude_ASL)
+            _ = environment.wind_velocity_x(altitude_asl)
             methods_called["wind_velocity_x"] = True
 
-            _ = environment.wind_velocity_y(altitude_ASL)
+            _ = environment.wind_velocity_y(altitude_asl)
             methods_called["wind_velocity_y"] = True
 
-            _ = environment.speed_of_sound(altitude_ASL)
+            _ = environment.speed_of_sound(altitude_asl)
             methods_called["speed_of_sound"] = True
 
-            _ = environment.pressure(altitude_ASL)
+            _ = environment.pressure(altitude_asl)
             methods_called["pressure"] = True
 
-            _ = environment.temperature(altitude_ASL)
+            _ = environment.temperature(altitude_asl)
             methods_called["temperature"] = True
 
             air_brakes.deployment_level = 0.3
         except AttributeError as e:
             # If any method is not accessible, the test should fail
-            raise AssertionError(f"Environment method not accessible: {e}")
+            raise AssertionError(f"Environment method not accessible: {e}") from e
 
         return (time, air_brakes.deployment_level)