search

fommil / matrix-toolkits-java

:rocket: High Performance Linear Algebra OOP
GNU Lesser General Public License v3.0
362 stars 104 forks source link

Junit4 Upgrade #78

Closed randyp closed 8 years ago

randyp commented 8 years ago

Upgrade to fully use junit4, which uses annotations to mark tests and specify simple setup/teardown methods. More complex setup/teardown/interceptor patterns can be employed with Rule and ClassRule.

To test the upgrade, I counted the # of tests run before and after upgrade, which both total 2453.

It also appears my IDE removed some redundant whitespace, let me know if that is a problem and will replace it.

Command to count tests per class: mvn clean test | grep '^Tests run:' | cut -d\ -f16 -f3' Example Output:

4, no.uib.cipr.matrix.BandCholeskyTest
3, no.uib.cipr.matrix.BandLUTest
61, no.uib.cipr.matrix.BandMatrixTest
4, no.uib.cipr.matrix.DenseCholeskyTest
4, no.uib.cipr.matrix.DenseLUTest
63, no.uib.cipr.matrix.DenseMatrixTest
23, no.uib.cipr.matrix.DenseVectorSubTest
23, no.uib.cipr.matrix.DenseVectorTest
2, no.uib.cipr.matrix.io.MatrixVectorIoTest
3, no.uib.cipr.matrix.KhatriRaoTest
61, no.uib.cipr.matrix.LowerSPDBandMatrixTest
61, no.uib.cipr.matrix.LowerSPDDenseMatrixTest
61, no.uib.cipr.matrix.LowerSPDPackMatrixTest
61, no.uib.cipr.matrix.LowerSymmBandMatrixTest
...
...
61, no.uib.cipr.matrix.UpperTriangDenseMatrixTest
61, no.uib.cipr.matrix.UpperTriangPackMatrixTest
2453,
fommil commented 8 years ago

Please squash commits into one logical commit.

randyp commented 8 years ago

done