CBG-4528 remove PushRevWithHistory

[torcolvin]

CBG-4528 remove PushRevWithHistory/PushUnsolicitedRev

This was used to push unsolicited rev and check failure of the message. Instead of wrapping this in blip code, send the message separately and check the actual error.

This is also an intermediate commit for full CBG-4528 but allows removing a large code pathway. The problem with PushRevWithHistory was that it called upsertDoc so an outgoing pull replication would see the changes. So this had to avoid being called at the same time as a pull replication to avoid a data race.

Additionally, the error TestBlipDeltaSyncPush was wrapped as followed fmt.Errorf("error %s %s from revResponse: %s", revResponse.Properties["Error-Domain"], revResponse.Properties["Error-Code"], rspBody) so the assertion was indirect at best since the verfication is on deltaSrc.

The test doesn't actually set the correct body for the delta but that is OK since the assertion is set on `deltaRev

Pre-review checklist

• Removed debug logging (fmt.Print, log.Print, ...)
• Logging sensitive data? Make sure it's tagged (e.g. base.UD(docID), base.MD(dbName))
• Updated relevant information in the API specifications (such as endpoint descriptions, schemas, ...) in docs/api

Integration Tests

• GSI=true,xattrs=true https://jenkins.sgwdev.com/job/SyncGateway-Integration/2968/