Unit tests: QBaseVector4, QBaseVector2, QVector2

[GoogleCodeExporter]

Hay que crear unit tests para las clases:

-QBaseVector4
-QVector4
-QBaseVector2
-QVector2

El grado de cobertura debe ser alto.

Original issue reported on code.google.com by Lince3D@gmail.com on 23 Jan 2012 at 12:11

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 26 Jan 2012 at 8:58

• Changed state: Started

[GoogleCodeExporter]

Ahora sólo incluye QBaseVector4, QBaseVector2, QVector2.

Original comment by Lince3D@gmail.com on 10 Mar 2012 at 12:55

• Changed title: Unit tests: QBaseVector4, QBaseVector2, QVector2

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 26 Mar 2012 at 5:57

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-Constructor4_ExceptionIsNotThrownWhenArrayIsNotFilled_Test: Este habría que 
quitarlo. Jmartin tenía razón en que no tenía mucho sentido (de QBaseVector3 
lo quité ya si no recuerdo mal).

-Usings en general: Vamos a tratar de meter los usings dentro de los métodos 
que los necesiten realmente, cuando hagamos unit tests. Esto permite darnos 
cuenta muchas veces de errores debidos al copypaste.

-FriendOperatorProduct1_ScalarIsCorrectlyMultipliedByVector_Test: Sobra el 1, 
dado que no hay más sobrecargas. Además, procuraremos poner los tests en el 
mismo orden en que nos encontremos los métodos de la clase a testear. Esto no 
hace falta que lo cambies, es una formalidad para mejorar la búsqueda de tests.

-Constructor2_ValuesAreSetProperly_Test, 
Constructor1_ValuesAreSetProperly_Test: Está repetido. Ver siguiente punto.

-Numeración de constructores: Las numeraciones de sobrecargas empezarán 
siempre desde 1. Supongo que lo tenía antes mal yo en QVector3. Creo que para 
QVector2 te valdrá cambiar el 0 por el 1 y ya lo tienes.

-OperatorProductAssignation1_VectorComponentIsCorrectlyMultipliedByScalarAndAssi
gnedToItself_Test: Hay un TODO que ya no sirve.

-Haría falta incluir un tests para OperatorDivisionAssignation2, que 
verificara que un vector puede ser dividido por sí mismo (mismo objeto). Éste 
test también falta en QVector3 y me temo que para QVector4. Si quieres me lo 
apunto y lo hago para los 3.

-GetPerpendicular_PerpendicullarOfACommonVectorIsCalculatedCorrectly_Test: 
Sobra una L, y en la descripción también, es "perpendicular". Una cosa, para 
ponértelo de ejemplo, la forma más correcta de comprobar que ha devuelto un 
vector perpendicular sería calculando que hay 90º entre ellos. Así, pongas 
lo que pongas como entrada, siempre va a validar el test. No hace falta que lo 
cambies, es sólo para que le pilles el tranquillo, cuanto más universal sea 
el test mejor (a menos que, como siempre, eso suponga un esfuerzo que no 
merezca la pena).

-Lerp1_CommonVectorIsCorrectlyInterpolated_Test: Sobra el 1.

-Transform6_VectorIsNullWhenTransformationMatrixIsNull_Test: Copypaste fallido 
;P

-Falta incluir los tests en el proyecto de Code::Blocks.

Original comment by Lince3D@gmail.com on 27 Mar 2012 at 8:11

• Changed state: Started

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 3 Apr 2012 at 10:00

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Falta aplicar esta a QBaseVector2 y QBaseVector4 --> 
Constructor4_ExceptionIsNotThrownWhenArrayIsNotFilled_Test: Este habría que 
quitarlo. Jmartin tenía razón en que no tenía mucho sentido (de QBaseVector3 
lo quité ya si no recuerdo mal).

-Falta aplicar esta a QBaseVector2 y QBaseVector4 --> 
Usings en general: Vamos a tratar de meter los usings dentro de los métodos 
que los necesiten realmente, cuando hagamos unit tests. Esto permite darnos 
cuenta muchas veces de errores debidos al copypaste.

-Sobra una línea comentada al principio del fichero de test de QVector2.

-La numeración de los unit tests de Constructor de QVector2 está mal, aparece 
como 1, 1, 2, 3, 4 y 5, por lo que habría que incrementar en uno desde el 
segundo unit test.

Original comment by Lince3D@gmail.com on 3 May 2012 at 6:18

• Changed state: Done