Fix unit test code path

358c8722 · Don Gagne · cdde6b74 · 358c8722
Commit 358c8722 authored Feb 07, 2017 by Don Gagne
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 3 deletions

SurveyMissionItem.cc src/MissionManager/SurveyMissionItem.cc +7 -3

No files found.
--- a/src/MissionManager/SurveyMissionItem.cc
+++ b/src/MissionManager/SurveyMissionItem.cc
@@ -136,9 +136,13 @@ SurveyMissionItem::SurveyMissionItem(Vehicle* vehicle, QObject* parent)

    connect(this, &SurveyMissionItem::cameraTriggerChanged,     this, &SurveyMissionItem::_cameraTriggerChanged);

-    connect(&_cameraTriggerDistanceFact,    &Fact::valueChanged,                        this, &SurveyMissionItem::timeBetweenShotsChanged);
-    connect(_vehicle,                       &Vehicle::cruiseSpeedChanged,               this, &SurveyMissionItem::timeBetweenShotsChanged);
-    connect(_vehicle,                       &Vehicle::hoverSpeedChanged,                this, &SurveyMissionItem::timeBetweenShotsChanged);
+    connect(&_cameraTriggerDistanceFact, &Fact::valueChanged, this, &SurveyMissionItem::timeBetweenShotsChanged);
+
+    // NULL check since object creation during unit testing passes NULL for vehicle
+    if (_vehicle) {
+        connect(_vehicle, &Vehicle::cruiseSpeedChanged, this, &SurveyMissionItem::timeBetweenShotsChanged);
+        connect(_vehicle, &Vehicle::hoverSpeedChanged,  this, &SurveyMissionItem::timeBetweenShotsChanged);
+    }
 }

 void SurveyMissionItem::_setSurveyDistance(double surveyDistance)