Fix test fail in modern systems

by adding the missing return. Sheesh.
vegastrike · Nov 14, 2024 · ed3957e · ed3957e
1 parent 089e004
commit ed3957e
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 12 deletions.
diff --git a/engine/src/components/drive_upgrade.cpp b/engine/src/components/drive_upgrade.cpp
@@ -162,6 +162,8 @@ bool DriveUpgrade::Upgrade(const std::string upgrade_name) {
     drive->max_roll_right.SetMaxValue(drive->max_roll_right.MaxValue() * max_roll_right);
 
     drive->SetConsumption(drive->GetConsumption() * fuel_consumption);
+
+    return true;
 }
 
 
diff --git a/engine/src/components/tests/drive_tests.cpp b/engine/src/components/tests/drive_tests.cpp
@@ -200,22 +200,22 @@ TEST(Drive, UpgradeDowngrade) {
     EXPECT_EQ(upgrade.fuel_consumption, 1.5);
 
 // Downgrade
-    // std::cout << "Downgrade\n---------\n";
-    // // Drive
-    // upgrade.Downgrade();
+    std::cout << "Downgrade\n---------\n";
+    // Drive
+    upgrade.Downgrade();
 
-    // EXPECT_EQ(drive.GetUpgradeName(), "");
-    // EXPECT_EQ(drive.GetMass(), 0.0);
+    EXPECT_EQ(drive.GetUpgradeName(), "");
+    EXPECT_EQ(drive.GetMass(), 0.0);
 
-    // DriveExpectEq(drive, 10.0);
+    DriveExpectEq(drive, 10.0);
 
-    // EXPECT_EQ(drive.GetConsumption(), 1.0);
+    EXPECT_EQ(drive.GetConsumption(), 1.0);
 
-    // // Upgrade
-    // EXPECT_EQ(upgrade.GetUpgradeName(), "");
-    // EXPECT_EQ(upgrade.GetMass(), 0.0);
+    // Upgrade
+    EXPECT_EQ(upgrade.GetUpgradeName(), "");
+    EXPECT_EQ(upgrade.GetMass(), 0.0);
 
-    // DriveUpgradeExpectEq(upgrade, 1.0);
+    DriveUpgradeExpectEq(upgrade, 1.0);
 
-    // EXPECT_EQ(upgrade.fuel_consumption, 1.0);
+    EXPECT_EQ(upgrade.fuel_consumption, 1.0);
 }