diff --git a/MekHQ/src/mekhq/campaign/personnel/enums/PersonnelRole.java b/MekHQ/src/mekhq/campaign/personnel/enums/PersonnelRole.java index 8d34663301..64a5e663a2 100644 --- a/MekHQ/src/mekhq/campaign/personnel/enums/PersonnelRole.java +++ b/MekHQ/src/mekhq/campaign/personnel/enums/PersonnelRole.java @@ -253,7 +253,7 @@ public boolean isAerospaceGrouping() { } public boolean isConventionalAirGrouping() { - return isAerospaceGrouping() || isConventionalAircraftPilot(); + return isConventionalAircraftPilot(); } public boolean isGroundVehicleCrew() { diff --git a/MekHQ/unittests/mekhq/campaign/personnel/enums/PersonnelRoleTest.java b/MekHQ/unittests/mekhq/campaign/personnel/enums/PersonnelRoleTest.java index eab131e15b..9a1febbe5d 100644 --- a/MekHQ/unittests/mekhq/campaign/personnel/enums/PersonnelRoleTest.java +++ b/MekHQ/unittests/mekhq/campaign/personnel/enums/PersonnelRoleTest.java @@ -447,9 +447,7 @@ void testIsAerospaceGrouping() { @Test void testIsConventionalAirGrouping() { for (final PersonnelRole personnelRole : roles) { - if ((personnelRole == PersonnelRole.LAM_PILOT) - || (personnelRole == PersonnelRole.AEROSPACE_PILOT) - || (personnelRole == PersonnelRole.CONVENTIONAL_AIRCRAFT_PILOT)) { + if (personnelRole == PersonnelRole.CONVENTIONAL_AIRCRAFT_PILOT) { assertTrue(personnelRole.isConventionalAirGrouping()); } else { assertFalse(personnelRole.isConventionalAirGrouping());