PDavid commented on code in PR #6571:
URL: https://github.com/apache/hbase/pull/6571#discussion_r1903761146
##########
hbase-server/src/test/java/org/apache/hadoop/hbase/http/TestInfoServersACL.java:
##########
@@ -193,7 +193,7 @@ public Void run() throws Exception {
@Test
public void testTableActionsAvailableForAdmins() throws Exception {
- final String expectedAuthorizedContent = "Actions:";
Review Comment:
The UI page does not have the colon anymore because I converted it to a HTML
heading (`<h2>Actions<h2>`).
Alternatively we could expect here the HTML tags as well (`<h2>Actions<h2>`)
but this way we are not depending on the actual HTML tag - if they change in
the future the test will still be green.
What do you think?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
