[jspwiki] branch master updated: remove now redundant assertNull checks

[juanpablo]

This is an automated email from the ASF dual-hosted git repository.

juanpablo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jspwiki.git


The following commit(s) were added to refs/heads/master by this push:
     new 32f1d8b  remove now redundant assertNull checks
32f1d8b is described below

commit 32f1d8b995c02b754743243ee7e696f5dcc3d147
Author: juanpablo <juanpa...@apache.org>
AuthorDate: Thu Oct 10 23:34:08 2019 +0200

    remove now redundant assertNull checks
---
 .../src/test/java/org/apache/wiki/search/SearchManagerTest.java    | 7 -------
 1 file changed, 7 deletions(-)

diff --git 
a/jspwiki-main/src/test/java/org/apache/wiki/search/SearchManagerTest.java 
b/jspwiki-main/src/test/java/org/apache/wiki/search/SearchManagerTest.java
index c831d01..1f7a935 100644
--- a/jspwiki-main/src/test/java/org/apache/wiki/search/SearchManagerTest.java
+++ b/jspwiki-main/src/test/java/org/apache/wiki/search/SearchManagerTest.java
@@ -97,7 +97,6 @@ public class SearchManagerTest {
         final Collection< SearchResult > res = new ArrayList<>();
         Awaitility.await( "testSimpleSearch" ).until( findsResultsFor( res, 
"mankind" ) );
 
-        Assertions.assertNotNull( res, "null result" );
         Assertions.assertEquals( 1, res.size(), "no pages" );
         Assertions.assertEquals( "TestPage", 
res.iterator().next().getPage().getName(), "page" );
         m_engine.deleteTestPage("TestPage");
@@ -112,7 +111,6 @@ public class SearchManagerTest {
         final Collection< SearchResult > res = new ArrayList<>();
         Awaitility.await( "testSimpleSearch2" ).until( findsResultsFor( 
res,"mankind" ) );
 
-        Assertions.assertNotNull( res, "null result" );
         Assertions.assertEquals( 1, res.size(), "no pages" );
         Assertions.assertEquals( "TestPage", 
res.iterator().next().getPage().getName(), "page" );
         m_engine.deleteTestPage( "TestPage" );
@@ -131,7 +129,6 @@ public class SearchManagerTest {
         Awaitility.await( "testSimpleSearch3" ).until( findsResultsFor( res, 
"Babylon" ) );
         res = m_mgr.findPages( "mankind", ctx ); // check for text present in 
1st m_engine.saveText() but not in 2nd
 
-        Assertions.assertNotNull( res, "found results" );
         Assertions.assertEquals( 0, res.size(), "empty results" );
 
         res = m_mgr.findPages( "Babylon", ctx );
@@ -152,7 +149,6 @@ public class SearchManagerTest {
         Collection< SearchResult > res = new ArrayList<>();
         Awaitility.await( "testSimpleSearch4" ).until( findsResultsFor( res, 
"mankind" ) );
 
-        Assertions.assertNotNull( res, "found results" );
         Assertions.assertEquals( 1, res.size(), "result not found" );
 
         m_engine.saveText( ctx, "[{ALLOW view Authenticated}] It was the dawn 
of the third age of mankind... page is blocked" );
@@ -172,7 +168,6 @@ public class SearchManagerTest {
         final Collection< SearchResult > res = new ArrayList<>();
         Awaitility.await( "testTitleSearch" ).until( findsResultsFor( res, 
"Test" ) );
 
-        Assertions.assertNotNull( res, "null result" );
         Assertions.assertEquals( 1, res.size(), "no pages" );
         Assertions.assertEquals( "TestPage", 
res.iterator().next().getPage().getName(), "page" );
         m_engine.deleteTestPage("TestPage");
@@ -186,7 +181,6 @@ public class SearchManagerTest {
         final Collection< SearchResult > res = new ArrayList<>();
         Awaitility.await( "testTitleSearch2" ).until( findsResultsFor( res, 
"TestPage" ) );
 
-        Assertions.assertNotNull( res, "null result" );
         Assertions.assertEquals( 1, res.size(), "no pages" );
         Assertions.assertEquals( "TestPage", 
res.iterator().next().getPage().getName(), "page" );
         m_engine.deleteTestPage("TestPage");
@@ -201,7 +195,6 @@ public class SearchManagerTest {
         final Collection< SearchResult > res = new ArrayList<>();
         Awaitility.await( "testKeywordsSearch" ).until( findsResultsFor( res, 
"perry" ) );
 
-        Assertions.assertNotNull( res, "null result" );
         Assertions.assertEquals( 1, res.size(), "no pages" );
         Assertions.assertEquals( "TestPage", 
res.iterator().next().getPage().getName(), "page" );
         m_engine.deleteTestPage("TestPage");