karthick-rn commented on pull request #2020:
URL: https://github.com/apache/accumulo/pull/2020#issuecomment-822422194
> I think this caused a failure in the WebViewsIT integration test. It's
probably due to a test case against one of the removed REST endpoints that
should also be removed, but I haven't investigated yet.
@ctubbsii To investigate this, I ran `mvn clean verify -Dit.test=WebViewsIT
-Dtest=foo` which resulted in the below failure.
> [ERROR] Failures:
[ERROR] WebViewsIT.finishMocks:107 On mock #0 (zero indexed):
Unexpected method calls:
ServerContext.getZooKeepers()
[ERROR] WebViewsIT.testGetTablesConstraintPassing:143 should return status
200 expected:<200> but was:<500>
[INFO]
[ERROR] Tests run: 4, Failures: 2, Errors: 0, Skipped: 0
As part of the change, I included `getZooKeepers()` in `WebViews.java` which
was missing in the relevant integration test.
After adding `getZooKeepers()` to `createMocks()` method (as highlighted
below) the integration test ran successfully. Let me know if you're ok with
this change? Thanks
<pre>
expect(contextMock.getInstanceID()).andReturn("foo").atLeastOnce();
expect(contextMock.getInstanceName()).andReturn("foo").anyTimes();
<b>expect(contextMock.getZooKeepers()).andReturn("foo:2181").anyTimes(); </b>
</pre>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
