Add boot testing to release builds

.github/workflows/windows.yml

@@ -110,6 +110,14 @@ jobs:
         working-directory: ${{ runner.temp }}\shadow_build_dir
         run: cmake --build . --target all --config ${{ matrix.BuildType }}
 
+      - name: Sanity check release builds
+        if: matrix.BuildType == 'Release'
+        working-directory: ${{ runner.temp }}/shadow_build_dir/Release
+        env:
+          QT_QUICK_BACKEND: "software"
+          QT_QPA_PLATFORM: "offscreen"
+        run: QGroundControl.exe --release-boot-test
+
       - name: Install
         working-directory: ${{ runner.temp }}\shadow_build_dir
         run: cmake --install . --config ${{ matrix.BuildType }}

src/QGCApplication.cc

@@ -112,18 +112,18 @@ QGCApplication::QGCApplication(int &argc, char *argv[], bool unitTesting)
     QNetworkProxyFactory::setUseSystemConfiguration(true);
 
     // Parse command line options
-    bool fClearSettingsOptions = false; // Clear stored settings
-    bool fClearCache = false;           // Clear parameter/airframe caches
-    bool logging = false;               // Turn on logging
+    bool fClearSettingsOptions = false;     // Clear stored settings
+    bool fClearCache = false;               // Clear parameter/airframe caches
+    bool logging = false;                   // Turn on logging
     QString loggingOptions;
 
     CmdLineOpt_t rgCmdLineOptions[] = {
-        { "--clear-settings",   &fClearSettingsOptions, nullptr },
-        { "--clear-cache",      &fClearCache,           nullptr },
-        { "--logging",          &logging,               &loggingOptions },
-        { "--fake-mobile",      &_fakeMobile,           nullptr },
-        { "--log-output",       &_logOutput,            nullptr },
-        // Add additional command line option flags here
+        { "--clear-settings",       &fClearSettingsOptions,     nullptr },
+        { "--clear-cache",          &fClearCache,               nullptr },
+        { "--logging",              &logging,                   &loggingOptions },
+        { "--fake-mobile",          &_fakeMobile,               nullptr },
+        { "--log-output",           &_logOutput,                nullptr },
+        { "--release-boot-test",    &_runReleaseBuildBootTest,  nullptr },
     };
 
     ParseCmdLineOptions(argc, argv, rgCmdLineOptions, std::size(rgCmdLineOptions), false);

src/QGCApplication.h

@@ -60,6 +60,9 @@ class QGCApplication : public QApplication
     /// Returns true if unit tests are being run
     bool runningUnitTests() const { return _runningUnitTests; }
 
+    /// Returns true if the application is running a release build boot test
+    bool runReleaseBuildBootTest() const { return _runReleaseBuildBootTest; }
+
     /// Returns true if Qt debug output should be logged to a file
     bool logOutput() const { return _logOutput; }
 
@@ -135,6 +138,7 @@ private slots:
     void _checkForNewVersion();
 
     bool _runningUnitTests = false;                                         ///< true: running unit tests, false: normal app
+    bool _runReleaseBuildBootTest = false;                                  ///< true: Run simple boot test to verify release builds aren't hosed in CI
     static constexpr int _missingParamsDelayedDisplayTimerTimeout = 1000;   ///< Timeout to wait for next missing fact to come in before display
     QTimer _missingParamsDelayedDisplayTimer;                               ///< Timer use to delay missing fact display
     QList<QPair<int,QString>> _missingParams;                               ///< List of missing parameter component id:name

src/QmlControls/QGroundControlQmlGlobal.cc

@@ -429,3 +429,8 @@ void QGroundControlQmlGlobal::clearDeleteAllSettingsNextBoot()
 {
     qgcApp()->clearDeleteAllSettingsNextBoot();
 }
+
+bool QGroundControlQmlGlobal::runBootTest() 
+{ 
+    return qgcApp()->runReleaseBuildBootTest(); 
+}

src/QmlControls/QGroundControlQmlGlobal.h

@@ -94,6 +94,7 @@ class QGroundControlQmlGlobal : public QObject
     Q_PROPERTY(QGeoCoordinate       flightMapPosition       READ    flightMapPosition       WRITE setFlightMapPosition  NOTIFY flightMapPositionChanged)
     Q_PROPERTY(double               flightMapZoom           READ    flightMapZoom           WRITE setFlightMapZoom      NOTIFY flightMapZoomChanged)
     Q_PROPERTY(double               flightMapInitialZoom    MEMBER  _flightMapInitialZoom   CONSTANT)   ///< Zoom level to use when either gcs or vehicle shows up for first time
+    Q_PROPERTY(bool                 runBootTest             READ    runBootTest             CONSTANT)
 
     Q_PROPERTY(QString  parameterFileExtension  READ parameterFileExtension CONSTANT)
     Q_PROPERTY(QString  missionFileExtension    READ missionFileExtension   CONSTANT)
@@ -215,6 +216,7 @@ class QGroundControlQmlGlobal : public QObject
     bool    singleVehicleSupport    ();
     bool    px4ProFirmwareSupported ();
     bool    apmFirmwareSupported    ();
+    bool    runBootTest             ();
 
     void    setIsVersionCheckEnabled    (bool enable);
     void    setMavlinkSystemID          (int  id);

src/UI/MainRootWindow.qml

@@ -45,6 +45,16 @@ ApplicationWindow {
 
         // Start the sequence of first run prompt(s)
         firstRunPromptManager.nextPrompt()
+
+        if (QGroundControl.runBootTest) {
+            bootTestTimer.start()
+        }
+    }
+
+    Timer {
+        id:             bootTestTimer
+        interval:       1000
+        onTriggered:    mainWindow.finishCloseProcess()
     }
 
     QtObject {