480Oswego2013 / CSC-HCI-480-2013-repo

8 stars 8 forks source link

InspectBuilderTest #23

Closed kevinwinahradsky closed 11 years ago

kevinwinahradsky commented 11 years ago

When catching exceptions that are unexpected, you should call fail() to fail the test.

There should be some more tests for setView() and setOffset() make sure they are checking bounds correctly by passing in numbers outside of their acceptable range. When doing this the test should fail if the exception is not thrown and pass if it is thrown.

Tests should be added to make sure all methods and constructors correctly guard against null arguments pass into them. They probably don't so the InspectBuilder code would need to be updated to handle this.