Este post es continuación y cierre de estos dos:

TDD… y las cosas, felizmente, no volvieron a ser las mismas…

TDD, ¿por dónde empezar?

De acuerdo a lo que expresé en los post anteriores, TDD se se lleva a cabo sobre ciclos. Al finalizar cada ciclo TDD, la técnica se completa con la práctica de Refactoring.

Refactoring es la aplicación de distintas técnicas que mejoran la calidad del código. Como en TDD, se recomienda realizar cambios pequeños, a fin de garantizar un tránsito entre estados deseables, en otras palabras, mantener la integridad de los Tests.

Refactoring, también se practica sobre ciclos.

El Cliclo de Refactoring lo podemos resumir en los siguientes pasos:

  • Identificar un problema,
  • Seleccionar la técnica de Refactoring a aplicar,
  • Realizar los cambios en el código,
  • Compilar,
  • Ejecutar los Tests.
  • Recomenzar

Los cambios en código operados por el Refactoring deben ser pequeños, esto garantiza la transición de una versión a otra de forma estable.

Vuelvo al código del ejemplo anterior. Repasemos la clase de la aplicación, Recupera, está encargada de recuperar un string:

C#

public class Recupera

{

    public string Valor()

    {

        return “Prueba”;

    }

}

 

VB

Public Class Recupera

    Public Function Valor() As String

        Return “Prueba”

    End Function

End Class

 

Voy a declarar la función Recupera como estática, de esa forma no va a ser necesario instanciarla para invocarla. Veamos como queda:

C#

public class Recupera

{

    public static string Valor()

    {

        return “Prueba”;

    }

}

 

VB

Public Class Recupera

    Public Shared Function Valor() As String

        Return “Prueba”

    End Function

End Class

 

Pero si vamos a nuestra clase de Test nos encontramos con un error. Es lógico, ya que estamos instanciando una clase estática y eso no es correcto.

C#

using NUnit.Core;

using NUnit.Framework;

using SyPCS;

 

namespace SyPCS.Test

{

    [TestFixture]

    public class Class1

    {

        [Test]

        public void Prueba()

        {

            Recupera recupera = new Recupera();

            string s = recupera.Valor();

            Assert.IsNotNull(s);

        }

    }

}

VB

Imports NUnit.Core

Imports NUnit.Framework

Imports SyPVB

 

Namespace SyPVB.Test

    <TestFixture()> _

    Public Class Class1

        <Test()> _

        Public Sub Prueba()

            Dim mRecupera As New Recupera

            Dim s As String = mRecupera.Valor

            Assert.IsNotNull(s)

        End Sub

    End Class

End Namespace

 

Practicamos el Refactoring correspondiente y obtenemos

C#

using NUnit.Core;

using NUnit.Framework;

using SyPCS;

 

namespace SyPCS.Test

{

    [TestFixture]

    public class Class1

    {

        [Test]

        public void Prueba()

        {

            Assert.IsNotNull(Recupera.Valor());

        }

    }

}

 

VB

Imports NUnit.Core

Imports NUnit.Framework

Imports SyPVB

 

Namespace SyPVB.Test

    <TestFixture()> _

    Public Class Class1

        <Test()> _

        Public Sub Prueba()

            Assert.IsNotNull(Recupera.Valor)

        End Sub

    End Class

End Namespace

 

Se lee un código más sintético, evitamos la declaración de ‘s’, que, todos sabemos, es un nombre horrible.

Hagamos un Build, <Ctrl>/<Shift>/<B> y vamos a NUnit, encotramos todos los indicadores en gris, porque hicimos un build.

Ejecutamos y…

¡Verde!

Seguimos adelante.

Ahora vamos por un Refactoring que modifique Recupera de tal forma que extraiga el valor de un lugar un poco más serio, por ejemplo de un archivo de configuración, en este caso app.config como el siguiente:

<?xml version=1.0 encoding=utf-8 ?>

<configuration>

  <appSettings>

    <add key=Clave value=Prueba/>

  </appSettings>

</configuration>

 

Dentro de configuration, en el bloque appSettings agregamos una entrada que identificamos como ‘Clave’ con el valor que querramos recuperar, en este caso ‘Prueba’.

Las clases de Testing quedarían así:

C#

using NUnit.Core;

using NUnit.Framework;

using SyPCS;

 

namespace SyPCS.Test

{

    [TestFixture]

    public class Class1

    {

        System.Configuration.AppSettingsReader configurationAppSettings = new System.Configuration.AppSettingsReader();

 

        [Test]

        public void Prueba()

