From cc97785d23465dfa622a029f68e04ad3a50a24a7 Mon Sep 17 00:00:00 2001 From: cagache Date: Mon, 14 May 2018 11:07:06 +0300 Subject: [PATCH] code review changes --- .../alfresco/rest/rm/community/base/BaseRMRestTest.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/rm-automation/rm-automation-community-rest-api/src/test/java/org/alfresco/rest/rm/community/base/BaseRMRestTest.java b/rm-automation/rm-automation-community-rest-api/src/test/java/org/alfresco/rest/rm/community/base/BaseRMRestTest.java index 8610d4460d..5de8128200 100644 --- a/rm-automation/rm-automation-community-rest-api/src/test/java/org/alfresco/rest/rm/community/base/BaseRMRestTest.java +++ b/rm-automation/rm-automation-community-rest-api/src/test/java/org/alfresco/rest/rm/community/base/BaseRMRestTest.java @@ -658,7 +658,8 @@ public class BaseRMRestTest extends RestTest names.add(childNode.onModel().getName()); }); break; - } else + } + else { counter++; } @@ -696,10 +697,11 @@ public class BaseRMRestTest extends RestTest } results = searchApi.searchForRecordsAsUser(user.getUsername(), user.getPassword(), term, sortby); - if ((results != null && !results.isEmpty()) && results.containsAll(expectedResults)) + if (!results.isEmpty() && results.containsAll(expectedResults)) { break; - } else + } + else { counter++; }