Minor changes to the README, mainwindow.cpp, pro, main.cpp for the unit tests

README.md

@@ -301,19 +301,19 @@ Este ejercicio **NO requiere programación**, debes hacer las pruebas **sin mira
 
 ###Ejercicio 3: Usar `assert` para realizar pruebas unitarias
 
-Hacer pruebas "a mano" cada vez que corres un programa es una tarea que resulta "cansona" bien rápido. En los ejercicios anteriores lo hiciste para unas pocas funciones simples. !Imagínate hacer lo mismo para un programa complejo completo como un buscador o un procesador de palabras!
+Hacer pruebas "a mano" cada vez que corres un programa es una tarea que resulta "cansona" bien rápido. En los ejercicios anteriores lo hiciste para unas pocas funciones simples. ¡Imagínate hacer lo mismo para un programa complejo completo como un navegador o un procesador de palabras!
 
 Las *pruebas unitarias* ayudan a los programadores a validar códigos y simplificar el proceso de depuración ("debugging") a la vez que evitan la tarea tediosa de hacer pruebas a mano en cada ejecución.
 
 ####Instrucciones:
 
-1. En el menú de QT, ve a `Build` y selecciona `Clean Project "Testing"`.
+1. En el menú de QtCreator, ve a `Build` y selecciona `Clean Project "Testing"`. Luego ve a `File` y selecciona `Close Project "Testing"`.
 
-2. Carga a Qt el proyecto `UnitTests`  haciendo doble "click" en el archivo `UnitTests.pro` en el directorio `Documents/eip/Testing` de tu computadora. También puedes ir a `http://bitbucket.org/eip-uprrp/testing` para descargar la carpeta `Testing` a tu computadora.
+2. Carga a QtCreator el proyecto `UnitTests` haciendo doble "click" en el archivo `UnitTests.pro` en el directorio `Documents/eip/Testing/UnitTests` de tu computadora. Si descargaste el directorio `Testing` a tu computadora durante el ejercicio 2, el directorio `UnitTests` estará bajo ese directorio.
 
 3. El proyecto solo contiene el archivo de código fuente `main.cpp`. Este archivo contiene cuatro funciones: `fact`, `isALetter`, `isValidTime`, y `gcd`, cuyos resultados son solo parcialmente correctos. 
 
-  Estudia la descripción de  cada una de las funciones que aparece como comentarios antes el código de la función para saber la tarea que se espera haga la función.
+  Estudia la documentación de cada función (los comentarios que aparecen previo a cada función) para que comprendas la tarea que se espera que haga cada función.
 
   Tu tarea es escribir pruebas unitarias para cada una de las funciones para identificar los resultados erróneos. ** No necesitas reescribir las funciones para corregirlas. **
 
@@ -669,9 +669,9 @@ Doing tests by hand each time you run a program is a tiresome task. In the previ
 
 ####Instructions:
 
-1. In the Qt menu, go to `Build` and select `Clean Project "Testing"`.
+1. In the QtCreator menu, go to `Build` and select `Clean Project "Testing"`. Then go to `File` and select `Close Project "Testing"`.
 
-2. Load the `UnitTests` project onto Qt by double clicking the `UnitTests.pro` file in the `Documents/eip/Testing` directory on your computer. You can also go to `http://bitbucket.org/eip-uprrp/testing` to download the `Testing` folder on your computer.
+2. Load the `UnitTests` project onto QtCreator by double clicking the `UnitTests.pro` file in the `Documents/eip/Testing` directory on your computer. If you downloaded the `Testing` directory in Exercise 2, the  `UnitTests` directory should inside `Testing`.
 
 3. The project only contains the source code file `main.cpp`. This file contains four functions: `fact`, `isALetter`, `isValidTime`, and `gcd`, whose results are only partially correct.
 

Testing.pro

@@ -25,3 +25,5 @@ FORMS    += mainwindow.ui
 
 RESOURCES += \
     images.qrc
+
+DEFINES     += QT_NO_DEBUG_OUTPUT

UnitTests/main.cpp

@@ -12,7 +12,7 @@ unsigned int fact(unsigned int n) {
 
     int result = 1;
 
-    for (int i = 1; i < n; i++)
+    for (unsigned int i = 1; i < n; i++)
         result = result * i;
 
     return result;
@@ -71,60 +71,14 @@ void test_fact() {
 }
 
 //
-// EXERCISE 3
+// EXERCISE 3: Write your test functions here!!!
 //
 
 
-//
-// Function test_isALetter():
-// This function is provided as an example unit test for the isALetter() function.
-//
-
-void test_isALetter() {
-    assert( isALetter('a') );
-    assert( isALetter('Z'));
-    assert( isALetter('2')==0);
-    assert( isALetter('$')==0);
-    assert( isALetter('^')==0);
-    cout << "Function isALetter() passed all the tests!!" << endl;
-}
-
-//
-// Function test_isValidTime():
-// This function is provided as an example unit test for the isValidTime() function.
-//
-
-void test_isValidTime() {
-    assert( isValidTime(0002) );
-    assert( isValidTime(1234) );
-    assert( isValidTime(4800)==0 );
-    assert( isValidTime(1278)==0 );
-    cout << "Function isValidTime() passed all the tests!!" << endl;
-}
-
-//
-// Function test_gcd():
-// This function is provided as an example unit test for the gcd() function.
-//
-
-void test_gcd() {
-    assert( gcd(1,1)==1 );
-    assert( gcd(2,2)==2 );
-    assert( gcd(3,2)==1 );
-    assert( gcd(12,9)==3 );
-    assert( gcd(12,6)==6 );
-    cout << "Function gcd() passed all the tests!!" << endl;
-}
-
-
-
 int main()
 {
     cout << "Go ahead and test!\n";
     test_fact();
- // test_gcd();
- // test_isALetter();
- // test_isValidTime();
-    
+ 
     return 0;
 }

mainwindow.cpp

@@ -378,7 +378,7 @@ void MainWindow::RPSs(){
 #include <QTimer>
 
 void MainWindow::paintDice() {
-    qDebug() << this->timerCounter++;
+    this->timerCounter++;
     int a, b;
     a = rand()%6;
     b = rand()%6;