From 0f6423500620b48738532d6de8842ae165593892 Mon Sep 17 00:00:00 2001 From: Tal Levy Date: Fri, 14 Dec 2018 00:24:56 -0800 Subject: [PATCH] [TEST] fix SecurityDocumentationIT#testGetUsers (#36622) this test was failing because of two reason - it was creating invalid users with passwords shorter than 6 characters - it was expecting 7 total users to be returned, but it should be 9 --- .../documentation/SecurityDocumentationIT.java | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/client/rest-high-level/src/test/java/org/elasticsearch/client/documentation/SecurityDocumentationIT.java b/client/rest-high-level/src/test/java/org/elasticsearch/client/documentation/SecurityDocumentationIT.java index 87bb8b62773be..7af8b6df0a029 100644 --- a/client/rest-high-level/src/test/java/org/elasticsearch/client/documentation/SecurityDocumentationIT.java +++ b/client/rest-high-level/src/test/java/org/elasticsearch/client/documentation/SecurityDocumentationIT.java @@ -111,9 +111,9 @@ public class SecurityDocumentationIT extends ESRestHighLevelClientTestCase { public void testGetUsers() throws Exception { final RestHighLevelClient client = highLevelClient(); String[] usernames = new String[] {"user1", "user2", "user3"}; - addUser(client, usernames[0], randomAlphaOfLength(4)); - addUser(client, usernames[1], randomAlphaOfLength(4)); - addUser(client, usernames[2], randomAlphaOfLength(4)); + addUser(client, usernames[0], randomAlphaOfLengthBetween(6, 10)); + addUser(client, usernames[1], randomAlphaOfLengthBetween(6, 10)); + addUser(client, usernames[2], randomAlphaOfLengthBetween(6, 10)); { //tag::get-users-request GetUsersRequest request = new GetUsersRequest(usernames[0]); @@ -128,7 +128,7 @@ public void testGetUsers() throws Exception { assertNotNull(response); assertThat(users.size(), equalTo(1)); - assertThat(users.get(0), is(usernames[0])); + assertThat(users.get(0).getUsername(), is(usernames[0])); } { @@ -156,8 +156,10 @@ public void testGetUsers() throws Exception { List users = new ArrayList<>(3); users.addAll(response.getUsers()); assertNotNull(response); - // 4 system users plus the three we created - assertThat(users.size(), equalTo(7)); + // 9 users are expected to be returned + // test_users (3): user1, user2, user3 + // system_users (6): elastic, beats_system, apm_system, logstash_system, kibana, remote_monitoring_user + assertThat(users.size(), equalTo(9)); } {