search

nmaguirre / minesweeper

A Simple MineSweeper Project
6 stars 4 forks source link

minesweeperBoard.getBoardCols has no test #76

Closed gamassimino closed 8 years ago

gamassimino commented 8 years ago

MinesweeperBoard.getBoardCols should have tests checking its expected functionality in a number of different scenarios. Statement coverage is expected from the tests of this routine.

lbuttignol commented 8 years ago

Solved in this rev a7d0a69

nricci commented 8 years ago

A test should end with an assertion. There is no point in testing other things after the first assertion (if, for instance, it were to fail). In this case you can, and should, make two tests.

juanpabloRV commented 8 years ago

Missing issue description

lbuttignol commented 8 years ago

Solved in this rev. 20ec21f

nricci commented 8 years ago

Seems alright. Closing for now.