Unit Tests: QScalingMatrix3x3

[GoogleCodeExporter]

Implementar los unit tests para la clase QScalingMatrix3x3.

Original issue reported on code.google.com by Lince3D@gmail.com on 26 Apr 2012 at 1:14

[GoogleCodeExporter]

Pillada

Original comment by borderpa...@gmail.com on 8 May 2012 at 1:53

• Changed state: Started

[GoogleCodeExporter]

-Nuevo archivo: QScalingMatrix3x3_Test.cpp con 23 nuevos casos de prueba para 
QScalingMatrix3x3. Basados en QRotationMatrix3x3_Test
-Nuevo archivo: QScalingMatrix3x3WhiteBox.h con las definiciones necesarias 
para implementar los tests de caja blanca de QScalingMatrix3x3. Basados en 
QRotationMatrix3x3WhiteBox.h
-Modificación en la documentación de QScalingMatrix3x3.h. Entiendo que el 
operador de producto entre dos QScalingMatrix3x3 SI debe ser conmutativo.
-Modificaciones menores de formato en QRotationMatrix3x3_Test. Añadido algún 
sufijo f de float y suprimida alguna directiva using innecesaria.

Original comment by borderpa...@gmail.com on 14 May 2012 at 7:11

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Constructor3_ScaleMatrixCorrectlyBuildedFromThreeScalingValues_Test: Tanto el 
nombre como en la documentación, "builded" => "built".

-Constructor4_ScaleMatrixCorrectlyBuildedFrom3DVector_Test: Tanto el nombre 
como en la documentación, "builded" => "built".

-Constructor4_NeutralScaleMatrixIsObtainedWhenScaling3DVectorComponentsAreEqualT
oOne_Test: "AreEqualToOne" => "EqualOne". Además, en la documentación "3D 
vector are equal to one." => "3D vector equal one".

-GetScale_ValuesAreCorrectlyExtracted_Test: Las variables que no son constantes 
no deben ponerse en mayúsculas. En la implementación de nuestros tests se 
entiende como constante aquello que no está pensado para que cambie durante la 
prueba. Por otro lado, el sufijo UT lo solemos poner a las variables que van a 
ser comprobadas.

-Documentación de GetScale_3DVectorValuesAreCorrectlyExtracted_Test: "Checks 
that 3D vector obtained scaling values equal expected values." está un poco 
raro, ¿falta borrar palabras?

-GetScale_3DVectorValuesAreCorrectlyExtracted_Test: Las variables que no son 
constantes no deben ponerse en mayúsculas. En la implementación de nuestros 
tests se entiende como constante aquello que no está pensado para que cambie 
durante la prueba. Por otro lado, el sufijo UT lo solemos poner a las variables 
que van a ser comprobadas.

-GetScale#_XXX: Falta la numeración.

-Documentación de GetDeterminant_ValueIsCorrect_Test: "Checks that determinant 
obtained value equal expected one" => "Checks that obtained determinant value 
equal expected one".

-GetDeterminant_ValueIsCorrect_Test: Las variables que no son constantes no 
deben ponerse en mayúsculas. En la implementación de nuestros tests se 
entiende como constante aquello que no está pensado para que cambie durante la 
prueba. Por otro lado, el sufijo UT lo solemos poner a las variables que van a 
ser comprobadas.

-ProductOperatorImp1_CommonMatricesAreCorrectlyMultiplied_MatrixTypeQMatrix4x3_M
ethod y 
ProductOperatorImp1_CommonMatricesAreCorrectlyMultiplied_MatrixTypeQMatrix4x4_Me
thod: Usas el operador de multipliación en la prueba en lugar de el método 
ProductOperatorImp1.

-ProductOperatorImp2_CommonMatricesAreCorrectlyMultiplied_MatrixTypeQMatrix4x3_M
ethod y 
ProductOperatorImp2_CommonMatricesAreCorrectlyMultiplied_MatrixTypeQMatrix4x4_Me
thod: Usas el operador de multipliación en la prueba en lugar de el método 
ProductOperatorImp2.

-OperatorProduct2_CommonMatricesAreCorrectlyMultiplied_Test: Fallan algunas 
pruebas. Si el resultado esperado calculado por otros medios difiere 
mínimamente con el obtenido en la prueba (una cantidad razonablemente 
despreciable), copia y pega el valor obtenido en la prueba para que el test 
pase.

Original comment by Lince3D@gmail.com on 15 May 2012 at 8:43

• Changed state: Started

[GoogleCodeExporter]

-Correcciones hechas e implementados nuevos test de no influencia de elementos 
no diagonales en productos.

Original comment by borderpa...@gmail.com on 17 May 2012 at 4:04

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

Original comment by Lince3D@gmail.com on 17 May 2012 at 5:43

• Changed state: Done