[chronojump] guiTests more responsive and with report at end

[Xavier de Blas <xaviblas src gnome org>]

commit f27529f30fdfa7329f52964c32eb6da3c5918bf2
Author: Xavier de Blas <xaviblas gmail com>
Date:   Fri Oct 21 00:13:15 2016 +0200

    guiTests more responsive and with report at end

 src/chronopic2016.cs |    4 ++--
 src/gui/guiTests.cs  |   33 +++++++++++++++++++++++++++------
 2 files changed, 29 insertions(+), 8 deletions(-)
---
diff --git a/src/chronopic2016.cs b/src/chronopic2016.cs
index 18915ae..b1a9aa1 100644
--- a/src/chronopic2016.cs
+++ b/src/chronopic2016.cs
@@ -335,9 +335,9 @@ public class Chronopic2016
        }
 
        //public method to access from guiTests.cs
-       public void TestsChangeMultitestFirmwareDo (int debounceChange)
+       public int TestsChangeMultitestFirmwareDo (int debounceChange)
        {
-               changeMultitestFirmwareDo(debounceChange);
+               return changeMultitestFirmwareDo(debounceChange);
        }
 
        // <----- change multitest firmware END
diff --git a/src/gui/guiTests.cs b/src/gui/guiTests.cs
index 1aef7d7..fa75120 100644
--- a/src/gui/guiTests.cs
+++ b/src/gui/guiTests.cs
@@ -224,12 +224,17 @@ public partial class ChronoJumpWindow
        private static bool testsActive = false;
        private CJTests cjTest;
        private int sessionID;
-       
+       private int testsSuccededCount;
+       private int testsFailedCount;
+
        private void chronojumpWindowTestsStart(int sessionID, List<CJTests.Types> sequence)
        {
                testsActive = true;
                this.sessionID = sessionID;
 
+               testsSuccededCount = 0;
+               testsFailedCount = 0;
+
                cjTest = new CJTests(sequence);
 
                chronojumpWindowTestsDo();
@@ -237,8 +242,10 @@ public partial class ChronoJumpWindow
        
        public void chronojumpWindowTestsNext() 
        {
-               if(! testsActive) 
+               if(! testsActive) {
+                       testsReport();
                        return;
+               }
 
                //needed for SequenceContactsExecute50_10 (but doesn't work properly). Find other solution:
                //http://www.mono-project.com/docs/gui/gtksharp/responsive-applications/
@@ -246,14 +253,16 @@ public partial class ChronoJumpWindow
                //force GUI update
                //System.Threading.Thread.Sleep(250); //ok there's an sleep, but screen is not updated. Maybe 
need to open a new thread before each next call to ensure gui is updated
                //http://mono.1490590.n4.nabble.com/Force-UI-Update-td3488126.html
-               //while (Gtk.Application.EventsPending ())
-               //              Gtk.Application.RunIteration ();
+               while (Gtk.Application.EventsPending ())
+                               Gtk.Application.RunIteration ();
                //System.Threading.Thread.Sleep(250); //ok there's an sleep, but screen is not updated. Maybe 
need to open a new thread before each next call to ensure gui is updated
 
                if(cjTest.Next())
                        chronojumpWindowTestsDo();
-               else
+               else {
                        testsActive = false;
+                       testsReport();
+               }
        }
 
        private void chronojumpWindowTestsDo() 
@@ -400,7 +409,11 @@ public partial class ChronoJumpWindow
        {
                LogB.TestStart("chronojumpWindowTestsContactsMultitestDo " + ms.ToString());
 
-               cp2016.TestsChangeMultitestFirmwareDo (ms);
+               int msRecorded = cp2016.TestsChangeMultitestFirmwareDo (ms);
+               if(msRecorded == ms)
+                       testsSuccededCount ++;
+               else
+                       testsFailedCount ++;
 
                LogB.TestEnd("chronojumpWindowTestsContactsMultitestDo " + ms.ToString());
                return true;
@@ -537,6 +550,14 @@ public partial class ChronoJumpWindow
                }
        }
        
+       void testsReport()
+       {
+               if(testsSuccededCount > 0 || testsFailedCount > 0)
+                       new DialogMessage(Constants.MessageTypes.INFO,
+                                       string.Format("Tests succeded: {0}\nTests failed: {1}",
+                                               testsSuccededCount, testsFailedCount));
+       }
+
        /*
         * TESTS END
         */