From d88084a342df55d6bfd7c0773d3c5f1211f3f7f0 Mon Sep 17 00:00:00 2001
From: sawinglogz <3787776-sawinglogz@users.noreply.gitlab.com>
Date: Fri, 10 Jul 2020 14:55:25 -0400
Subject: [PATCH] Disable oximeter download tests by default for now.

They're not yet stable enough to be enabled as regression tests in master.
---
 oscar/tests/deviceconnectiontests.cpp | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/oscar/tests/deviceconnectiontests.cpp b/oscar/tests/deviceconnectiontests.cpp
index 100b79d8..ed944c19 100644
--- a/oscar/tests/deviceconnectiontests.cpp
+++ b/oscar/tests/deviceconnectiontests.cpp
@@ -92,7 +92,9 @@ void DeviceConnectionTests::testSerialPortScanning()
     list3 = SerialPortInfo::availablePorts();
 }
 
+#define ENABLE 0
 
+#if ENABLE
 static void testDownload(const QString & loaderName)
 {
     SerialOximeter * oxi = qobject_cast<SerialOximeter*>(lookupLoader(loaderName));
@@ -125,6 +127,7 @@ static void testDownload(const QString & loaderName)
     }
     oxi->trashRecords();
 }
+#endif
 
 void DeviceConnectionTests::testOximeterConnection()
 {
@@ -135,6 +138,7 @@ void DeviceConnectionTests::testOximeterConnection()
     const char* argv = "test";
     QCoreApplication app(argc, (char**) &argv);
     
+#if ENABLE
     DeviceConnectionManager & devices = DeviceConnectionManager::getInstance();
     /*
     QString string;
@@ -165,6 +169,10 @@ void DeviceConnectionTests::testOximeterConnection()
     qDebug().noquote() << string;
     */
     
+    QFile out("test.xml");
+    Q_ASSERT(out.open(QFile::ReadWrite));
+    devices.record(&out);
+
     QFile file("cms50f37.xml");
     if (!file.exists()) {
         qDebug() << "Recording oximeter connection";
@@ -181,4 +189,5 @@ void DeviceConnectionTests::testOximeterConnection()
     testDownload(cms50f37_class_name);
     devices.replay(nullptr);
     file.close();
+#endif
 }