        {

            Assert.IsNotNull(configurationAppSettings.GetValue(“Clave”, typeof(string)));

        }

    }

}

 

VB

Imports NUnit.Core

Imports NUnit.Framework

Imports SyPVB

 

Namespace SyPVB.Test

    <TestFixture()> _

    Public Class Class1

        Public configurationAppSettings As New System.Configuration.AppSettingsReader

 

        <Test()> _

        Public Sub Prueba()

            Assert.IsNotNull(configurationAppSettings.GetValue(“Clave”, GetType(System.String)))

        End Sub

    End Class

End Namespace

 

Compilamos, no da errores, entonces hacemos un Test:

¡Verde!… A Recomenzar…

Si miramos las clases de Testing podremos ver que la referencia al proyecto SyPCS, en C#, y SyPVB, en VB, ya no son necesarios. Nuestro próximo ciclo se trata de eliminar esas referencias y su Namespace correspondiente.

Esto tiene un doble beneficio, independiza el proyecto de Testing del proyecto de desarrollo y como consecuencia resulta un código más claro de leer.

C#

using NUnit.Core;

using NUnit.Framework;

 

[TestFixture]

public class Class1

{

    System.Configuration.AppSettingsReader configurationAppSettings = new System.Configuration.AppSettingsReader();

 

    [Test]

    public void Prueba()

    {

        Assert.IsNotNull(configurationAppSettings.GetValue(“Clave”, typeof(string)));

    }

}

 

VB

Imports NUnit.Core

Imports NUnit.Framework

 

<TestFixture()> _

Public Class Class1

    Public configurationAppSettings As New System.Configuration.AppSettingsReader

 

    <Test()> _

    Public Sub Prueba()

        Assert.IsNotNull(configurationAppSettings.GetValue(“Clave”, GetType(System.String)))

    End Sub

End Class

 

Nuevamente compilamos y ejecutamos Testing:

¡Verde!…

Podemos dar por concluido nuestro Refactoring e incorporar la llamada al archivo de configuración al proyecto de desarrollo con la seguridad que el mismo funciona. Primero creamos en cada uno de ellos un archivo de configuración como los que creamos en los proyectos de Testing y luego hacemos el reemplazo:

C#

public class Recupera

{

    public static string Valor()

    {

        System.Configuration.AppSettingsReader configurationAppSettings = new System.Configuration.AppSettingsReader();

        return ((string)(configurationAppSettings.GetValue(“Clave”, typeof(string))));

    }

}

 

VB

Public Class Recupera

    Public Shared Function Valor() As String

        Dim configurationAppSettings As New System.Configuration.AppSettingsReader

        Return CType(configurationAppSettings.GetValue(“Clave”, GetType(System.String)), String)

    End Function

End Class

 

Un último detalle en nuestros proyectos de Testing.

Podemos pensar en que el valor levantado del archivo de configuración se va a utilizar en otros Test que van a irse desarrollando, entonces podemos pensar que es ineficiente que en cada Test que se ejecute, se esté recuperando exactamente el mismo valor.

Para este tipo de situaciones existe el tag <Setup> en NUnit. Este tag indica a NUnit que la rutina que clasifica es común a todos los Tests que encierra la clase.

Veamos cómo quedan nuestros ejemplos:

C#

using NUnit.Core;

using NUnit.Framework;

 

[TestFixture()]

public class Class1

{

    public System.Configuration.AppSettingsReader configurationAppSettings = new System.Configuration.AppSettingsReader();

    public string sKeyClave;

 

    [SetUp()]

    public void Setup()

    {

        sKeyClave = (string)(configurationAppSettings.GetValue(“Clave”, typeof(string)));

    }

 

    [Test()]

    public void Prueba()

    {

        Assert.IsNotNull(sKeyClave);

    }

}

 

VB

Imports NUnit.Core

Imports NUnit.Framework

 

<TestFixture()> _

Public Class Class1

    Public configurationAppSettings As New System.Configuration.AppSettingsReader

    Public sKeyClave As String

 

    <SetUp()> _

    Public Sub Setup()

        sKeyClave = CType(configurationAppSettings.GetValue(“Clave”, GetType(System.String)), String)

    End Sub

 

    <Test()> _

    Public Sub Prueba()

        Assert.IsNotNull(sKeyClave)

    End Sub

End Class

 

Y con esto estamos en condiciones de seguir practicando TDD.

Cierro este post agradeciendo los comentarios elogiosos y el aliento que me brindaron comentando en este blog y en el grupo CodeGeneration de Google a Ángel “Java” López, Martín Salías, Luis Petek y Luis Lobo Borobia.🙂