Unit tests for ONOS are built using the JUnit framework, version 4.11. Unit tests are used to verify correctness of implementations and are run as part of every full build of ONOS. The nightly SonarQube build runs all of the unit tests and produces coverage data that can be seen here.
How to Write Good Tests
Unit tests should be as short, fast and reliable as they can be. Every developer on the project will be running your tests, they should be easy to run and produce accurate results. Here are some key ways to write tests that are not brittle:
- Don't use sleep(), since it often leads to brittle tests. If you find that you have to wait for an event or wait for some work to be done by another thread, prefer latches or thread notifications to sleeps.
- Try to keep individual tests small, and only test one thing per test.
- Use mocking when you need to include a complicated service to satisfy dependencies. ONOS uses the EasyMock framework, version 3.2.
- Maven may choose to run multiple tests in the same Java virtual machine. If you use static variables in classes, be sure to reset them to known starting values before each test runs.
- Do not use local resources like files and IP Addresses in tests.
Basic Tests
If you add a class, consider adding the following basic tests for it first:
- Any class that defines equals(), hashCode(), or toString() must test these methods. The Google Guava EqualsTester class provides full coverage for these methods:
final FlowId flowId1 = FlowId.valueOf(1); final FlowId sameAsFlowId1 = FlowId.valueOf(1); final FlowId flowId2 = FlowId.valueOf(2); /** * Checks the operation of equals(), hashCode() and toString() methods. */ @Test public void testEquals() { new EqualsTester() .addEqualityGroup(flowId1, sameAsFlowId1) .addEqualityGroup(flowId2) .testEquals(); }
- Utility test methods are provided to assure that classes are properly defined immutable classes, immutable base classes or utility classes. Using these methods will assure that other developers don't make changes to your classes that violate your assumptions.
import static org.onlab.junit.ImmutableClassChecker.assertThatClassIsImmutable; /** * Checks that the FlowId class is immutable. */ @Test public void testImmutability() { assertThatClassIsImmutable(FlowId.class); }
import static org.onlab.junit.ImmutableClassChecker.assertThatClassIsImmutableBaseClass; /** * Checks that the DefaultFlowRule class is immutable but can be inherited * from. */ @Test public void testImmutability() { assertThatClassIsImmutableBaseClass(DefaultFlowRule.class); }
import static org.onlab.junit.UtilityClassChecker.assertThatClassIsUtility; /** * Check that the Criteria class is a valid utility class. */ @Test public void testCriteriaUtility() { assertThatClassIsUtility(Criteria.class); }
- A basic test for each one of your constructors and a check for the values returned by data access methods should always be included:
/** * Checks the construction of a FlowId object. */ @Test public void testConstruction() { final long flowIdValue = 7777L; final FlowId flowId = FlowId.valueOf(flowIdValue); assertThat(flowId, is(notNullValue())); assertThat(flowId.value(), is(flowIdValue)); }
Hamcrest is a powerful framework for defining matchers for data values in tests. ONOS uses version 1.3 of Hamcrest. Contributors are encouraged to learn about and use the Hamcrest framework, particularly when working with arrays and collections.