remove errant log; fix test

Author: Dan Laine

indexer/client_test.go

@@ -48,7 +48,7 @@ func TestIndexClient(t *testing.T) {
 	id = ids.GenerateTestID()
 	client.EndpointRequester = &mockClient{
 		f: func(reply interface{}) error {
-			*(reply.(**GetContainerRangeResponse)) = &GetContainerRangeResponse{Containers: []FormattedContainer{{ID: id}}}
+			*(reply.(*GetContainerRangeResponse)) = GetContainerRangeResponse{Containers: []FormattedContainer{{ID: id}}}
 			return nil
 		},
 	}

indexer/indexer.go

@@ -206,7 +206,6 @@ func (i *indexer) RegisterChain(name string, ctx *snow.Context, engine common.En
 	}
 
 	if !i.allowIncompleteIndex && isIncomplete && (previouslyIndexed || i.hasRunBefore) {
-		i.log.Fatal("previously indexed: %v. has run before: %v", previouslyIndexed, i.hasRunBefore)
 		i.log.Fatal("index %s is incomplete but incomplete indices are disabled. Shutting down", name)
 		if err := i.close(); err != nil {
 			i.log.Error("error while closing indexer: %s", err)

indexer/indexer_test.go

@@ -502,10 +502,6 @@ func TestIncompleteIndex(t *testing.T) {
 	assert.NoError(err)
 	idxr, ok = idxrIntf.(*indexer)
 	assert.True(ok)
-
-	// Register the chain again. Should still die due to incomplete index.
-	idxr.RegisterChain("chain1", chain1Ctx, chainEngine)
-	assert.True(idxr.closed)
 }
 
 // Ensure we only index chains in the primary network