[FAB-3244] Re-enable CouchDB bad connect unit test

The CouchDB test bad connection unit test was disabled
when retry logic was added.

Hard coding the maxRetriesOnStartup to 3 will allow the
test to complete without a excessive delay in overall test
execution.

Change-Id: I834ebe0b7ef6fb5859286edefa2fffac4e4249ec
Signed-off-by: Chris Elder <[email protected]>

Author: Chris Elder

core/ledger/util/couchdb/couchdb_test.go

@@ -209,15 +209,12 @@ func TestDBBadDatabaseName(t *testing.T) {
 
 func TestDBBadConnection(t *testing.T) {
 
-	// TestDBBadConnectionDef skipped since retry logic stalls the unit tests for two minutes.
-	// TODO Re-enable once configurable retry logic is introduced
-	t.Skip()
-
 	if ledgerconfig.IsCouchDBEnabled() {
 
 		//create a new instance and database object
+		//Limit the maxRetriesOnStartup to 3 in order to reduce time for the failure
 		_, err := CreateCouchInstance(badConnectURL, couchDBDef.Username, couchDBDef.Password,
-			couchDBDef.MaxRetries, couchDBDef.MaxRetriesOnStartup, couchDBDef.RequestTimeout)
+			couchDBDef.MaxRetries, 3, couchDBDef.RequestTimeout)
 		testutil.AssertError(t, err, fmt.Sprintf("Error should have been thrown for a bad connection"))
 	}
 }