Fix tests that were hiding the setUp() mthod in the JerseyTest framework
REST API tests were defining a set up method called setUp(), which had
the unwanted side effect of hinding the parent class' implementation.
Made the set up and tear down methods for the tests have unique names so
both will be executed.
Also fixed some javadocs while editing the files.
Change-Id: Ifaa75fe650d5909b066769e5e12cb5c834267a20
diff --git a/web/api/src/test/java/org/onosproject/rest/HostResourceTest.java b/web/api/src/test/java/org/onosproject/rest/HostResourceTest.java
index 2d9cdeb..a5c4b77 100644
--- a/web/api/src/test/java/org/onosproject/rest/HostResourceTest.java
+++ b/web/api/src/test/java/org/onosproject/rest/HostResourceTest.java
@@ -67,8 +67,11 @@
final HostService mockHostService = createMock(HostService.class);
final HashSet<Host> hosts = new HashSet<>();
+ /**
+ * Initializes test mocks and environment.
+ */
@Before
- public void setUp() {
+ public void setUpTest() {
expect(mockHostService.getHosts()).andReturn(hosts).anyTimes();
// Register the services needed for the test
@@ -82,9 +85,11 @@
BaseResource.setServiceDirectory(testDirectory);
}
+ /**
+ * Verifies mocks.
+ */
@After
- public void tearDown() throws Exception {
- super.tearDown();
+ public void tearDownTest() {
verify(mockHostService);
}