[SOLR-5067] TestReplicationHandler doTestReplicateAfterWrite2Slave bad test

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Component/s: replication (java)
• Assignee: Unassigned
• Reporter: Vadim Kirilchuk
• Created: 23/Jul/13 10:16
• Updated: 23/Jul/13 10:40

Description

Hi,

TestReplicationHandler#doTestReplicateAfterWrite2Slave has some commented code which actually performs necessary assertions.

https://github.com/apache/lucene-solr/blob/trunk/solr/core/src/test/org/apache/solr/handler/TestReplicationHandler.java

While these assertions commented out it checks nothing. Also as index fetching starts in a new thread it's worth to perform fetchindex with 'wait' parameter.