From b722c80b3f9f367fdc5758064c4110ecc8216221 Mon Sep 17 00:00:00 2001
From: NissimHadar <nissim@highfidelity.io>
Date: Tue, 1 May 2018 14:25:33 -0700
Subject: [PATCH] WIP

---
 tools/auto-tester/src/ui/AutoTester.cpp | 12 ++++++++----
 tools/auto-tester/src/ui/AutoTester.h   |  6 +++---
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/tools/auto-tester/src/ui/AutoTester.cpp b/tools/auto-tester/src/ui/AutoTester.cpp
index 3f7d2cba28..860868565a 100644
--- a/tools/auto-tester/src/ui/AutoTester.cpp
+++ b/tools/auto-tester/src/ui/AutoTester.cpp
@@ -10,8 +10,6 @@
 //
 #include "AutoTester.h"
 
-#include <QMessageBox>
-
 AutoTester::AutoTester(QWidget *parent) : QMainWindow(parent) {
     ui.setupUi(this);
     ui.checkBoxInteractiveMode->setChecked(true);
@@ -91,11 +89,17 @@ void AutoTester::saveImage(int index) {
     QPixmap pixmap;
     pixmap.loadFromData(downloaders[index]->downloadedData());
 
+    int sdf = pixmap.width();
     QImage image = pixmap.toImage();
     image = image.convertToFormat(QImage::Format_ARGB32);
 
     QString fullPathname = _directoryName + "/" + _filenames[index];
-    image.save(fullPathname, 0, 100);
+    if (!image.save(fullPathname, 0, 100)) {
+        QMessageBox messageBox;
+        messageBox.information(0, "Test Aborted", "Failed to save image: " + _filenames[index]);
+        ui.progressBar->setVisible(false);
+        return;
+    }
 
     ++_numberOfImagesDownloaded;
 
@@ -109,4 +113,4 @@ void AutoTester::saveImage(int index) {
 void AutoTester::about() {
     QMessageBox messageBox;
     messageBox.information(0, "About", QString("Built ") + __DATE__ + " : " + __TIME__);
-}
\ No newline at end of file
+}
diff --git a/tools/auto-tester/src/ui/AutoTester.h b/tools/auto-tester/src/ui/AutoTester.h
index 03cb7fbcec..d911b6aaff 100644
--- a/tools/auto-tester/src/ui/AutoTester.h
+++ b/tools/auto-tester/src/ui/AutoTester.h
@@ -52,9 +52,9 @@ private:
     // Used to enable passing a parameter to slots
     QSignalMapper* signalMapper;
 
-    int _numberOfImagesToDownload;
-    int _numberOfImagesDownloaded;
-    int _index;
+    int _numberOfImagesToDownload { 0 };
+    int _numberOfImagesDownloaded { 0 };
+    int _index { 0 };
 };
 
 #endif // hifi_AutoTester_h
\ No newline at end of file