Skip to content

Commit 2af8abf

Browse files
authored
Fix WebViewsIT (apache#5388)
1 parent 1fcf88d commit 2af8abf

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

server/monitor/src/test/java/org/apache/accumulo/monitor/it/WebViewsIT.java

+3-4
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ protected void configureClient(ClientConfig config) {
8080
config.register(WebViewsIT.HashMapWriter.class);
8181
}
8282

83-
private static AtomicReference<Monitor> monitor = new AtomicReference<>(null);
83+
private static final AtomicReference<Monitor> monitor = new AtomicReference<>(null);
8484

8585
@BeforeAll
8686
public static void createMocks() throws TableNotFoundException {
@@ -123,18 +123,17 @@ public void testGetTablesConstraintViolations() {
123123
* a registered MessageBodyWriter capable of serializing/writing the object returned from your
124124
* endpoint. We're using a simple stubbed out inner class HashMapWriter for this.
125125
*
126-
* @throws Exception not expected
127126
*/
128127
@Test
129-
public void testGetTablesConstraintPassing() throws Exception {
128+
public void testGetTablesConstraintPassing() {
130129
// Using the mocks we can verify that the getModel method gets called via debugger
131130
// however it's difficult to continue to mock through the jersey MVC code for the properly built
132131
// response.
133132
// Our silly HashMapWriter registered in the configure method gets wired in and used here.
134133
Response output = target("tables/foo").request().get();
135134
assertEquals(200, output.getStatus(), "should return status 200");
136135
String responseBody = output.readEntity(String.class);
137-
assertTrue(responseBody.contains("tableID=foo") && responseBody.contains("table=bar"));
136+
assertTrue(responseBody.contains("tableId=foo") && responseBody.contains("table=bar"));
138137
}
139138

140139
/**

0 commit comments

Comments
 (0)